Fix #682 Update now returns undefined

Setting default value with a flag within the method now instead of with
return value.

Author: Michael

src/jquery.multiselect.js

@@ -249,8 +249,7 @@
 
       this._setButtonWidth();
 
-      // remember default value
-      this.button[0].defaultValue = this.update();
+      this.update(true);
 
       // broadcast refresh event; useful for widgets
       if(!init) {
@@ -259,7 +258,7 @@
     },
 
     // updates the button text. call refresh() to rebuild
-    update: function() {
+    update: function(isDefault) {
       var o = this.options;
       var $inputs = this.inputs;
       var $checked = $inputs.filter(':checked');
@@ -279,8 +278,10 @@
       }
 
       this._setButtonValue(value);
+      if(isDefault) {
+        this.button[0].defaultValue = value;
+      }
 
-      return value;
     },
 
     // this exists as a separate method so that the developer
@@ -941,7 +942,7 @@
           break;
         case 'selectedListSeparator':
           this.options[key] = value;
-          this.button[0].defaultValue = this.update();
+          this.update(true);
           break;
       }
 

tests/unit/options.js

@@ -368,7 +368,7 @@
         equals(matched.length, 8, "The 9 selected values are joined by <br> tags");
         el.multiselect("option", "selectedListSeparator", ", ");
         text = $(button()).text();
-        matched = text.match(/\,/g)
+        matched = text.match(/\,/g);
         equals(matched.length, 8, "The 9 selected values are joined by commas again after calling the option method");
         el.multiselect("destroy");
     });