Make $on chainable

Author: bendrucker

angularfire.js

@@ -341,13 +341,12 @@
           self._timeout(function() {
             callback();
           });
-          return;
-        }
-        if (self._on.hasOwnProperty(type)) {
+        } else if (self._on.hasOwnProperty(type)) {
           self._on[type].push(callback);
         } else {
           throw new Error("Invalid event type " + type + " specified");
         }
+        return object;
       };
 
       // Detach an event handler from a specified event type. If no callback

angularfire.min.js

@@ -126,6 +126,14 @@ casper.then(function() {
   }, "Testing if $add, $set, $save and $remove return a promise");
 });
 
+casper.then(function() {
+  this.test.assertEval(function() {
+    var empty = function() {};
+    var obj = _scope.messages.$on('loaded', empty).$on('change', empty);
+    return JSON.stringify(_scope.messages) == JSON.stringify(obj);
+  }, "Testing chaining of $on method");
+});
+
 casper.run(function() {
   this.test.done();
 });