From 7afe6dcc0837ac00cea7fbb7de6fb95b745148c2 Mon Sep 17 00:00:00 2001 From: jeresig Date: Tue, 12 Jan 2010 10:31:51 -0500 Subject: [PATCH] Make sure that Opera fires events after an aborted Ajax attempt. Fixes #5787. --- Makefile | 2 +- src/ajax.js | 20 +++++++++++++++++++- 2 files changed, 20 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index fdb57bda..9221e088 100644 --- a/Makefile +++ b/Makefile @@ -57,7 +57,7 @@ ${JQ}: ${MODULES} sed 's/Date:./&'"${DATE}"'/' | \ ${VER} > ${JQ}; -selector: init +selector: @@echo "Building selector code from Sizzle" @@sed '/EXPOSE/r src/sizzle-jquery.js' src/sizzle/sizzle.js > src/selector.js diff --git a/src/ajax.js b/src/ajax.js index 193a85ae..07e706c4 100644 --- a/src/ajax.js +++ b/src/ajax.js @@ -393,7 +393,9 @@ jQuery.extend({ // The request was aborted, clear the interval and decrement jQuery.active if ( !xhr || xhr.readyState === 0 ) { requestDone = true; - xhr.onreadystatechange = jQuery.noop; + if ( xhr ) { + xhr.onreadystatechange = jQuery.noop; + } // Handle the global AJAX counter if ( s.global && ! --jQuery.active ) { @@ -447,6 +449,22 @@ jQuery.extend({ } }; + // Override the abort handler, if we can (IE doesn't allow it, but that's OK) + // Opera doesn't fire onreadystatechange at all on abort + try { + var oldAbort = xhr.abort; + xhr.abort = function() { + oldAbort.call( xhr ); + if ( xhr ) { + xhr.readyState = 0; + } + if ( !requestDone ) { + complete(); + } + onreadystatechange(); + }; + } catch(e) { } + // Timeout checker if ( s.async && s.timeout > 0 ) { setTimeout(function() {