Bug Tracker

Modify

Ticket #1584 (closed bug: fixed)

Opened 4 years ago

Last modified 4 years ago

cross-domain get[Script|JSON] not trigger ajaxStop

Reported by: MiRacLe Owned by:
Priority: minor Milestone: 1.2.2
Component: ajax Version: 1.2.1
Keywords: Cc:
Blocking: Blocked by:

Description (last modified by john) (diff)

script.onload = script.onreadystatechange = function(){

if ( !done && (!this.readyState
this.readyState == "loaded" this.readyState == "complete") ) {

done = true;

----------------patch---------- if ( s.global) {

jQuery.active--; jQuery.event.trigger('ajaxStop');

} if (s.success && jQuery.isFunction(s.success)) s.success(); if (s.complete && jQuery.isFunction(s.complete)) s.complete(); --------------end-patch---------- head.removeChild( script ); } };

Change History

comment:1 Changed 4 years ago by john

  • Status changed from new to closed
  • Resolution set to fixed
  • Description modified (diff)

I can't duplicate this at all - ajaxStop always fires for me, with both getScript and getJSON, both local and remotely.

HOWEVER - I was able to duplicate it with JSONP requests, and resolved the bug in SVN rev [3294]. Thanks for your help!

comment:2 Changed 4 years ago by MiRacLe

  • Status changed from closed to reopened
  • Resolution fixed deleted

hello again i'm create two pages for demonstrate this bug:  http://rpz.name/jquery-bug/latest-jquery.html - ajaxStop never executed  http://rpz.name/jquery-bug/patched-jquery.html - work fine.


--- jquery-1.2.1.js Tue Sep 18 16:06:43 2007 +++ jquery-patched.js Tue Sep 18 16:50:26 2007 @@ -2264,3 +2264,3 @@

Handle Script loading

if ( !jsonp && (s.success
s.complete) ) {

+ if ( !jsonp) {

var done = false;


comment:3 Changed 4 years ago by MiRacLe

--- jquery-1.2.1.js
+++ jquery-patched.js
@@ -2264,3 +2264,3 @@
 			// Handle Script loading
-			if ( !jsonp && (s.success || s.complete) ) {
+			if ( !jsonp) {
 				var done = false;

comment:4 Changed 4 years ago by john

  • Status changed from reopened to closed
  • Version changed from 1.2 to 1.2.1
  • Resolution set to fixed
  • Milestone changed from 1.2.1 to 1.2.2

Fixed in SVN rev [3668].

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.