deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Make sure .val() works after form.reset() in IE. Fixes #2551.

Browse source
This commit is contained in:
rwldrn 2011-02-10 23:50:02 +01:00 committed by Anton M
parent 78fc79fad4
commit 43a41ba7ec
2 changed files with 24 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/attributes.js
View file

@ -198,6 +198,11 @@ jQuery.fn.extend({
} }
} }
// Fixes Bug #2551 -- select.val() broken in IE after form.reset()
if ( one && !values.length && options.length ) {
return jQuery( options[ index ] ).val();
}
return values; return values;
} }

19
test/unit/attributes.js
View file

@ -546,6 +546,25 @@ test("val(Function) with incoming value", function() {
equals( jQuery("#select1").val(), "4", "Should be possible to set the val() to a newly created option" ); equals( jQuery("#select1").val(), "4", "Should be possible to set the val() to a newly created option" );
}); });
// testing if a form.reset() breaks a subsequent call to a select element's .val() (in IE only)
test("val(select) after form.reset() (Bug #2551)", function() {
expect(3);
jQuery('<form id="kk" name="kk"><select id="kkk"><option value="cf">cf</option><option value="gf">gf</option></select></form>').appendTo("#main");
jQuery("#kkk").val( "gf" );
document.kk.reset();
equal( jQuery("#kkk")[0].value, "cf", "Check value of select after form reset." );
equal( jQuery("#kkk").val(), "cf", "Check value of select after form reset." );
// re-verify the multi-select is not broken (after form.reset) by our fix for single-select
same( jQuery('#select3').val(), ['1', '2'], 'Call val() on a multiple="multiple" select' );
jQuery("#kk").remove();
});
var testAddClass = function(valueObj) { var testAddClass = function(valueObj) {
expect(5); expect(5);
var div = jQuery("div"); var div = jQuery("div");