X-Git-Url: http://git.asbjorn.biz/?a=blobdiff_plain;f=src%2Fajax%2Fajax.js;h=4df372bfc0ea2bce573a60491163ff47f9f08869;hb=f1c91fd023ad0bbd01a386bca4d8503e0e27df73;hp=652bd2d7153b22a3002b95e68731ce151474bb7c;hpb=953bf593a6233d9f480ee3f706d21950590f40ef;p=jquery.git

diff --git a/src/ajax/ajax.js b/src/ajax/ajax.js
index 652bd2d..4df372b 100644
--- a/src/ajax/ajax.js
+++ b/src/ajax/ajax.js
@@ -124,12 +124,7 @@ jQuery.fn.extend({
 				// for some weird reason, it doesn't work if the callback is ommited
 				jQuery.getScript( this.src );
 			else {
-				// TODO extract into $.eval
-				var data = this.text || this.textContent || this.innerHTML || "";
-				if (window.execScript)
-					window.execScript( data );
-				else
-					window.setTimeout( data, 0 );
+				jQuery.globalEval( this.text || this.textContent || this.innerHTML || "" );
 			}
 		}).end();
 	}
@@ -139,10 +134,7 @@ jQuery.fn.extend({
 // If IE is used, create a wrapper for the XMLHttpRequest object
 if ( jQuery.browser.msie && typeof XMLHttpRequest == "undefined" )
 	XMLHttpRequest = function(){
-		return new ActiveXObject(
-			navigator.userAgent.indexOf("MSIE 5") >= 0 ?
-			"Microsoft.XMLHTTP" : "Msxml2.XMLHTTP"
-		);
+		return new ActiveXObject("Microsoft.XMLHTTP");
 	};
 
 // Attach a bunch of functions for handling common AJAX events
@@ -309,6 +301,10 @@ jQuery.extend({
 	 * Loads, and executes, a remote JavaScript file using an HTTP GET request.
 	 * All of the arguments to the method (except URL) are optional.
 	 *
+	 * Warning: Safari <= 2.0.x is unable to evalulate scripts in a global
+	 * context sychronously. If you load functions via getScript, make sure
+	 * to call them after a delay.
+	 *
 	 * @example $.getScript("test.js")
 	 *
 	 * @example $.getScript("test.js", function(){
@@ -473,7 +469,7 @@ jQuery.extend({
 	 * function gets passed two arguments: The XMLHttpRequest object and a
 	 * string describing the type the success of the request.
 	 *
- 	 * (String) data - Data to be sent to the server. Converted to a query
+ 	 * (Object|String) data - Data to be sent to the server. Converted to a query
 	 * string, if not already a string. Is appended to the url for GET-requests.
 	 * Override processData option to prevent processing.
 	 *
@@ -488,6 +484,9 @@ jQuery.extend({
 	 * (Boolean) async - By default, all requests are send asynchronous (set to true).
 	 * If you need synchronous requests, set this option to false.
 	 *
+	 * (Function) beforeSend - A pre-callback to set custom headers etc., the
+	 * XMLHttpRequest is passed as the only argument.
+	 *
 	 * @example $.ajax({
 	 *   type: "GET",
 	 *   url: "test.js",
@@ -525,7 +524,8 @@ jQuery.extend({
 			data: null,
 			contentType: "application/x-www-form-urlencoded",
 			processData: true,
-			async: true
+			async: true,
+			beforeSend: null
 		}, s);
 
 		// if data available
@@ -566,6 +566,10 @@ jQuery.extend({
 		// Make sure the browser sends the right content length
 		if ( xml.overrideMimeType )
 			xml.setRequestHeader("Connection", "close");
+			
+		// Allow custom headers/mimetypes
+		if( s.beforeSend )
+			s.beforeSend(xml);
 
 		// Wait for a response to come back
 		var onreadystatechange = function(isTimeout){
@@ -688,12 +692,8 @@ jQuery.extend({
 		data = type == "xml" || data ? r.responseXML : r.responseText;
 
 		// If the type is "script", eval it in global context
-		// TODO extract as $.eval
 		if ( type == "script" ) {
-			if (window.execScript)
-				window.execScript( data );
-			else
-				window.setTimeout( data, 0 );
+			jQuery.globalEval( data );
 		}
 
 		// Get the JavaScript object, if JSON is used.
@@ -721,10 +721,10 @@ jQuery.extend({
 		} else {
 			// Serialize the key/values
 			for ( var j in a ) {
-				//if one value is array then treat each array value in part
-				if (typeof a[j] == 'object') {
+				// If the value is an array then the key names need to be repeated
+				if( a[j].constructor == Array ) {
 					for (var k = 0; k < a[j].length; k++) {
-						s.push( j + "[]=" + encodeURIComponent( a[j][k] ) );
+						s.push( j + "=" + encodeURIComponent( a[j][k] ) );
 					}
 				} else {
 					s.push( j + "=" + encodeURIComponent( a[j] ) );
@@ -734,6 +734,18 @@ jQuery.extend({
 
 		// Return the resulting serialization
 		return s.join("&");
+	},
+	
+	// evalulates a script in global context
+	// not reliable for safari
+	globalEval: function(data) {
+		if (window.execScript)
+			window.execScript( data );
+		else if(jQuery.browser.safari)
+			// safari doesn't provide a synchronous global eval
+			window.setTimeout( data, 0 );
+		else
+			eval.call( window, data );
 	}
 
 });