feat(form) If css style is BS3 then has-error is now applied to form-group by ng-class

mchapman · mchapman · commit 84595b39d253 · 2015-02-20T12:50:58.000Z
Same idea as previous commit, but this works before model is updated and with nested sub schemas.

Known issues: Only works with sub schemas one level deep at the moment
diff --git a/js/controllers/base.js b/js/controllers/base.js
@@ -19,6 +19,22 @@ formsAngular.controller('BaseCtrl', [
 
         $rootScope.$broadcast('fngFormLoadStart', $scope);
 
+        // Invalid field styling for BS3 - only works for one level of nesting
+        $scope.hasError = function(name, index) {
+          var form = $scope[$scope.topLevelFormName];
+          var field;
+          if (typeof index === 'undefined') {
+            field = form['f_' + name.replace(/\./g,'_')];
+          } else {
+            var parts = name.split('.');
+            form = form['form_' + parts[0] + index];
+            field = form[name.replace(/\./g,'-')];
+          }
+          if (field && field.$invalid) {
+            return true
+          }
+        };
+
         formGenerator.decorateScope($scope, formGenerator, recordHandler, sharedStuff);
         recordHandler.decorateScope($scope, $modal, recordHandler, ctrlState);
 
diff --git a/js/services/form-markup-helper.js b/js/services/form-markup-helper.js
@@ -64,7 +64,7 @@ formsAngular.factory('formMarkupHelper', [
           classes += ' col-sm-' + inputSizeHelper.sizeAsNumber(info.size);
           closeTag += '</div>';
         }
-        template += '<div' + addAllService.addAll(scope, 'Group', classes, options);
+        template += '<div' + addAllService.addAll(scope, 'Group', classes, options)+' ng-class="{\'has-error\': hasError(\'' + info.name + '\', $index)}"';
         closeTag += '</div>';
       } else {
         if (exports.isHorizontalStyle(options.formstyle)) {

Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@ formsAngular.factory('formMarkupHelper', [`
`64`	`64`	`classes += ' col-sm-' + inputSizeHelper.sizeAsNumber(info.size);`
`65`	`65`	`closeTag += '</div>';`
`66`	`66`	`}`
`67`		`- template += '<div' + addAllService.addAll(scope, 'Group', classes, options);`
	`67`	`+ template += '<div' + addAllService.addAll(scope, 'Group', classes, options)+' ng-class="{\'has-error\': hasError(\'' + info.name + '\', $index)}"';`
`68`	`68`	`closeTag += '</div>';`
`69`	`69`	`} else {`
`70`	`70`	`if (exports.isHorizontalStyle(options.formstyle)) {`