Commit f91b944c authored by John Resig's avatar John Resig

Objects with length properties weren't getting serialized properly by jQuery.param(). Fixes #5862.

parent 76236a15
...@@ -603,7 +603,6 @@ jQuery.extend({ ...@@ -603,7 +603,6 @@ jQuery.extend({
// Serialize an array of form elements or a set of // Serialize an array of form elements or a set of
// key/values into a query string // key/values into a query string
param: function( a, traditional ) { param: function( a, traditional ) {
var s = []; var s = [];
// Set traditional to true for jQuery <= 1.3.2 behavior. // Set traditional to true for jQuery <= 1.3.2 behavior.
...@@ -611,12 +610,6 @@ jQuery.extend({ ...@@ -611,12 +610,6 @@ jQuery.extend({
traditional = jQuery.ajaxSettings.traditional; traditional = jQuery.ajaxSettings.traditional;
} }
function add( key, value ) {
// If value is a function, invoke it and return its value
value = jQuery.isFunction(value) ? value() : value;
s[ s.length ] = encodeURIComponent(key) + "=" + encodeURIComponent(value);
}
// If an array was passed in, assume that it is an array of form elements. // 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 ) {
// Serialize the form elements // Serialize the form elements
...@@ -627,8 +620,15 @@ jQuery.extend({ ...@@ -627,8 +620,15 @@ jQuery.extend({
} else { } else {
// If traditional, encode the "old" way (the way 1.3.2 or older // If traditional, encode the "old" way (the way 1.3.2 or older
// did it), otherwise encode params recursively. // did it), otherwise encode params recursively.
jQuery.each( a, function buildParams( prefix, obj ) { for ( var prefix in a ) {
buildParams( prefix, a[prefix] );
}
}
// Return the resulting serialization
return s.join("&").replace(r20, "+");
function buildParams( prefix, obj ) {
if ( jQuery.isArray(obj) ) { if ( jQuery.isArray(obj) ) {
// Serialize array item. // Serialize array item.
jQuery.each( obj, function( i, v ) { jQuery.each( obj, function( i, v ) {
...@@ -657,11 +657,12 @@ jQuery.extend({ ...@@ -657,11 +657,12 @@ jQuery.extend({
// Serialize scalar item. // Serialize scalar item.
add( prefix, obj ); add( prefix, obj );
} }
});
} }
// Return the resulting serialization function add( key, value ) {
return s.join("&").replace(r20, "+"); // If value is a function, invoke it and return its value
value = jQuery.isFunction(value) ? value() : value;
s[ s.length ] = encodeURIComponent(key) + "=" + encodeURIComponent(value);
}
} }
}); });
...@@ -979,6 +979,19 @@ test("jQuery.getJSON(String, Function) - JSON object with absolute url to local ...@@ -979,6 +979,19 @@ test("jQuery.getJSON(String, Function) - JSON object with absolute url to local
}); });
}); });
test("jQuery.post - data", function() {
expect(2);
stop();
jQuery.post(url("data/name.php"), {xml: "5-2", length: 3}, function(xml){
jQuery('math', xml).each(function() {
equals( jQuery('calculation', this).text(), '5-2', 'Check for XML' );
equals( jQuery('result', this).text(), '3', 'Check for XML' );
});
start();
});
});
test("jQuery.post(String, Hash, Function) - simple with xml", function() { test("jQuery.post(String, Hash, Function) - simple with xml", function() {
expect(4); expect(4);
stop(); stop();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment