deep copy

Author: dblythy

spec/ParseLiveQueryServer.spec.js

@@ -776,14 +776,6 @@ describe('ParseLiveQueryServer', function () {
 
     await addMockSubscription(parseLiveQueryServer, clientId, requestId2);
 
-    parseLiveQueryServer._matchesSubscription = function (parseObject, subscription) {
-      if (!parseObject) {
-        return false;
-      }
-      subscription.query.date = { $exists: true };
-      const matchesQuery = require('../lib/LiveQuery/QueryTools').matchesQuery;
-      return matchesQuery(parseObject, subscription.query);
-    };
     parseLiveQueryServer._matchesACL = function () {
       return Promise.resolve(true);
     };

src/LiveQuery/ParseLiveQueryServer.js

@@ -24,6 +24,7 @@ import UserRouter from '../Routers/UsersRouter';
 import DatabaseController from '../Controllers/DatabaseController';
 import { isDeepStrictEqual } from 'util';
 import Deprecator from '../Deprecator/Deprecator';
+import deepcopy from 'deepcopy';
 
 class ParseLiveQueryServer {
   clients: Map;
@@ -496,7 +497,7 @@ class ParseLiveQueryServer {
     if (!parseObject) {
       return false;
     }
-    return matchesQuery(parseObject, subscription.query);
+    return matchesQuery(deepcopy(parseObject), subscription.query);
   }
 
   async _clearCachedRoles(userId: string) {

src/LiveQuery/QueryTools.js

@@ -1,7 +1,6 @@
 var equalObjects = require('./equalObjects');
 var Id = require('./Id');
 var Parse = require('parse/node');
-const deepcopy = require('deepcopy');
 
 /**
  * Query Hashes are deterministic hashes for Parse Queries.
@@ -125,7 +124,6 @@ function contains(haystack: Array, needle: any): boolean {
  * queries, we can avoid building a full-blown query tool.
  */
 function matchesQuery(object: any, query: any): boolean {
-  object = deepcopy(object);
   if (query instanceof Parse.Query) {
     var className = object.id instanceof Id ? object.id.className : object.className;
     if (className !== query.className) {