commit 978234238abc5f2a8771dffca054feb3a22dfb3c
author Jeremy Daggett <jeremy.daggett@rackspace.com> Wed Aug 27 16:44:49 2014 -0700
committer Jeremy Daggett <jeremy.daggett@rackspace.com> Thu Sep 04 10:01:36 2014 -0700
tree 1b221025702e4b0c4f32d2d6ed5c2bbf27f67a57
parent e7a52c1aecef2f989f9bde57c1b83e97a81538f9

JCLOUDS-686: Glance API ListImageOptions should support changesSince()

openstack-glance/src/main/java/org/jclouds/openstack/glance/v1_0/options/ListImageOptions.java

diff --git a/openstack-glance/src/main/java/org/jclouds/openstack/glance/v1_0/options/ListImageOptions.java b/openstack-glance/src/main/java/org/jclouds/openstack/glance/v1_0/options/ListImageOptions.java
index 2e69f88..22e8697 100644
--- a/openstack-glance/src/main/java/org/jclouds/openstack/glance/v1_0/options/ListImageOptions.java
+++ b/openstack-glance/src/main/java/org/jclouds/openstack/glance/v1_0/options/ListImageOptions.java
@@ -16,6 +16,7 @@
  */
 package org.jclouds.openstack.glance.v1_0.options;
 
+import static com.google.common.base.Preconditions.checkNotNull;
 import static org.jclouds.openstack.glance.v1_0.options.ImageField.CONTAINER_FORMAT;
 import static org.jclouds.openstack.glance.v1_0.options.ImageField.DISK_FORMAT;
 import static org.jclouds.openstack.glance.v1_0.options.ImageField.IS_PUBLIC;
@@ -45,8 +46,7 @@
  * // this will list the first 10 images with the name "name", minimum required disk of 5GB.
  * list = api.list(name("newName"), limit(10), minDisk(5));
  * <code>
- * 
- * @see <a href="http://glance.openstack.org/glanceapi.html"/>
+ *
  */
 public class ListImageOptions extends PaginationOptions {
    public static final ListImageOptions NONE = new ListImageOptions();
@@ -54,6 +54,25 @@
    /**
     * Return only those images having a matching name attribute
     */
+   public ListImageOptions changesSince(Date ifModifiedSince) {
+      queryParameters.put("changes-since",
+            checkNotNull(ifModifiedSince, "modification date should not be null").getTime() / 1000 + "");
+      return this;
+   }
+
+   @Override
+   public ListImageOptions marker(String marker) {
+      return ListImageOptions.class.cast(super.marker(marker));
+   }
+
+   @Override
+   public ListImageOptions limit(int limit) {
+      return ListImageOptions.class.cast(super.limit(limit));
+   }
+
+   /**
+    * Return only those images having a matching name attribute
+    */
    public ListImageOptions name(String name) {
       queryParameters.put(NAME.asParam(), name);
       return this;
@@ -250,18 +269,4 @@
       }
    }
 
-   @Override
-   public ListImageOptions changesSince(Date ifModifiedSince) {
-      return ListImageOptions.class.cast(super.changesSince(ifModifiedSince));
-   }
-
-   @Override
-   public ListImageOptions marker(String marker) {
-      return ListImageOptions.class.cast(super.marker(marker));
-   }
-
-   @Override
-   public ListImageOptions limit(int limit) {
-      return ListImageOptions.class.cast(super.limit(limit));
-   }
 }