fixed; use populate filters on single OIDs too

Automattic · Oct 5, 2011 · 479ec62 · 479ec62
1 parent 3f4c26f
commit 479ec62
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/lib/model.js b/lib/model.js
@@ -144,7 +144,9 @@ Model.prototype._populate = function populate (schema, oid, options, fn) {
       fn(null, oid);
     }
   } else {
-    this.model(schema.options.ref).findById(oid, options.fields, fn)
+    options.filters || (options.filters = {});
+    options.filters._id = oid;
+    this.model(schema.options.ref).findOne(options.filters, options.fields, fn)
   }
 };
 

diff --git a/test/model.ref.test.js b/test/model.ref.test.js
@@ -99,7 +99,7 @@ module.exports = {
     var origFind = User.findById;
 
     // mock an error
-    User.findById = function () {
+    User.findOne = function () {
       var args = Array.prototype.map.call(arguments, function (arg) {
         return 'function' == typeof arg ? function () {
           arg(new Error('woot'));