Merge pull request #29 from ckniffen/bugfix/multiCheckbox-required

Required type validation only worked for the last checkbox.

Author: Kent C. Dodds

src/types/multiCheckbox.js

@@ -17,6 +17,14 @@ export default ngModule => {
           valueProp: c.string.optional
         })
       },
+      defaultOptions: {
+        ngModelAttrs:{
+          required: {
+            attribute: '',
+            bound: ''
+          }
+        }
+      },
       apiCheckInstance: c,
       controller: /* @ngInject */ function($scope) {
         const to = $scope.to;
@@ -35,15 +43,42 @@ export default ngModule => {
           });
         }
 
+        function checkValidity(expressionValue){
+          var valid = angular.isArray($scope.model[opts.key]) && 
+            $scope.model[opts.key].length > 0 && 
+            expressionValue;
+
+          $scope.fc.$setValidity('required', valid);
+        }
+
         function setModel() {
           $scope.model[opts.key] = [];
           angular.forEach($scope.multiCheckbox.checked, (checkbox, index) => {
             if (checkbox) {
               $scope.model[opts.key].push(to.options[index][to.valueProp || 'value']);
             }
           });
+
+          // Must make sure we mark as touched because only the last checkbox due to a bug in angular.
+          $scope.fc.$setTouched();
+          checkValidity(true);
+        }
+
+        if(opts.expressionProperties && opts.expressionProperties.required){
+          $scope.$watch($scope.options.expressionProperties.required, function(newValue){
+            checkValidity(newValue);
+          });
+        }
+
+        if($scope.to.required){
+          var unwatchFormControl = $scope.$watch('fc', function(newValue){
+            if(!newValue){ return; }
+            checkValidity(true);
+            unwatchFormControl;
+          });
         }
       }
     });
   }
+
 };