use hasOwnProp

caramboleyo · Sep 24, 2021 · 028bc73 · 028bc73
1 parent 11a33e0
commit 028bc73
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/core.ts b/src/core.ts
@@ -347,7 +347,7 @@ const createNamedOperation = (
 
 export const containsOperation = (query: any, options: Options) => {
   for (const key in query) {
-    if (options.operations[key]) return true;
+    if (options.operations.hasOwnProperty(key)) return true;
   }
   return false;
 };
@@ -426,7 +426,7 @@ const createQueryOperations = (
     return [selfOperations, nestedOperations];
   }
   for (const key in query) {
-    if (options.operations[key]) {
+    if (options.operations.hasOwnProperty(key)) {
       const op = createNamedOperation(key, query[key], query, options);
 
       if (op) {

diff --git a/test/basic-test.js b/test/basic-test.js
@@ -536,6 +536,8 @@ describe(__filename + "#", function() {
   });
 
   it("Throws error if operations are mixed with props", () => {
+    sift({ _id: { $in: [new ObjectID("610b6bc9e29dbd1bb5f045bf")] } });
+
     const test = sift({
       _id: { $in: [new ObjectID("610b6bc9e29dbd1bb5f045bf")] }
     });