fix(select) 5449-update-model-properties-when-directly-changed

src/ng/directive/input.js

@@ -1712,14 +1712,13 @@ var NgModelController = ['$scope', '$exceptionHandler', '$attrs', '$element', '$
   var ctrl = this;
 
   $scope.$watch(function ngModelWatch() {
-    var value = ngModelGet($scope);
+    var value = ngModelGet($scope),
+      formatters = ctrl.$formatters,
+      idx = formatters.length;
 
     // if scope model value and ngModel value are out of sync
     if (ctrl.$modelValue !== value) {
 
-      var formatters = ctrl.$formatters,
-          idx = formatters.length;
-
       ctrl.$modelValue = value;
       while(idx--) {
         value = formatters[idx](value);
@@ -1729,8 +1728,13 @@ var NgModelController = ['$scope', '$exceptionHandler', '$attrs', '$element', '$
         ctrl.$viewValue = value;
         ctrl.$render();
       }
-    }
+    } else if ((value) && (value instanceof Array) && (value.length === 0)) {
+      // case where model is modifed directly, this helps to validate the change
 
+      while (idx--) {
+        value = formatters[idx](value);
+      }
+    }
     return value;
   });
 }];

test/ng/directive/selectSpec.js

@@ -1331,5 +1331,15 @@ describe('select', function() {
       }).toThrowMinErr('ng','badname', 'hasOwnProperty is not a valid "option value" name');
     });
 
+    it('should become invalid if model length is set to zero in scope', function() {
+      scope.myselect = [];
+      compile('<select name="selection" ng-model="myselect" required multiple>' +
+                '<option selected>A</option><option>B</option></select>');
+      scope.$apply (function() {
+        scope.myselect.length = 0;
+      });
+      expect(element).toBeInvalid();
+    });
+
   });
 });