feat(ngModel): pass the previous and current model to view change listeners

Author: luaks

src/ng/directive/ngModel.js

@@ -547,7 +547,7 @@ var NgModelController = ['$scope', '$exceptionHandler', '$attrs', '$element', '$
         ctrl.$modelValue = allValid ? modelValue : undefined;
 
         if (ctrl.$modelValue !== prevModelValue) {
-          ctrl.$$writeModelToScope();
+          ctrl.$$writeModelToScope(ctrl.$modelValue, prevModelValue);
         }
       }
     });
@@ -719,16 +719,16 @@ var NgModelController = ['$scope', '$exceptionHandler', '$attrs', '$element', '$
 
     function writeToModelIfNeeded() {
       if (ctrl.$modelValue !== prevModelValue) {
-        ctrl.$$writeModelToScope();
+        ctrl.$$writeModelToScope(ctrl.$modelValue, prevModelValue);
       }
     }
   };
 
-  this.$$writeModelToScope = function() {
+  this.$$writeModelToScope = function(currentModelValue, previousModelValue) {
     ngModelSet($scope, ctrl.$modelValue);
     forEach(ctrl.$viewChangeListeners, function(listener) {
       try {
-        listener();
+        listener(currentModelValue, previousModelValue);
       } catch (e) {
         $exceptionHandler(e);
       }

test/ng/directive/ngModelSpec.js

@@ -229,6 +229,17 @@ describe('ngModel', function() {
       });
 
 
+      it('should pass the previous view value to the change listener', function() {
+        ctrl.$setViewValue('val');
+
+        var spy = jasmine.createSpy('viewChangeListener');
+        ctrl.$viewChangeListeners.push(spy);
+        ctrl.$setViewValue('val2');
+
+        expect(spy).toHaveBeenCalledWith('val2', 'val');
+      });
+
+
       it('should reset the model when the view is invalid', function() {
         ctrl.$setViewValue('aaaa');
         expect(ctrl.$modelValue).toBe('aaaa');