From d1931a8241dcac1617cc8388f6dd6284c89c545d Mon Sep 17 00:00:00 2001 From: malsup Date: Thu, 29 Apr 2010 11:45:34 +0800 Subject: [PATCH] Fix for http://dev.jquery.com/ticket/6451 --- src/ajax.js | 28 ++++++++++++++++++---------- test/unit/ajax.js | 6 ++++++ 2 files changed, 24 insertions(+), 10 deletions(-) diff --git a/src/ajax.js b/src/ajax.js index 7cf280a..ffd870c 100644 --- 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 index be4b3f0..fe44ba3 100644 --- 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 ); +}); + + } //} -- 1.7.10.4