toggle has-error on parent form-group with $validationProvider valid/invalid callbacks

tb · tb · commit 1b9d52713c32 · 2015-02-07T18:37:16.000+01:00
diff --git a/README.md b/README.md
@@ -122,6 +122,18 @@ $validationProvider.setErrorHTML(function (msg) {
 
 You can add the bootstrap class `.has-success` in a similar fashion.
 
+To toggle `.has-error` class on bootstrap `.form-group` wrapper for labels and controls, add:
+
+```javascript
+angular.extend($validationProvider, {
+    validCallback: function (element){
+        $(element).parents('.form-group:first').removeClass('has-error');
+    },
+    invalidCallback: function (element) {
+        $(element).parents('.form-group:first').addClass('has-error');
+    }
+});
+```
 
 CHANGELOG
 =====
diff --git a/src/directive.js b/src/directive.js
@@ -33,6 +33,7 @@
                     }
                     ctrl.$setValidity(ctrl.$name, true);
                     if (scope.validCallback) scope.validCallback();
+                    if ($validationProvider.validCallback) $validationProvider.validCallback(element);
 
                     return true;
                 };
@@ -64,6 +65,7 @@
                     }
                     ctrl.$setValidity(ctrl.$name, false);
                     if (scope.inValidCallback) scope.inValidCallback();
+                    if ($validationProvider.invalidCallback) $validationProvider.invalidCallback(element);
 
                     return false;
                 };
diff --git a/src/provider.js b/src/provider.js
@@ -265,6 +265,8 @@
                         showErrorMessage: this.showErrorMessage,
                         checkValid: this.checkValid,
                         validate: this.validate,
+                        validCallback: this.validCallback,
+                        invalidCallback: this.invalidCallback,
                         reset: this.reset
                     };
                 }

Original file line number	Diff line number	Diff line change
`@@ -33,6 +33,7 @@`
`33`	`33`	`}`
`34`	`34`	`ctrl.$setValidity(ctrl.$name, true);`
`35`	`35`	`if (scope.validCallback) scope.validCallback();`
	`36`	`+ if ($validationProvider.validCallback) $validationProvider.validCallback(element);`
`36`	`37`
`37`	`38`	`return true;`
`38`	`39`	`};`
`@@ -64,6 +65,7 @@`
`64`	`65`	`}`
`65`	`66`	`ctrl.$setValidity(ctrl.$name, false);`
`66`	`67`	`if (scope.inValidCallback) scope.inValidCallback();`
	`68`	`+ if ($validationProvider.invalidCallback) $validationProvider.invalidCallback(element);`
`67`	`69`
`68`	`70`	`return false;`
`69`	`71`	`};`
Original file line number	Diff line number	Diff line change
`@@ -265,6 +265,8 @@`
`265`	`265`	`showErrorMessage: this.showErrorMessage,`
`266`	`266`	`checkValid: this.checkValid,`
`267`	`267`	`validate: this.validate,`
	`268`	`+ validCallback: this.validCallback,`
	`269`	`+ invalidCallback: this.invalidCallback,`
`268`	`270`	`reset: this.reset`
`269`	`271`	`};`
`270`	`272`	`}`