]> source.dussan.org Git - jquery.git/commitdiff
Make sure that null params aren't traversed. Fixes #5794.
authorjeresig <jeresig@gmail.com>
Wed, 13 Jan 2010 14:29:01 +0000 (09:29 -0500)
committerjeresig <jeresig@gmail.com>
Wed, 13 Jan 2010 14:29:01 +0000 (09:29 -0500)
src/ajax.js
test/unit/ajax.js

index 3ef2d4f1e675405bff17c97623708655710d6365..3c199af95ba7b49d00317d2f4d269194557ba081 100644 (file)
@@ -665,7 +665,7 @@ jQuery.extend({
                                                }
                                        });
                                        
-                               } else if ( !traditional && typeof obj === "object" ) {
+                               } else if ( !traditional && obj != null && typeof obj === "object" ) {
                                        // Serialize object item.
                                        jQuery.each( obj, function( k, v ) {
                                                buildParams( prefix + "[" + k + "]", v );
index 298fb5bab03b285e711da9bdc500e9ba56c9937c..582aa37b63d0d7e08e922c69526cc0a02a819a7d 100644 (file)
@@ -282,7 +282,7 @@ test("serialize()", function() {
 });
 
 test("jQuery.param()", function() {
-       expect(17);
+       expect(18);
        
   equals( !jQuery.ajaxSettings.traditional, true, "traditional flag, falsy by default" );
   
@@ -336,7 +336,8 @@ test("jQuery.param()", function() {
        params = { a:[1,2], b:{ c:3, d:[4,5], e:{ x:[6], y:7, z:[8,9] }, f:true, g:false, h:undefined }, i:[10,11], j:true, k:false, l:[undefined,0], m:"cowboy hat?" };
        equals( decodeURIComponent( jQuery.param(params,false) ), "a[]=1&a[]=2&b[c]=3&b[d][]=4&b[d][]=5&b[e][x][]=6&b[e][y]=7&b[e][z][]=8&b[e][z][]=9&b[f]=true&b[g]=false&b[h]=undefined&i[]=10&i[]=11&j=true&k=false&l[]=undefined&l[]=0&m=cowboy+hat?", "huge structure, forced not traditional" );
        
-
+       params = { param1: null };
+       equals( jQuery.param(params,false), "param1=null", "Make sure that null params aren't traversed." );
 });
 
 test("synchronous request", function() {