allow unsetting document arrays

Automattic · Feb 24, 2012 · b30e128 · b30e128
1 parent f3f3028
commit b30e128
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/schema/documentarray.js b/lib/schema/documentarray.js
@@ -69,7 +69,7 @@ DocumentArray.prototype.doValidate = function (array, fn, scope) {
   SchemaType.prototype.doValidate.call(this, array, function(err){
     if (err) return fn(err);
 
-    var count = array.length
+    var count = array && array.length
       , error = false;
 
     if (!count) return fn();

diff --git a/test/model.querying.test.js b/test/model.querying.test.js
@@ -1286,12 +1286,14 @@ module.exports = {
             b.comments[1].body.should.equal('cranberries');
 
             b.meta = undefined;
+            b.comments = undefined;
             b.save(function (err) {
               should.strictEqual(null, err);
               B.collection.findOne({ _id: b._id}, function (err, b) {
                 db.close();
                 should.strictEqual(null, err);
                 should.strictEqual(undefined, b.meta);
+                should.strictEqual(undefined, b.comments);
               });
             });
           });