X-Git-Url: http://git.asbjorn.biz/?a=blobdiff_plain;f=src%2Fajax.js;h=d92d26d15ae1e6522e4e9973033ec24bd391afcc;hb=c639405c4d1ec462b032ccfb571a6c73212ba101;hp=02322ab3944482848f3589b8387d3b3182640993;hpb=61983cbf176c599687c36ffbf4b64ae8697486a3;p=jquery.git diff --git a/src/ajax.js b/src/ajax.js index 02322ab..d92d26d 100644 --- a/src/ajax.js +++ b/src/ajax.js @@ -45,18 +45,19 @@ jQuery.fn.extend({ } } + var self = this; + // Request the remote document jQuery.ajax({ url: url, type: type, dataType: "html", data: params, - context:this, complete: function( res, status ) { // If successful, inject the HTML into all the matched elements if ( status === "success" || status === "notmodified" ) { // See if a selector was specified - this.html( selector ? + self.html( selector ? // Create a dummy div to hold the results jQuery("
") // inject the contents of the document in, removing the scripts @@ -71,7 +72,7 @@ jQuery.fn.extend({ } if ( callback ) { - this.each( callback, [res.responseText, status, res] ); + self.each( callback, [res.responseText, status, res] ); } } }); @@ -457,7 +458,9 @@ jQuery.extend({ xhr.abort = function() { if ( xhr ) { oldAbort.call( xhr ); - xhr.readyState = 0; + if ( xhr ) { + xhr.readyState = 0; + } } onreadystatechange(); @@ -476,7 +479,7 @@ jQuery.extend({ // Send the data try { - xhr.send( type === "POST" || type === "PUT" ? s.data : null ); + xhr.send( type === "POST" || type === "PUT" || type === "DELETE" ? s.data : null ); } catch(e) { jQuery.handleError(s, xhr, null, e); // Fire the complete handlers @@ -528,7 +531,7 @@ jQuery.extend({ handleError: function( s, xhr, status, e ) { // If a local callback was specified, fire it if ( s.error ) { - s.error.call( s.context || window, xhr, status, e ); + s.error.call( s.context || s, xhr, status, e ); } // Fire the global callback @@ -576,7 +579,7 @@ jQuery.extend({ data = xml ? xhr.responseXML : xhr.responseText; if ( xml && data.documentElement.nodeName === "parsererror" ) { - throw "parsererror"; + jQuery.error( "parsererror" ); } // Allow a pre-filtering function to sanitize the response @@ -589,23 +592,7 @@ jQuery.extend({ if ( typeof data === "string" ) { // Get the JavaScript object, if JSON is used. if ( type === "json" || !type && ct.indexOf("json") >= 0 ) { - // Make sure the incoming data is actual JSON - // Logic borrowed from http://json.org/json2.js - if (/^[\],:{}\s]*$/.test(data.replace(/\\(?:["\\\/bfnrt]|u[0-9a-fA-F]{4})/g, "@") - .replace(/"[^"\\\n\r]*"|true|false|null|-?\d+(?:\.\d*)?(?:[eE][+\-]?\d+)?/g, "]") - .replace(/(?:^|:|,)(?:\s*\[)+/g, ""))) { - - // Try to use the native JSON parser first - if ( window.JSON && window.JSON.parse ) { - data = window.JSON.parse( data ); - - } else { - data = (new Function("return " + data))(); - } - - } else { - throw "Invalid JSON: " + data; - } + data = jQuery.parseJSON( data ); // If the type is "script", eval it in global context } else if ( type === "script" || !type && ct.indexOf("javascript") >= 0 ) { @@ -663,7 +650,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 );