X-Git-Url: http://git.asbjorn.it/?a=blobdiff_plain;f=src%2Fajax.js;h=edb5c54beeb1d6501021113a3edff030f34cb22b;hb=5dc6b7ce3469eaadb37a151d449e8d36571d1894;hp=7c5c83888bb2fd2d6b121ddff5bbd800021d395d;hpb=fbc73d45b487dd863886c7fd3f0af1fd4dec261b;p=jquery.git diff --git a/src/ajax.js b/src/ajax.js index 7c5c838..edb5c54 100644 --- a/src/ajax.js +++ b/src/ajax.js @@ -1,7 +1,7 @@ var jsc = now(), - rscript = //g, + rscript = //gi, rselectTextarea = /select|textarea/i, - rinput = /text|hidden|password|search/i, + rinput = /color|date|datetime|email|hidden|month|number|password|range|search|tel|text|time|url|week/i, jsre = /=\?(&|$)/, rquery = /\?/, rts = /(\?|&)_=.*?(&|$)/, @@ -199,7 +199,7 @@ jQuery.extend({ var s = jQuery.extend(true, {}, jQuery.ajaxSettings, origSettings); var jsonp, status, data, - callbackContext = s.context || window, + callbackContext = s.context || s, type = s.type.toUpperCase(); // convert data if not already a string @@ -430,7 +430,7 @@ jQuery.extend({ // Fire the complete handlers complete(); - if ( isTimeout ) { + if ( isTimeout === "timeout" ) { xhr.abort(); } @@ -591,8 +591,11 @@ jQuery.extend({ // Serialize an array of form elements or a set of // key/values into a query string param: function( a ) { + var s = [], - param_traditional = jQuery.param.traditional; + + // Set jQuery.param.traditional to true for jQuery <= 1.3.2 behavior. + traditional = jQuery.param.traditional; function add( key, value ){ // If value is a function, invoke it and return its value @@ -600,8 +603,7 @@ jQuery.extend({ 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 ) { // Serialize the form elements jQuery.each( a, function() { @@ -609,35 +611,41 @@ jQuery.extend({ }); } else { - // Encode parameters from object, recursively. If - // jQuery.param.traditional is set, encode the "old" way - // (the way 1.3.2 or older did it) + // If jQuery.param.traditional is true, encode the "old" way (the + // way 1.3.2 or older did it), otherwise encode params recursively. jQuery.each( a, function buildParams( prefix, obj ) { if ( jQuery.isArray(obj) ) { + // Serialize array item. jQuery.each( obj, function(i,v){ - // Due to rails' limited request param syntax, numeric array - // indices are not supported. To avoid serialization ambiguity - // issues, serialized arrays can only contain scalar values. php - // does not have this issue, but we should go with the lowest - // common denominator - add( prefix + ( param_traditional ? "" : "[]" ), v ); + if ( traditional ) { + // Treat each array item as a scalar. + add( prefix, v ); + } else { + // If array item is non-scalar (array or object), encode its + // numeric index to resolve deserialization ambiguity issues. + // Note that rack (as of 1.0.0) can't currently deserialize + // nested arrays properly, and attempting to do so may cause + // a server error. Possible fixes are to modify rack's + // deserialization algorithm or to provide an option or flag + // to force array serialization to be shallow. + buildParams( prefix + "[" + ( typeof v === "object" || jQuery.isArray(v) ? i : "" ) + "]", v ); + } + }); + + } else if ( !traditional && typeof obj === "object" ) { + // Serialize object item. + jQuery.each( obj, function(k,v){ + buildParams( prefix + "[" + k + "]", v ); }); - } else if ( typeof obj == "object" ) { - if ( param_traditional ) { - add( prefix, obj ); - - } else { - jQuery.each( obj, function(k,v){ - buildParams( prefix ? prefix + "[" + k + "]" : k, v ); - }); - } } else { + // Serialize scalar item. add( prefix, obj ); } }); } + // Return the resulting serialization return s.join("&").replace(r20, "+"); }