chore(release): bump package version and update changelog

Author: Kerry McCullough

CHANGELOG.md

@@ -1,3 +1,9 @@
+<a name="0.14.10"></a>
+## [0.14.10](https://github.com/angular-ui/ui-select/compare/v0.14.9...v0.14.10) (2016-03-07)
+
+
+
+
 <a name="0.14.9"></a>
 ## [0.14.9](https://github.com/angular-ui/ui-select/compare/v0.14.9...v0.14.9) (2016-03-06)
 

dist/select.css

@@ -1,7 +1,7 @@
 /*!
  * ui-select
  * http://github.com/angular-ui/ui-select
- * Version: 0.14.8 - 2016-02-18T22:01:43.880Z
+ * Version: 0.14.10 - 2016-03-07T18:29:24.965Z
  * License: MIT
  */
 

dist/select.js

@@ -1,7 +1,7 @@
 /*!
  * ui-select
  * http://github.com/angular-ui/ui-select
- * Version: 0.14.8 - 2016-02-18T22:01:43.792Z
+ * Version: 0.14.10 - 2016-03-07T18:29:24.905Z
  * License: MIT
  */
 
@@ -482,7 +482,11 @@ uis.controller('uiSelectCtrl',
         ctrl.setItemsFn(data);
       }else{
         if ( data !== undefined ) {
-          var filteredItems = data.filter(function(i) {return selectedItems && selectedItems.indexOf(i) < 0;});
+          var filteredItems = data.filter(function(i) {
+            return selectedItems.every(function(selectedItem) {
+              return !angular.equals(i, selectedItem);
+            });
+          });
           ctrl.setItemsFn(filteredItems);
         }
       }
