]> source.dussan.org Git - jquery.git/commitdiff
Fixes #7945. Make jQuery.param() serialize plain objects with a property named jquery...
authorAnton Kovalyov <anton@kovalyov.net>
Tue, 1 Feb 2011 23:32:29 +0000 (00:32 +0100)
committerAnton M <obhvsbypqghgc@gmail.com>
Tue, 1 Feb 2011 23:32:29 +0000 (00:32 +0100)
src/ajax.js
test/unit/ajax.js

index 4c3f1dd53429fb3ef1eeab09a34f7fce00348374..1a19d0ca16077086f9ad86e59afb5fdaf9f8706b 100644 (file)
@@ -698,7 +698,7 @@ jQuery.extend({
                }
 
                // If an array was passed in, assume that it is an array of form elements.
-               if ( jQuery.isArray( a ) || a.jquery ) {
+               if ( jQuery.isArray( a ) || ( a.jquery && !jQuery.isPlainObject( a ) ) ) {
                        // Serialize the form elements
                        jQuery.each( a, function() {
                                add( this.name, this.value );
index 3041834751d375679170959975c99778cde9a610..b81031384d455fa766404c2db60ba467e4741787 100644 (file)
@@ -930,7 +930,7 @@ test("serialize()", function() {
 });
 
 test("jQuery.param()", function() {
-       expect(23);
+       expect(25);
 
        equals( !jQuery.ajaxSettings.traditional, true, "traditional flag, falsy by default" );
 
@@ -965,6 +965,11 @@ test("jQuery.param()", function() {
        equals( jQuery.param({"foo": {"bar": [], foo: 1} }), "foo%5Bbar%5D=&foo%5Bfoo%5D=1", "Empty array param" );
        equals( jQuery.param({"foo": {"bar": {}} }), "foo%5Bbar%5D=", "Empty object param" );
 
+       // #7945
+       equals( jQuery.param({"jquery": "1.4.2"}), "jquery=1.4.2", "Check that object with a jQuery property get serialized correctly" );
+
+       equals( jQuery.param(jQuery("#form :input")), "action=Test&text2=Test&radio1=on&radio2=on&check=on&=on&hidden=&foo%5Bbar%5D=&name=name&search=search&button=&=foobar&select1=&select2=3&select3=1&select4=1&select5=3", "Make sure jQuery objects are properly serialized");
+
        jQuery.ajaxSetup({ traditional: true });
 
        var params = {foo:"bar", baz:42, quux:"All your base are belong to us"};