Closes #2688

[jquery.git] / src / ajax.js

diff --git a/src/ajax.js b/src/ajax.js
index 6628b1f..77537e7 100644 (file)
--- a/src/ajax.js
+++ b/src/ajax.js
@@ -144,14 +144,24 @@ jQuery.extend({
                contentType: "application/x-www-form-urlencoded",
                processData: true,
                async: true,
-               data: null
+               data: null,
+               username: null,
+               password: null,
+               accepts: {
+                       xml: "application/xml, text/xml",
+                       html: "text/html",
+                       script: "text/javascript, application/javascript",
+                       json: "application/json, text/javascript",
+                       text: "text/plain",
+                       _default: "*/*"
+               }
        },
        
        // Last-Modified header cache for next request
        lastModified: {},
 
        ajax: function( s ) {
-               var jsonp, jsre = /=(\?|%3F)/g, status, data;
+               var jsonp, jsre = /=\?(&|$)/g, status, data;
 
                // Extend the settings, but re-extend 's' so that it can be
                // checked again later (in the test suite, specifically)
@@ -177,8 +187,8 @@ jQuery.extend({
 
                        // Replace the =? sequence both in the query string and the data
                        if ( s.data )
-                               s.data = (s.data + "").replace(jsre, "=" + jsonp);
-                       s.url = s.url.replace(jsre, "=" + jsonp);
+                               s.data = (s.data + "").replace(jsre, "=" + jsonp + "$1");
+                       s.url = s.url.replace(jsre, "=" + jsonp + "$1");
 
                        // We need to make sure
                        // that a JSONP style response is executed properly
@@ -192,6 +202,8 @@ jQuery.extend({
                                // Garbage collect
                                window[ jsonp ] = undefined;
                                try{ delete window[ jsonp ]; } catch(e){}
+                               if ( head )
+                                       head.removeChild( script );
                        };
                }
 
@@ -220,7 +232,7 @@ jQuery.extend({
 
                // If we're requesting a remote document
                // and trying to load JSON or Script with a GET
-               if ( (!s.url.indexOf("http") || !s.url.indexOf("//")) && ( s.dataType == "script" || s.dataType =="json" ) && s.type.toLowerCase() == "get" ) {
+               if ( (!s.url.indexOf("http") || !s.url.indexOf("//")) && s.dataType == "script" && s.type.toLowerCase() == "get" ) {
                        var head = document.getElementsByTagName("head")[0];
                        var script = document.createElement("script");
                        script.src = s.url;
@@ -256,7 +268,7 @@ jQuery.extend({
                var xml = window.ActiveXObject ? new ActiveXObject("Microsoft.XMLHTTP") : new XMLHttpRequest();
 
                // Open the socket
-               xml.open(s.type, s.url, s.async);
+               xml.open(s.type, s.url, s.async, s.username, s.password);
 
                // Need an extra try/catch for cross domain requests in Firefox 3
                try {
@@ -271,12 +283,20 @@ jQuery.extend({
 
                        // Set header so the called script knows that it's an XMLHttpRequest
                        xml.setRequestHeader("X-Requested-With", "XMLHttpRequest");
+
+                       // Set the Accepts header for the server, depending on the dataType
+                       xml.setRequestHeader("Accept", s.dataType && s.accepts[ s.dataType ] ?
+                               s.accepts[ s.dataType ] + ", */*" :
+                               s.accepts._default );
                } catch(e){}
 
                // Allow custom headers/mimetypes
                if ( s.beforeSend )
-                       s.beforeSend(xml);
+                       var beforeSendSuccess = s.beforeSend(xml, s);
                        
+               if( beforeSendSuccess === false )
+                 return false;
+               
                if ( s.global )
                        jQuery.event.trigger("ajaxSend", [xml, s]);