Merge pull request #251 from ivanvoznyakovsky/master

set default value to empty array

Author: Jefiozie

src/uiSelectController.js

@@ -201,7 +201,7 @@ uis.controller('uiSelectCtrl',
     }
 
     function setPlainItems(items) {
-      ctrl.items = items;
+      ctrl.items = items || [];
     }
 
     ctrl.setItemsFn = groupByExp ? updateGroups : setPlainItems;

test/select.spec.js

@@ -259,6 +259,13 @@ describe('ui-select tests', function() {
     scope.$digest();
   }
 
+  it('should initialize selected choices with an array if choices source is undefined', function(){
+    var el = createUiSelect(),
+        ctrl = el.scope().$select;
+
+    ctrl.setItemsFn(); // setPlainItems
+    expect(ctrl.items).toEqual([]);
+  });
 
   // Tests
   //uisRepeatParser
@@ -1816,6 +1823,14 @@ describe('ui-select tests', function() {
         );
     }
 
+    it('should initialize selected choices with an empty array when choices source is undefined', function(){
+      var el = createUiSelectMultiple({groupBy: "'age'"}),
+          ctrl = el.scope().$select;
+
+      ctrl.setItemsFn(); // updateGroups
+      expect(ctrl.items).toEqual([]);
+    });
+
     it('should render initial state', function() {
         var el = createUiSelectMultiple();
         expect(el).toHaveClass('ui-select-multiple');