Fix validate.reset method

hueitan · hueitan · commit 0a96acb59f02 · 2014-04-13T15:52:33.000+08:00
diff --git a/dist/angular-validation.js b/dist/angular-validation.js
@@ -223,10 +223,6 @@
             this.reset = function (form) {
                 for (var k in form) {
                     if (form[k].hasOwnProperty('$dirty')) {
-                        form[k].$setViewValue(null);
-                        form[k].$setPristine();
-                        form[k].$setValidity(form[k].$name, false);
-                        form[k].$render();
                         $scope.$broadcast(k + 'reset');
                     }
                 }
@@ -384,6 +380,10 @@
                      * Reset the validation for specific form
                      */
                     scope.$on(ctrl.$name + 'reset', function () {
+                        ctrl.$setViewValue(null);
+                        ctrl.$setPristine();
+                        ctrl.$setValidity(ctrl.$name, false);
+                        ctrl.$render();
                         element.next().html('');
                     });
 
diff --git a/dist/angular-validation.min.js b/dist/angular-validation.min.js
diff --git a/src/directive.js b/src/directive.js
@@ -123,6 +123,10 @@
                      * Reset the validation for specific form
                      */
                     scope.$on(ctrl.$name + 'reset', function () {
+                        ctrl.$setViewValue(null);
+                        ctrl.$setPristine();
+                        ctrl.$setValidity(ctrl.$name, false);
+                        ctrl.$render();
                         element.next().html('');
                     });
 
diff --git a/src/provider.js b/src/provider.js
@@ -220,10 +220,6 @@
             this.reset = function (form) {
                 for (var k in form) {
                     if (form[k].hasOwnProperty('$dirty')) {
-                        form[k].$setViewValue(null);
-                        form[k].$setPristine();
-                        form[k].$setValidity(form[k].$name, false);
-                        form[k].$render();
                         $scope.$broadcast(k + 'reset');
                     }
                 }

Original file line number	Diff line number	Diff line change
`@@ -220,10 +220,6 @@`
`220`	`220`	`this.reset = function (form) {`
`221`	`221`	`for (var k in form) {`
`222`	`222`	`if (form[k].hasOwnProperty('$dirty')) {`
`223`		`- form[k].$setViewValue(null);`
`224`		`- form[k].$setPristine();`
`225`		`- form[k].$setValidity(form[k].$name, false);`
`226`		`- form[k].$render();`
`227`	`223`	`$scope.$broadcast(k + 'reset');`
`228`	`224`	`}`
`229`	`225`	`}`