Adding DocumentSnapshot support for Cursors (#100)

googleapis · Jan 12, 2018 · 563f8d0 · 563f8d0
1 parent 65d904a
commit 563f8d0
Show file tree

Hide file tree

Showing 6 changed files with 561 additions and 67 deletions.
diff --git a/src/path.js b/src/path.js
@@ -394,11 +394,20 @@ class ResourcePath extends Path {
    * @returns {number} -1 if current < other, 1 if current > other, 0 if equal
    */
   compareTo(other) {
-    if (this._projectId < other._projectId) {
-      return -1;
-    }
-    if (this._projectId > other._projectId) {
-      return 1;
+    // Ignore DocumentReference with {{projectId}} placeholders and assume that
+    // the resolved IDs match the provided ResourcePath. We could alternatively
+    // try to resolve the Project ID here, but this is asynchronous as it
+    // requires Disk I/O.
+    if (
+      this._projectId !== '{{projectId}}' &&
+      other._projectId !== '{{projectId}}'
+    ) {
+      if (this._projectId < other._projectId) {
+        return -1;
+      }
+      if (this._projectId > other._projectId) {
+        return 1;
+      }
     }
 
     if (this._databaseId < other._databaseId) {