fix: LiveQuery throws on create event using Parse.Query.containedIn with null value in array or using Parse.Query.equalTo for Parse.Object with key value null

spec/ParseLiveQuery.spec.js

@@ -1308,4 +1308,59 @@ describe('ParseLiveQuery', function () {
     await new Promise(resolve => setTimeout(resolve, 100));
     expect(createSpy).toHaveBeenCalledTimes(1);
   });
+
+  it('Live query should work if needle is ParsePointer and haystack is any[], checking QueryTools.js>contains', async () => {
+    await reconfigureServer({
+      liveQuery: {
+        classNames: ['TestObject'],
+      },
+      startLiveQueryServer: true,
+      verbose: false,
+      silent: true,
+    });
+
+    const child1 = new Parse.Object('Child');
+    await child1.save();
+    const child2 = new Parse.Object('Child');
+    await child2.save();
+    const child3 = new Parse.Object('Child');
+    await child3.save();
+
+    const query = new Parse.Query(TestObject);
+    query.containedIn('childs', [child1, child2, null]);
+
+    const subscription = await query.subscribe();
+    subscription.on('create', () => {});
+
+    // Do not need any expect block, just make sure that the server doesn't crash or throw error
+    const object1 = new TestObject();
+    object1.set('childs', [child3]);
+    await object1.save();
+  });
+
+
+  it('Live query should work if we set equalTo(someKey,someParseObject) and new Parse object is created but someKey = null', async () => {
+    await reconfigureServer({
+      liveQuery: {
+        classNames: ['TestObject'],
+      },
+      startLiveQueryServer: true,
+      verbose: false,
+      silent: true,
+    });
+
+    const child = new Parse.Object('Child');
+    await child.save();
+
+    const query = new Parse.Query(TestObject);
+    query.equalTo('child', child);
+
+    const subscription = await query.subscribe();
+    subscription.on('create', () => {});
+
+    // Do not need any expect block, just make sure that the server doesn't crash or throw error
+    const object1 = new TestObject();
+    object1.set('child', null);
+    await object1.save();
+  });
 });

src/LiveQuery/QueryTools.js

@@ -99,7 +99,7 @@ function contains(haystack: Array, needle: any): boolean {
       if (typeof ptr === 'string' && ptr === needle.objectId) {
         return true;
       }
-      if (ptr.className === needle.className && ptr.objectId === needle.objectId) {
+      if (ptr && ptr.className === needle.className && ptr.objectId === needle.objectId) {
         return true;
       }
     }
@@ -212,6 +212,7 @@ function matchesKeyConstraints(object, key, constraints) {
       return equalObjectsGeneric(object[key], constraints, function (obj, ptr) {
         return (
           typeof obj !== 'undefined' &&
+          obj !== null &&
           ptr.className === obj.className &&
           ptr.objectId === obj.objectId
         );