[Live Query] `matchesKeyConstraints` throwing error when `constraints` is `{$ne : null}`

[RahulLanjewar93]

New Issue Checklist

• [ x] I am not disclosing a vulnerability.
• [ x] I am not just asking a question.
• [ x] I have searched through existing issues.
• [ x] I can reproduce the issue with the latest version of Parse Server.

Issue Description

matchesKeyConstraints function is not handling not equal to null constraint properly. Due to which error is thrown `Cannot read properties of null (reading '__type')

Steps to reproduce

Create a subscription on a class with constraint notEqualTo(key,null)

Actual Outcome

Throws error.

Expected Outcome

Shouldn't throw an error.

Environment

18.19.0

Server

• Parse Server version: 5.5.5
• Operating system: Mac
• Local or remote host (AWS, Azure, Google Cloud, Heroku, Digital Ocean, etc): self hosted

Database

• System (MongoDB or Postgres): Mongo
• Local or remote host (MongoDB Atlas, mLab, AWS, Azure, Google Cloud, etc): Local

Client

• SDK (iOS, Android, JavaScript, PHP, Unity, etc): Javascript
• SDK version: 4.0.1

Logs

[parse-github-assistant[bot]]

Thanks for opening this issue!

• 🚀 You can help us to fix this issue faster by opening a pull request with a failing test. See our Contribution Guide for how to make a pull request, or read our New Contributor's Guide if this is your first time contributing.

[RahulLanjewar93]

@mtrezza i think we need some changes in the code block

QueryTools.js > matchesKeyConstraints

here we need to check if compareTo is not null/undefined

[RahulLanjewar93]

@mtrezza please have a look

[mtrezza]

Will this issue be fixed by https://github.com/parse-community/Parse-SDK-JS/pull/2102 once the Parse SDK is upgraded in Parse Server? @dplewis

[dplewis]

@mtrezza Thats a fix for the Local Datastore this issue is for LiveQuery. The query engines for both are almost the same. You can basically copy the fix over to Parse Server and it should work for Live Query.

[mtrezza]

@RahulLanjewar93 do you want to give it a try?

[RahulLanjewar93]

@RahulLanjewar93 do you want to give it a try?

sure!

[RahulLanjewar93]

@RahulLanjewar93 do you want to give it a try?

added required change in #8835

[parseplatformorg]

🎉 This change has been released in version 7.1.0-alpha.10

[parseplatformorg]

🎉 This change has been released in version 7.1.0-beta.1

[parseplatformorg]

🎉 This change has been released in version 7.1.0