base - destroy only once

korgx9 · Jun 19, 2012 · 0e5d682 · 0e5d682
1 parent 060d78a
commit 0e5d682
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/spec/BaseSpec.js b/spec/BaseSpec.js
@@ -39,6 +39,10 @@ describe("Base", function () {
     var base = new Base(eventManager);
     spyOn(eventManager, 'fireEvent');
     base.hit();
-    expect(eventManager.fireEvent).toHaveBeenCalledWith({'name': Base.Event.HIT, 'base': base});
+    var EVENT = {'name': Base.Event.HIT, 'base': base};
+    expect(eventManager.fireEvent).toHaveBeenCalledWith(EVENT);
+    eventManager.fireEvent.reset();
+    base.hit();
+    expect(eventManager.fireEvent).not.toHaveBeenCalledWith(EVENT);
   });
 });
diff --git a/src/Base.js b/src/Base.js
@@ -38,6 +38,9 @@ Base.prototype.notify = function (event) {
 };
 
 Base.prototype.hit = function () {
+  if (this._hit) {
+    return;
+  }
   this._hit = true;
   this._eventManager.fireEvent({'name': Base.Event.HIT, 'base': this});
 };