Made sure that the .val() logic for setting radios and checkboxes was correct. Fixes...
authorjeresig <jeresig@gmail.com>
Tue, 22 Dec 2009 06:18:49 +0000 (01:18 -0500)
committerjeresig <jeresig@gmail.com>
Tue, 22 Dec 2009 06:18:49 +0000 (01:18 -0500)
src/attributes.js
test/unit/manipulation.js

index 91a0fe1..eecf90f 100644 (file)
@@ -158,7 +158,7 @@ jQuery.fn.extend({
                        }
 
                        if ( jQuery.isArray(val) && /radio|checkbox/.test( this.type ) ) {
                        }
 
                        if ( jQuery.isArray(val) && /radio|checkbox/.test( this.type ) ) {
-                               this.checked = jQuery.inArray(this.value || this.name, val) >= 0;
+                               this.checked = jQuery.inArray( this.value, val ) >= 0;
 
                        } else if ( jQuery.nodeName( this, "select" ) ) {
                                var values = jQuery.makeArray(val);
 
                        } else if ( jQuery.nodeName( this, "select" ) ) {
                                var values = jQuery.makeArray(val);
index 7521c76..0ebae16 100644 (file)
@@ -619,7 +619,7 @@ test("clone() on XML nodes", function() {
 }
 
 test("val()", function() {
 }
 
 test("val()", function() {
-       expect(11);
+       expect(15);
 
        document.getElementById('text1').value = "bla";
        equals( jQuery("#text1").val(), "bla", "Check for modified value of input element" );
 
        document.getElementById('text1').value = "bla";
        equals( jQuery("#text1").val(), "bla", "Check for modified value of input element" );
@@ -646,6 +646,22 @@ test("val()", function() {
        jQuery('#select3').val("");
        same( jQuery('#select3').val(), [''], 'Call val() on a multiple="multiple" select' );
 
        jQuery('#select3').val("");
        same( jQuery('#select3').val(), [''], 'Call val() on a multiple="multiple" select' );
 
+       var checks = jQuery("<input type='checkbox' name='test' value='1'/>").appendTo("#form")
+               .add( jQuery("<input type='checkbox' name='test' value='2'/>").appendTo("#form") )
+               .add( jQuery("<input type='checkbox' name='test' value=''/>").appendTo("#form") );
+
+       same( checks.serialize(), "", "Get unchecked values." );
+
+       checks.val([ "2" ]);
+       same( checks.serialize(), "test=2", "Get a single checked value." );
+
+       checks.val([ "1", "" ]);
+       same( checks.serialize(), "test=1&test=", "Get multiple checked values." );
+
+       checks.val([ "", "2" ]);
+       same( checks.serialize(), "test=2&test=", "Get multiple checked values." );
+
+       checks.remove();
 });
 
 var testVal = function(valueObj) {
 });
 
 var testVal = function(valueObj) {