Make _itemsChanged depend on multi.

Polymer · Aug 6, 2015 · 1b21397 · 1b21397
1 parent 0892e93
commit 1b21397
Showing 1 changed file with 12 additions and 7 deletions.
diff --git a/src/lib/template/array-selector.html b/src/lib/template/array-selector.html
@@ -73,8 +73,7 @@
        * An array containing items from which selection will be made.
        */
       items: {
-        type: Array,
-        observer: '_itemsChanged'
+        type: Array
       },
 
       /**
@@ -98,10 +97,17 @@
        * `selected` is an array of currently selected items).  When `false`,
        * only one item may be selected at a time.
        */
-      multi: Boolean
+      multi: {
+        type: Boolean,
+        value: false
+      }
     },
 
-    _itemsChanged: function() {
+    observers: [
+      '_itemsChanged(items, multi)'
+    ],
+
+    _itemsChanged: function(items, multi) {
       // Unbind previous selection
       if (Array.isArray(this.selected)) {
         for (var i=0; i<this.selected.length; i++) {
@@ -111,7 +117,7 @@
         this.unlinkPaths('selected');
       }
       // Initialize selection
-      if (this.multi) {
+      if (multi) {
         this.selected = [];
       } else {
         this.selected = null;
@@ -151,8 +157,7 @@
         // var sidx = this.selected.indexOf(item);
         // if (sidx < 0) {
         var scol = Polymer.Collection.get(this.selected);
-        var skey = scol.getKey(item);
-        if (skey >= 0) {
+        if (scol.getKey(item) !== undefined) {
           if (this.toggle) {
             this.deselect(item);
           }