diff --git a/CHANGELOG b/CHANGELOG index 74dbc127d..747b0f165 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -1,3 +1,5 @@ +* Rename variable in Form.Element.Serializers.select. [jddalton] + * Replace browser sniffing with capability detection in IE fork of String#unescapeHTML. [kangax] * Coerce Opera's version string into a number whenever we need to sniff. [Sam Holman, jddalton] diff --git a/src/form.js b/src/form.js index d28feb885..ea7bf4332 100644 --- a/src/form.js +++ b/src/form.js @@ -248,22 +248,22 @@ Form.Element.Serializers = { else element.value = value; }, - select: function(element, index) { - if (Object.isUndefined(index)) + select: function(element, value) { + if (Object.isUndefined(value)) return this[element.type == 'select-one' ? 'selectOne' : 'selectMany'](element); else { - var opt, value, single = !Object.isArray(index); + var opt, currentValue, single = !Object.isArray(value); for (var i = 0, length = element.length; i < length; i++) { opt = element.options[i]; - value = this.optionValue(opt); + currentValue = this.optionValue(opt); if (single) { - if (value == index) { + if (currentValue == value) { opt.selected = true; return; } } - else opt.selected = index.include(value); + else opt.selected = value.include(currentValue); } } },