Added field options for zone create method.

mderka · mderka · commit 3183f4a7445d · 2016-02-04T14:04:45.000-08:00
diff --git a/gcloud-java-dns/src/main/java/com/google/gcloud/dns/Dns.java b/gcloud-java-dns/src/main/java/com/google/gcloud/dns/Dns.java
@@ -421,14 +421,15 @@ public static ChangeRequestListOption sortOrder(SortingOrder order) {
    *
    * <p>Returns {@link ZoneInfo} object representing the new zone's information. In addition to the
    * name, dns name and description (supplied by the user within the {@code zoneInfo} parameter),
-   * the returned object will include the following read-only fields supplied by the server:
-   * creation time, id, and list of name servers.
+   * the returned object can include the following read-only fields supplied by the server: creation
+   * time, id, and list of name servers. The returned fields can be optionally restricted by
+   * specifying {@link ZoneOption}s.
    *
    * @throws DnsException upon failure
    * @see <a href="https://cloud.google.com/dns/api/v1/managedZones/create">Cloud DNS Managed Zones:
    * create</a>
    */
-  ZoneInfo create(ZoneInfo zoneInfo);
+  ZoneInfo create(ZoneInfo zoneInfo, ZoneOption... options);
 
   /**
    * Returns the zone by the specified zone name. Returns {@code null} if the zone is not found. The
diff --git a/gcloud-java-dns/src/main/java/com/google/gcloud/spi/DefaultDnsRpc.java b/gcloud-java-dns/src/main/java/com/google/gcloud/spi/DefaultDnsRpc.java
@@ -54,9 +54,12 @@ public DefaultDnsRpc(DnsOptions options) {
   }
 
   @Override
-  public ManagedZone create(ManagedZone zone) throws DnsException {
+  public ManagedZone create(ManagedZone zone, Map<Option, ?> options) throws DnsException {
     try {
-      return dns.managedZones().create(this.options.projectId(), zone).execute();
+      return dns.managedZones()
+          .create(this.options.projectId(), zone)
+          .setFields(FIELDS.getString(options))
+          .execute();
     } catch (IOException ex) {
       throw translate(ex);
     }
diff --git a/gcloud-java-dns/src/main/java/com/google/gcloud/spi/DnsRpc.java b/gcloud-java-dns/src/main/java/com/google/gcloud/spi/DnsRpc.java
@@ -87,10 +87,11 @@ public String pageToken() {
    * Creates a new zone.
    *
    * @param zone a zone to be created
+   * @param options a map of options for the service call
    * @return Updated {@code ManagedZone} object
    * @throws DnsException upon failure
    */
-  ManagedZone create(ManagedZone zone) throws DnsException;
+  ManagedZone create(ManagedZone zone, Map<Option, ?> options) throws DnsException;
 
   /**
    * Retrieves and returns an existing zone.

Original file line number	Diff line number	Diff line change
`@@ -54,9 +54,12 @@ public DefaultDnsRpc(DnsOptions options) {`
`54`	`54`	`}`
`55`	`55`
`56`	`56`	`@Override`
`57`		`- public ManagedZone create(ManagedZone zone) throws DnsException {`
	`57`	`+ public ManagedZone create(ManagedZone zone, Map<Option, ?> options) throws DnsException {`
`58`	`58`	`try {`
`59`		`- return dns.managedZones().create(this.options.projectId(), zone).execute();`
	`59`	`+ return dns.managedZones()`
	`60`	`+ .create(this.options.projectId(), zone)`
	`61`	`+ .setFields(FIELDS.getString(options))`
	`62`	`+ .execute();`
`60`	`63`	`} catch (IOException ex) {`
`61`	`64`	`throw translate(ex);`
`62`	`65`	`}`