add constructor "destroy" event

component · Nov 4, 2012 · c389b5b · c389b5b
1 parent d945526
commit c389b5b
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/lib/proto.js b/lib/proto.js
@@ -118,6 +118,7 @@ exports.validate = function(){
  * Events:
  *
  *  - `destroy` on deletion
+ *  - `destroy` (model) on the constructor
  *
  * @param {Function} [fn]
  * @api public
@@ -132,8 +133,9 @@ exports.destroy = function(fn){
   this.emit('destroying');
   request.del(url, function(res){
     if (res.error) return fn(error(res));
-    self.emit('destroy');
     self.destroyed = true;
+    self.emit('destroy');
+    self.model.emit('destroy', self);
     fn();
   });
 };

diff --git a/test/model.js b/test/model.js
@@ -162,6 +162,18 @@ describe('Model#destroy()', function(){
         pet.destroy();
       });
     })
+
+    it('should emit "destroy" on the constructor', function(done){
+      var pet = new Pet({ name: 'Tobi' });
+      pet.save(function(err){
+        assert(!err);
+        Pet.once('destroy', function(obj){
+          assert(pet == obj);
+          done();
+        });
+        pet.destroy();
+      });
+    })
   })
 })