fix: in to include

ishowta · Feb 15, 2022 · d2d4f7e · d2d4f7e
1 parent dafa6bf
commit d2d4f7e
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/src/GraphQueryListener.ts b/src/GraphQueryListener.ts
@@ -275,8 +275,8 @@ export class GraphQueryListener {
     const newSnapshotKeys = Object.keys(newSnapshot);
     for (const snapshotKey of union(prevSnapshotKeys, newSnapshotKeys)) {
       if (
-        snapshotKey in prevSnapshotKeys &&
-        !(snapshotKey in newSnapshotKeys)
+        prevSnapshotKeys.includes(snapshotKey) &&
+        !newSnapshotKeys.includes(snapshotKey)
       ) {
         // key removed
         this.logger.debug('key removed');
@@ -291,8 +291,8 @@ export class GraphQueryListener {
         }
       }
       if (
-        !(snapshotKey in prevSnapshotKeys) &&
-        snapshotKey in newSnapshotKeys
+        !prevSnapshotKeys.includes(snapshotKey) &&
+        newSnapshotKeys.includes(snapshotKey)
       ) {
         // key created
         this.logger.debug('key created');
@@ -305,7 +305,10 @@ export class GraphQueryListener {
           this.createSubQueryListener(newSnapshot, snapshotKey, newQuery);
         }
       }
-      if (snapshotKey in prevSnapshotKeys && snapshotKey in newSnapshotKeys) {
+      if (
+        prevSnapshotKeys.includes(snapshotKey) &&
+        newSnapshotKeys.includes(snapshotKey)
+      ) {
         // key not changed
         if (!dryRun && this.result) {
           this.result['data'][snapshotKey] = newSnapshot.data[snapshotKey];