Resolving conflict with package manifest.

lib/schema.js

@@ -557,7 +557,14 @@ Schema.prototype.pre = function(){
  */
 
 Schema.prototype.post = function(method, fn){
-  return this.queue('on', arguments);
+  // assuming that all callbacks with arity < 2 are synchronous post hooks
+  if (fn.length < 2)
+    return this.queue('on', arguments);
+  return this.queue('post', [arguments[0], function(next){
+    // wrap original function so that the callback goes last,
+    // for compatibility with old code that is using synchronous post hooks
+    fn.call(this, this, next);
+  }]);
 };
 
 /**

package.json

@@ -5,7 +5,7 @@
   , "author": "Guillermo Rauch <[email protected]>"
   , "keywords": ["mongodb", "document", "model", "schema", "database", "odm", "data", "datastore", "query", "nosql", "orm", "db"]
   , "dependencies": {
-        "hooks": "0.2.1"
+        "hooks": "0.3.2"
       , "mongodb": "1.4.3"
       , "ms": "0.1.0"
       , "sliced": "0.0.5"

test/model.middleware.test.js

@@ -38,11 +38,13 @@ describe('model middleware', function(){
       called++;
     });
 
-    schema.post('save', function (obj) {
-      assert.equal(obj.title,'Little Green Running Hood');
-      assert.equal(2, called);
-      db.close();
-      done();
+    schema.post('save', function(obj, next){
+      setTimeout(function(){
+        assert.equal(obj.title,'Little Green Running Hood');
+        assert.equal(2, called);
+        called++;
+        next();
+      }, 0);
     });
 
     var db = start()
@@ -52,8 +54,12 @@ describe('model middleware', function(){
 
     test.save(function(err){
       assert.ifError(err);
+      assert.equal(test.title,'Little Green Running Hood');
+      assert.equal(3, called);
+      db.close();
+      done();
     });
-  })
+  });
 
   it('works', function(done){
     var schema = new Schema({