Merge pull request #23 from platogo/fix-unbind-with-undefined-callback

Do not remove event listener on unbind without callback

asevented.js

@@ -55,7 +55,7 @@
       parts = event.split(/\s+/);
       for (i = 0, num = parts.length; i < num; i++) {
         if ((eventName = parts[i]) in events !== false) {
-          index = (fn) ? _indexOf(events[eventName], fn) : 0;
+          index = (fn) ? _indexOf(events[eventName], fn) : -1;
           if (index !== -1) {
             events[eventName].splice(index, 1);
           }

test/asevented_test.js

@@ -36,7 +36,7 @@ $(function() {
     var callback = function () { obj.counter += 1; };
     obj.bind('event', callback);
     obj.trigger('event');
-    obj.unbind('event');
+    obj.unbind('event', callback);
     obj.trigger('event');
     equals(obj.counter, 1, 'counter should have only been incremented once.');
   });
@@ -226,7 +226,7 @@ $(function() {
     asEvented.call(obj);
 
     obj.bind('load ready whatever', callback);
-    obj.unbind('ready');
+    obj.unbind('ready', callback);
 
     obj.trigger('load');
     obj.trigger('ready');
@@ -241,7 +241,7 @@ $(function() {
     asEvented.call(obj);
 
     obj.bind('load ready whatever', callback);
-    obj.unbind('ready load whatever');
+    obj.unbind('ready load whatever', callback);
 
     obj.trigger('load');
     obj.trigger('ready');
@@ -263,4 +263,16 @@ $(function() {
     equals(obj.count, 1, 'obj.count should have been incremented once.');
   });
 
+  test('unbind without callback', function() {
+    var obj = { count: 0 };
+    var callback = function() { obj.count += 1; };
+    asEvented.call(obj);
+
+    obj.bind('whatever', callback);
+    obj.trigger('whatever');
+    obj.unbind('whatever', undefined);
+    obj.trigger('whatever');
+
+    equals(obj.count, 2, 'obj.count should have been incremented twice.');
+  });
 });