MultiSelector should clear filtering if options are being reset

codeforamerica · May 2, 2012 · 526a35d · 526a35d
1 parent 6e18bf1
commit 526a35d
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/js/MultiSelector.js b/js/MultiSelector.js
@@ -101,6 +101,7 @@ MultiSelector.prototype = {
     for (var i=0, len=options.length; i < len; i++) {
       this.addOption(options[i]);
     }
+    this.clearFilter();
   },
 
   addOption: function (option) {
@@ -119,8 +120,8 @@ MultiSelector.prototype = {
   },
 
   filterOptions: function (filter) {
-    var alwaysMatch = !filter;
     filter = filter && filter.toUpperCase();
+    var alwaysMatch = !filter;
     for (var i=0, len=this.options.length; i < len; i++) {
       var matches = alwaysMatch || this.options[i].name.toUpperCase().indexOf(filter) > -1;
       this.options[i].element.style.display = matches ? "" : "none";