@@ -940,6 +944,11 @@ uis.directive('uiSelect',
         //Set reference to ngModel from uiSelectCtrl
         $select.ngModel = ngModel;
 
+        //Properly check for empty if set to multiple
+        ngModel.$isEmpty = function(value) {
+          return !value || angular.isDefined(attrs.multiple) && value.length === 0;
+        };
+
         $select.choiceGrouped = function(group){
           return $select.isGrouped && group && group.name;
         };
@@ -1580,9 +1589,19 @@ uis.directive('uiSelectMultiple', ['uiSelectMinErr','$timeout', function(uiSelec
               stashArr = stashArr.slice(1,stashArr.length);
             }
             newItem = $select.tagging.fct($select.search);
-            newItem.isTag = true;
-            // verify the the tag doesn't match the value of an existing item
-            if ( stashArr.filter( function (origItem) { return angular.equals( origItem, $select.tagging.fct($select.search) ); } ).length > 0 ) {
+            // verify the new tag doesn't match the value of a possible selection choice or an already selected item.
+            if (
+              stashArr.some(function (origItem) {
+                 return angular.equals(origItem, $select.tagging.fct($select.search));
+              }) ||
+              $select.selected.some(function (origItem) {
+                return angular.equals(origItem, newItem);
+              })
+            ) {
+              scope.$evalAsync(function () {
+                $select.activeIndex = 0;
+                $select.items = items;
+              });
               return;
             }
             newItem.isTag = true;
@@ -1977,38 +1996,48 @@ uis.service('uisRepeatParser', ['uiSelectMinErr','$parse', function(uiSelectMinE
 
 
     var match;
-    var isObjectCollection = /\(\s*([\$\w][\$\w]*)\s*,\s*([\$\w][\$\w]*)\s*\)/.test(expression);
+    //var isObjectCollection = /\(\s*([\$\w][\$\w]*)\s*,\s*([\$\w][\$\w]*)\s*\)/.test(expression);
     // If an array is used as collection
 
     // if (isObjectCollection){
-      //00000000000000000000000000000111111111000000000000000222222222222220033333333333333333333330000444444444444444444000000000000000556666660000077777777777755000000000000000000000088888880000000
-    match = expression.match(/^\s*(?:([\s\S]+?)\s+as\s+)?(?:([\$\w][\$\w]*)|(?:\(\s*([\$\w][\$\w]*)\s*,\s*([\$\w][\$\w]*)\s*\)))\s+in\s+(([\w\.]+)?\s*(|\s*[\s\S]+?))?(?:\s+track\s+by\s+([\s\S]+?))?\s*$/);      
+    // 000000000000000000000000000000111111111000000000000000222222222222220033333333333333333333330000444444444444444444000000000000000055555555555000000000000000000000066666666600000000
+    match = expression.match(/^\s*(?:([\s\S]+?)\s+as\s+)?(?:([\$\w][\$\w]*)|(?:\(\s*([\$\w][\$\w]*)\s*,\s*([\$\w][\$\w]*)\s*\)))\s+in\s+(\s*[\s\S]+?)?(?:\s+track\s+by\s+([\s\S]+?))?\s*$/);
 
     // 1 Alias
     // 2 Item
     // 3 Key on (key,value)
     // 4 Value on (key,value)
-    // 5 Collection expresion (only used when using an array collection)
-    // 6 Object that will be converted to Array when using (key,value) syntax
-    // 7 Filters that will be applied to #6 when using (key,value) syntax
-    // 8 Track by
+    // 5 Source expression (including filters)
+    // 6 Track by
 
     if (!match) {
       throw uiSelectMinErr('iexp', "Expected expression in form of '_item_ in _collection_[ track by _id_]' but got '{0}'.",
               expression);
     }
-    if (!match[6] && isObjectCollection) {
-      throw uiSelectMinErr('iexp', "Expected expression in form of '_item_ as (_key_, _item_) in _ObjCollection_ [ track by _id_]' but got '{0}'.",
-              expression);
+    
+    var source = match[5], 
+        filters = '';
+
+    // When using (key,value) ui-select requires filters to be extracted, since the object
+    // is converted to an array for $select.items 
+    // (in which case the filters need to be reapplied)
+    if (match[3]) {
+      // Remove any enclosing parenthesis
+      source = match[5].replace(/(^\()|(\)$)/g, '');
+      // match all after | but not after ||
+      var filterMatch = match[5].match(/^\s*(?:[\s\S]+?)(?:[^\|]|\|\|)+([\s\S]*)\s*$/);
+      if(filterMatch && filterMatch[1].trim()) {
+        filters = filterMatch[1];
+        source = source.replace(filters, '');
+      }      
     }
 
     return {
       itemName: match[4] || match[2], // (lhs) Left-hand side,
       keyName: match[3], //for (key, value) syntax
-      source: $parse(!match[3] ? match[5] : match[6]),
-      sourceName: match[6],
-      filters: match[7],
-      trackByExp: match[8],
+      source: $parse(source),
+      filters: filters,
+      trackByExp: match[6],
       modelMapper: $parse(match[1] || match[4] || match[2]),
       repeatExpression: function (grouped) {
         var expression = this.itemName + ' in ' + (grouped ? '$group.items' : '$select.items');

dist/select.min.css

@@ -7,7 +7,7 @@
     "url": "git://github.com/angular-ui/ui-select.git"
   },
   "style": "dist/select.css",
-  "version": "0.14.10",
+  "version": "0.14.11",
   "devDependencies": {
     "bower": "~1.3",
     "conventional-changelog": "^0.5.3",

dist/select.min.js

@@ -61,6 +61,11 @@ uis.directive('uiSelect',
         //Set reference to ngModel from uiSelectCtrl
         $select.ngModel = ngModel;
 
+        //Properly check for empty if set to multiple
+        ngModel.$isEmpty = function(value) {
+          return !value || angular.isDefined(attrs.multiple) && value.length === 0;
+        };
+
         $select.choiceGrouped = function(group){
           return $select.isGrouped && group && group.name;
         };

package.json

@@ -260,7 +260,7 @@ describe('ui-select tests', function() {
 
   });
 
-  it('should parse simple property binding repeat syntax with a basic filter', function () {
+  it('should parse simple property binding repeat syntax with a basic filter', function () {
 
     var locals = {};
     locals.people = [{ name: 'Wladimir' }, { name: 'Samantha' }];
@@ -269,11 +269,11 @@ describe('ui-select tests', function() {
     var parserResult = uisRepeatParser.parse('person.name as person in people | filter: { name: \'Samantha\' }');
     expect(parserResult.itemName).toBe('person');
     expect(parserResult.modelMapper(locals)).toBe(locals.person.name);
-    expect(parserResult.source(locals)).toEqual([locals.person]);
+    expect(parserResult.source(locals)).toEqual([locals.person]);
 
   });
 
-  it('should parse simple property binding repeat syntax with track by', function () {
+  it('should parse simple property binding repeat syntax with track by', function () {
 
     var locals = {};
     locals.people = [{ name: 'Wladimir' }, { name: 'Samantha' }];
@@ -282,8 +282,8 @@ describe('ui-select tests', function() {
     var parserResult = uisRepeatParser.parse('person.name as person in people track by person.name');
     expect(parserResult.itemName).toBe('person');
     expect(parserResult.modelMapper(locals)).toBe(locals.person.name);
-    expect(parserResult.source(locals)).toBe(locals.people);
-
+    expect(parserResult.source(locals)).toBe(locals.people);
+
   });
 
   it('should parse (key, value) repeat syntax', function() {
@@ -2220,6 +2220,11 @@ describe('ui-select tests', function() {
       expect(searchEl.length).toEqual(1);
       expect(searchEl[0].id).toEqual('inid');
     });
+
+    it('should properly check for empty if required', function () {
+      var el = createUiSelectMultiple({required: true});
+      expect(el.hasClass('ng-invalid-required')).toBeTruthy();
+    });
   });
 
   it('should add an id to the search input field', function () {