---

schmidt-sebastian · web-flow · commit 35e513486b43 · 2017-11-09T16:07:44.000-08:00
yaml --- r: 8487 b: refs/heads/master c: 7c93395 h: refs/heads/master i: 8485: b6e0e17 8483: fdde1a2 8479: 3ccbaaa
diff --git a/[refs] b/[refs]
@@ -1,5 +1,5 @@
 ---
-refs/heads/master: 3ed6862a05e886e4de632d10dd555b201b5cfcca
+refs/heads/master: 7c93395874187c4502e078679e04eb15402bde0e
 refs/heads/travis: 47e4fee4fd5af9b2a8ce46f23c72ec95f9b195b2
 refs/heads/gh-pages: 3e16a39145437096333db5811e5c0292719c1823
 refs/tags/0.0.9: 22f1839238f66c39e67ed4dfdcd273b1ae2e8444
diff --git a/trunk/google-cloud-firestore/src/main/java/com/google/cloud/firestore/BasePath.java b/trunk/google-cloud-firestore/src/main/java/com/google/cloud/firestore/BasePath.java
@@ -96,4 +96,13 @@ boolean isPrefixOf(B path) {
   abstract String[] splitChildPath(String path);
 
   abstract B createPathWithSegments(ImmutableList<String> segments);
+
+  boolean isPrefixOf(BasePath<B> field) {
+    if (field.getSegments().size() < this.getSegments().size()) {
+      return false;
+    }
+
+    return this.equals(
+        createPathWithSegments(field.getSegments().subList(0, this.getSegments().size())));
+  }
 }
diff --git a/trunk/google-cloud-firestore/src/main/java/com/google/cloud/firestore/FieldPath.java b/trunk/google-cloud-firestore/src/main/java/com/google/cloud/firestore/FieldPath.java
@@ -28,7 +28,7 @@
  * field in the document).
  */
 @AutoValue
-public abstract class FieldPath extends BasePath<FieldPath> {
+public abstract class FieldPath extends BasePath<FieldPath> implements Comparable<FieldPath> {
 
   /**
    * A special sentinel FieldPath to refer to the ID of a document. It can be used in queries to
@@ -152,4 +152,9 @@ FieldPath createPathWithSegments(ImmutableList<String> segments) {
   public String toString() {
     return getEncodedPath();
   }
+
+  @Override
+  public int compareTo(@Nonnull FieldPath other) {
+    return getEncodedPath().compareTo(other.getEncodedPath());
+  }
 }
diff --git a/trunk/google-cloud-firestore/src/main/java/com/google/cloud/firestore/UpdateBuilder.java b/trunk/google-cloud-firestore/src/main/java/com/google/cloud/firestore/UpdateBuilder.java
@@ -32,6 +32,8 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Map.Entry;
+import java.util.SortedSet;
+import java.util.TreeSet;
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 
@@ -56,10 +58,18 @@ abstract class UpdateBuilder<T extends UpdateBuilder> {
   private static Map<String, Object> expandObject(Map<FieldPath, Object> data) {
     Map<String, Object> result = new HashMap<>();
 
-    for (Map.Entry<FieldPath, Object> entrySet : data.entrySet()) {
-      List<String> segments = entrySet.getKey().getSegments();
-      Object value = entrySet.getValue();
+    SortedSet<FieldPath> sortedFields = new TreeSet<>(data.keySet());
 
+    FieldPath lastField = null;
+
+    for (FieldPath field : sortedFields) {
+      if (lastField != null && lastField.isPrefixOf(field)) {
+        throw new IllegalArgumentException(
+            String.format("Detected ambiguous definition for field '%s'.", lastField));
+      }
+
+      List<String> segments = field.getSegments();
+      Object value = data.get(field);
       Map<String, Object> currentMap = result;
 
       for (int i = 0; i < segments.size(); ++i) {
@@ -69,9 +79,12 @@ private static Map<String, Object> expandObject(Map<FieldPath, Object> data) {
           if (!currentMap.containsKey(segments.get(i))) {
             currentMap.put(segments.get(i), new HashMap<>());
           }
+
           currentMap = (Map<String, Object>) currentMap.get(segments.get(i));
         }
       }
+
+      lastField = field;
     }
 
     return result;
diff --git a/trunk/google-cloud-firestore/src/test/java/com/google/cloud/firestore/DocumentReferenceTest.java b/trunk/google-cloud-firestore/src/test/java/com/google/cloud/firestore/DocumentReferenceTest.java
@@ -584,6 +584,45 @@ public void updateWithDotsInFieldName() throws Exception {
     assertCommitEquals(expectedCommit, commitCapture.getValue());
   }
 
+  @Test
+  public void updateConflictingFields() throws Exception {
+   try {
+     documentReference
+         .update("a.b", "foo", "a", "foo")
+         .get();
+     fail();
+   } catch (IllegalArgumentException e) {
+      assertEquals(e.getMessage(), "Detected ambiguous definition for field 'a'.");
+   }
+
+    try {
+      documentReference
+          .update("a.b", "foo", "a.b.c", "foo")
+          .get();
+      fail();
+    } catch (IllegalArgumentException e) {
+      assertEquals(e.getMessage(), "Detected ambiguous definition for field 'a.b'.");
+    }
+
+    try {
+      documentReference
+          .update("a.b", SINGLE_FIELD_MAP, "a", SINGLE_FIELD_MAP)
+          .get();
+      fail();
+    } catch (IllegalArgumentException e) {
+      assertEquals(e.getMessage(), "Detected ambiguous definition for field 'a'.");
+    }
+
+    try {
+      documentReference
+          .update("a.b", SINGLE_FIELD_MAP, "a.b.c", SINGLE_FIELD_MAP)
+          .get();
+      fail();
+    } catch (IllegalArgumentException e) {
+      assertEquals(e.getMessage(), "Detected ambiguous definition for field 'a.b'.");
+    }
+  }
+
   @Test
   public void deleteField() throws Exception {
     doReturn(SINGLE_WRITE_COMMIT_RESPONSE)
