Merge pull request #290 from angular-ui/fix-multiselect-modelsync

fix(multiselect): run $formatters when changing model directly

Author: dimirc

src/select.js

@@ -763,6 +763,10 @@
         });
 
         if ($select.multiple){
+          scope.$watchCollection(function(){ return ngModel.$modelValue; }, function(newValue, oldValue) {
+            if (oldValue != newValue)
+              ngModel.$modelValue = null; //Force scope model value and ngModel value to be out of sync to re-run formatters
+          });
           scope.$watchCollection('$select.selected', function() {
             ngModel.$setViewValue(Date.now()); //Set timestamp as a unique string to force changes
           });

test/select.spec.js

@@ -1443,6 +1443,33 @@ describe('ui-select tests', function() {
 
     });
 
+
+    it('should run $formatters when changing model directly', function () {
+      
+      scope.selection.selectedMultiple = ['[email protected]', '[email protected]'];
+
+      var el = compileTemplate(
+          '<ui-select multiple ng-model="selection.selectedMultiple" theme="bootstrap" style="width: 800px;"> \
+              <ui-select-match placeholder="Pick one...">{{$item.name}} &lt;{{$item.email}}&gt;</ui-select-match> \
+              <ui-select-choices repeat="person.email as person in people | filter: $select.search"> \
+                <div ng-bind-html="person.name | highlight: $select.search"></div> \
+                <div ng-bind-html="person.email | highlight: $select.search"></div> \
+              </ui-select-choices> \
+          </ui-select> \
+          '
+      );
+
+      // var el2 = compileTemplate('<span class="resultDiv" ng-bind="selection.selectedMultiple"></span>');
+
+      scope.selection.selectedMultiple.push("[email protected]");
+
+      scope.$digest();
+      scope.$digest(); //2nd $digest needed when using angular 1.3.0-rc.1+, might be related with the fact that the value is an array
+
+      expect(el.find('.ui-select-match-item [uis-transclude-append]:not(.ng-hide)').text())
+         .toBe("Wladimir <[email protected]>Samantha <[email protected]>Nicole <[email protected]>");
+
+    });
   });