Fix for http://dev.jquery.com/ticket/6451

jquery · Jun 15, 2010 · d1931a8 · d1931a8
1 parent 1533bf7
commit d1931a8
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 10 deletions.
diff --git a/src/ajax.js b/src/ajax.js
@@ -237,17 +237,24 @@ jQuery.extend({
 			s.dataType = "script";
 
 			// Handle JSONP-style loading
-			window[ jsonp ] = window[ jsonp ] || function( tmp ) {
+			var customJsonp = window[ jsonp ];
+			window[ jsonp ] = function( tmp ) {
 				data = tmp;
 				jQuery.ajax.handleSuccess( s, xhr, status, data );
 				jQuery.ajax.handleComplete( s, xhr, status, data );
-				// Garbage collect
-				window[ jsonp ] = undefined;
-
-				try {
-					delete window[ jsonp ];
-				} catch( jsonpError ) {}
-
+
+				if ( jQuery.isFunction( customJsonp ) ) {
+				    customJsonp( tmp );
+			    }
+			    else {
+    				// Garbage collect
+    				window[ jsonp ] = undefined;
+
+    				try {
+    					delete window[ jsonp ];
+    				} catch( jsonpError ) {}
+                }
+
 				if ( head ) {
 					head.removeChild( script );
 				}
@@ -436,8 +443,9 @@ jQuery.extend({
 				}
 
 				// Fire the complete handlers
-				jQuery.ajax.handleComplete( s, xhr, status, data );
-
+				if ( !jsonp ) {
+				    jQuery.ajax.handleComplete( s, xhr, status, data );
+                }
 				if ( isTimeout === "timeout" ) {
 					xhr.abort();
 				}

diff --git a/test/unit/ajax.js b/test/unit/ajax.js
@@ -1225,6 +1225,12 @@ test("jQuery.ajax - Etag support", function() {
 	});
 });
 
+
+test("jQuery.ajax - active counter", function() {
+    ok( jQuery.ajax.active == 0, "ajax active counter should be zero: " + jQuery.ajax.active );
+});
+
+
 }
 
 //}