Validate checkboxes on change

Fix for #377

Author: davidlgj

src/directives/array.js

@@ -19,6 +19,7 @@ angular.module('schemaForm').directive('sfArray', ['sfSelect', 'schemaForm', 'sf
       link: function(scope, element, attrs, ngModel) {
         var formDefCache = {};
 
+        scope.validateArray = angular.noop;
 
         if (ngModel) {
           // We need the ngModelController on several places,
@@ -41,9 +42,8 @@ angular.module('schemaForm').directive('sfArray', ['sfSelect', 'schemaForm', 'sf
           // We only modify the same array instance but someone might change the array from
           // the outside so let's watch for that. We use an ordinary watch since the only case
           // we're really interested in is if its a new instance.
-          scope.$watch('model' + sfPath.normalize(form.key), function() {
-            list = sfSelect(form.key, scope.model);
-            scope.modelArray = list;
+          scope.$watch('model' + sfPath.normalize(form.key), function(value) {
+            scope.modelArray = value;
           });
 
           // Since ng-model happily creates objects in a deep path when setting a
@@ -126,19 +126,15 @@ angular.module('schemaForm').directive('sfArray', ['sfSelect', 'schemaForm', 'sf
             }
 
             // Trigger validation.
-            if (scope.validateArray) {
-              scope.validateArray();
-            }
+            scope.validateArray();
             return list;
           };
 
           scope.deleteFromArray = function(index) {
             list.splice(index, 1);
 
             // Trigger validation.
-            if (scope.validateArray) {
-              scope.validateArray();
-            }
+            scope.validateArray();
 
             // Angular 1.2 lacks setDirty
             if (ngModel && ngModel.$setDirty) {
@@ -172,29 +168,29 @@ angular.module('schemaForm').directive('sfArray', ['sfSelect', 'schemaForm', 'sf
               form.titleMap.forEach(function(item) {
                 scope.titleMapValues.push(arr.indexOf(item.value) !== -1);
               });
-
             };
             //Catch default values
             updateTitleMapValues(scope.modelArray);
             scope.$watchCollection('modelArray', updateTitleMapValues);
 
             //To get two way binding we also watch our titleMapValues
-            scope.$watchCollection('titleMapValues', function(vals) {
-              if (vals) {
+            scope.$watchCollection('titleMapValues', function(vals, old) {
+              if (vals && vals !== old) {
                 var arr = scope.modelArray;
 
                 // Apparently the fastest way to clear an array, readable too.
                 // http://jsperf.com/array-destroy/32
                 while (arr.length > 0) {
                   arr.pop();
                 }
-
                 form.titleMap.forEach(function(item, index) {
                   if (vals[index]) {
                     arr.push(item.value);
                   }
                 });
 
+                // Time to validate the rebuilt array.
+                scope.validateArray();
               }
             });
           }

src/directives/decorators/bootstrap/checkboxes.html

@@ -9,7 +9,6 @@
              sf-changed="form"
              class="{{form.fieldHtmlClass}}"
              ng-model="titleMapValues[$index]"
-             schema-vaidate="form"
              name="{{form.key.slice(-1)[0]}}">
       <span ng-bind-html="form.titleMap[$index].name"></span>
     </label>

src/directives/schema-validate.js

@@ -1,5 +1,5 @@
-angular.module('schemaForm').directive('schemaValidate', ['sfValidator', '$parse',
-  function(sfValidator, $parse) {
+angular.module('schemaForm').directive('schemaValidate', ['sfValidator', '$parse', 'sfSelect',
+  function(sfValidator, $parse, sfSelect) {
 
     return {
       restrict: 'A',