Fixes #368 - trigger $watch event on $save

Modified $FirebaseObject's watch `event` to be "value"

Author: katowulf

src/FirebaseArray.js

@@ -102,10 +102,15 @@
          */
         $save: function(indexOrItem) {
           this._assertNotDestroyed('$save');
-          var item = this._resolveItem(indexOrItem);
-          var key = this.$keyAt(item);
+          var self = this;
+          var item = self._resolveItem(indexOrItem);
+          var key = self.$keyAt(item);
           if( key !== null ) {
-            return this.$inst().$set(key, $firebaseUtils.toJSON(item));
+            return self.$inst().$set(key, $firebaseUtils.toJSON(item))
+              .then(function(ref) {
+                self._notify('child_changed', key);
+                return ref;
+              })
           }
           else {
             return $firebaseUtils.reject('Invalid record; could determine its key: '+indexOrItem);

src/FirebaseObject.js

@@ -59,7 +59,7 @@
             }
             // be sure to do this after setting up data and init state
             angular.forEach(self.$$conf.listeners, function (parts) {
-              parts[0].call(parts[1], {event: 'updated', key: self.$id});
+              parts[0].call(parts[1], {event: 'value', key: self.$id});
             });
           }
         };
@@ -74,7 +74,12 @@
          * @returns a promise which will resolve after the save is completed.
          */
         $save: function () {
-          return this.$inst().$set($firebaseUtils.toJSON(this));
+          var notify = this.$$conf.notify;
+          return this.$inst().$set($firebaseUtils.toJSON(this))
+            .then(function(ref) {
+              notify();
+              return ref;
+            });
         },
 
         /**

tests/unit/FirebaseArray.spec.js

@@ -203,6 +203,16 @@ describe('$FirebaseArray', function () {
         arr.$save(0);
       }).toThrowError(Error);
     });
+
+    it('should trigger watch event', function() {
+      var spy = jasmine.createSpy('$watch');
+      arr.$watch(spy);
+      var key = arr.$keyAt(1);
+      arr[1].foo = 'watchtest';
+      arr.$save(1);
+      flushAll();
+      expect(spy).toHaveBeenCalledWith(jasmine.objectContaining({event: 'child_changed', key: key}));
+    });
   });
 
   describe('$remove', function() {

tests/unit/FirebaseObject.spec.js

@@ -57,6 +57,15 @@ describe('$FirebaseObject', function() {
       expect(whiteSpy).not.toHaveBeenCalled();
       expect(blackSpy).toHaveBeenCalledWith('test_fail');
     });
+
+    it('should trigger watch event', function() {
+      var spy = jasmine.createSpy('$watch');
+      obj.$watch(spy);
+      obj.foo = 'watchtest';
+      obj.$save();
+      flushAll();
+      expect(spy).toHaveBeenCalledWith(jasmine.objectContaining({event: 'value', key: obj.$id}));
+    });
   });
 
   describe('$loaded', function () {