Add capability to not validate input fields until blur event

Pat Gannon · Pat Gannon · commit 18f49a58ab0c · 2013-01-09T14:14:25.000-08:00
diff --git a/src/ng/directive/input.js b/src/ng/directive/input.js
@@ -406,33 +406,36 @@ function textInputType(scope, element, attr, ctrl, $sniffer, $browser) {
     }
   };
 
-  // if the browser does support "input" event, we are fine - except on IE9 which doesn't fire the
-  // input event on backspace, delete or cut
-  if ($sniffer.hasEvent('input')) {
-    element.bind('input', listener);
+  if (attr.ngUpdateOn) {
+    element.bind(attr.ngUpdateOn, listener);
   } else {
-    var timeout;
+    // if the browser does support "input" event, we are fine - except on IE9 which doesn't fire the
+    // input event on backspace, delete or cut
+    if ($sniffer.hasEvent('input')) {
+      element.bind('input', listener);
+    } else {
+      var timeout;
 
-    element.bind('keydown', function(event) {
-      var key = event.keyCode;
+      element.bind('keydown', function(event) {
+        var key = event.keyCode;
 
-      // ignore
-      //    command            modifiers                   arrows
-      if (key === 91 || (15 < key && key < 19) || (37 <= key && key <= 40)) return;
+        // ignore
+        //    command            modifiers                   arrows
+        if (key === 91 || (15 < key && key < 19) || (37 <= key && key <= 40)) return;
 
-      if (!timeout) {
-        timeout = $browser.defer(function() {
-          listener();
-          timeout = null;
-        });
-      }
-    });
+        if (!timeout) {
+          timeout = $browser.defer(function() {
+            listener();
+            timeout = null;
+          });
+        }
+      });
 
-    // if user paste into input using mouse, we need "change" event to catch it
-    element.bind('change', listener);
+      // if user paste into input using mouse, we need "change" event to catch it
+      element.bind('change', listener);
+    }
   }
 
-
   ctrl.$render = function() {
     element.val(isEmpty(ctrl.$viewValue) ? '' : ctrl.$viewValue);
   };
diff --git a/test/ng/directive/inputSpec.js b/test/ng/directive/inputSpec.js
@@ -378,14 +378,24 @@ describe('input', function() {
   });
 
 
-  it('should update the model on "blur" event', function() {
+  it('should update the model on input event', function() {
     compileInput('<input type="text" ng-model="name" name="alias" ng-change="change()" />');
 
     changeInputValueTo('adam');
     expect(scope.name).toEqual('adam');
   });
 
 
+  it('should not update the model until blur event when ng-update-on="blur" set on input element', function() {
+    compileInput('<input type="text" ng-model="name" name="alias" ng-change="change()" ng-update-on="blur" />');
+
+    changeInputValueTo('pat');
+    expect(scope.name).toEqual(undefined);
+    browserTrigger(inputElm, 'blur');
+    expect(scope.name).toEqual('pat');
+  });
+
+
   it('should update the model and trim the value', function() {
     compileInput('<input type="text" ng-model="name" name="alias" ng-change="change()" />');
 
