Allow $exists operator to work on arrays

Fix #581
louischatriot · Mar 11, 2019 · 3397b58 · 3397b58
1 parent aa3302e
commit 3397b58
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/lib/model.js b/lib/model.js
@@ -634,7 +634,7 @@ comparisonFunctions.$exists = function (value, exists) {
   }
 
   if (value === undefined) {
-    return !exists
+    return !exists;
   } else {
     return exists;
   }
@@ -661,6 +661,7 @@ comparisonFunctions.$elemMatch = function (obj, value) {
 };
 arrayComparisonFunctions.$size = true;
 arrayComparisonFunctions.$elemMatch = true;
+arrayComparisonFunctions.$exists = true;
 
 
 /**

diff --git a/test/model.test.js b/test/model.test.js
@@ -1329,6 +1329,20 @@ describe('Model', function () {
         model.match({ childrens: [ { name: "Huey", age: 3 }, { name: "Dewey", age: 7 }, { name: "Louie", age: 12 } ] }, { "childrens": { $elemMatch: { name: "Dewey", age: { $gt: 6, $lt: 7 } } } }).should.equal(false);
         model.match({ childrens: [ { name: "Huey", age: 3 }, { name: "Dewey", age: 7 }, { name: "Louie", age: 12 } ] }, { "childrens": { $elemMatch: { name: "Louie", age: { $gt: 6, $lte: 7 } } } }).should.equal(false);
       });
+
+      it('$exists', function () {
+        model.match({ a: [5, 6] }, { a: { $exists: true } }).should.equal(true);
+
+        model.match({ a: [5] }, { a: { $exists: true } }).should.equal(true);
+        model.match({ a: [5] }, { b: { $exists: false } }).should.equal(true);
+
+        model.match({ a: [0] }, { a: { $exists: true } }).should.equal(true);
+        model.match({ a: [null] }, { a: { $exists: true } }).should.equal(true);
+        model.match({ a: [undefined] }, { a: { $exists: true } }).should.equal(true);
+
+        model.match({ a: [] }, { a: { $exists: false } }).should.equal(false);
+        model.match({ a: [] }, { a: { $exists: true } }).should.equal(true);
+      });
     });