fix(ngRequired): set valid to false if select value is required (add isSelectedOptionValid function)

Author: m-amr

src/ng/directive/ngOptions.js

@@ -387,6 +387,34 @@ var ngOptionsDirective = ['$compile', '$parse', function($compile, $parse) {
           selectValueMap[selectValue] = optionItem;
         }
 
+        /**
+         *
+         * @returns {boolean}
+         */
+        function isSelectedOptionValid(){
+
+          var selectedValue = selectElement.val();
+
+          if(!selectedValue){
+            return false;
+          }
+
+          // in case  select is multiple then check
+          // if one of the selected values matched with the options  return true
+          // or return false if no value match
+          if (angular.isArray(selectedValue)) {
+            for (var i = 0, length = selectedValue.length; i < length; i++) {
+              if (selectValueMap[selectedValue[i]]) {
+                return true
+              }
+            }
+            return false
+          }
+
+          return !!selectValueMap[selectedValue];
+
+        }
+
         return {
           items: optionItems,
           selectValueMap: selectValueMap,
@@ -397,7 +425,9 @@ var ngOptionsDirective = ['$compile', '$parse', function($compile, $parse) {
             // If the viewValue could be an object that may be mutated by the application,
             // we need to make a copy and not return the reference to the value on the option.
             return trackBy ? angular.copy(option.viewValue) : option.viewValue;
-          }
+          },
+
+          isSelectedOptionValid: isSelectedOptionValid
         };
       }
     };
@@ -584,10 +614,7 @@ var ngOptionsDirective = ['$compile', '$parse', function($compile, $parse) {
       var $$isEmpty = ngModelCtrl.$isEmpty;
 
       ngModelCtrl.$isEmpty = function(value) {
-        if ($$isEmpty(value)) {
-          return true;
-        }
-        return !isViewOptionValid(value);
+        return $$isEmpty(value) || !options.isSelectedOptionValid();
       };
 
       if (providedEmptyOption) {
@@ -762,7 +789,7 @@ var ngOptionsDirective = ['$compile', '$parse', function($compile, $parse) {
         ngModelCtrl.$render();
 
         // Check to see if the value has changed due to the update to the options
-        if (!ngModelCtrl.$isEmpty(previousValue)) {
+        if (!$$isEmpty(previousValue)) {
           var nextValue = selectCtrl.readValue();
           var isNotPrimitive = ngOptions.trackBy || multiple;
           if (isNotPrimitive ? !equals(previousValue, nextValue) : previousValue !== nextValue) {