diff options
author | John Resig <jeresig@gmail.com> | 2010-09-21 15:22:34 -0400 |
---|---|---|
committer | John Resig <jeresig@gmail.com> | 2010-09-21 15:22:34 -0400 |
commit | ba9e0fc177841bd74cc5ea4e52f09cd87d747bf5 (patch) | |
tree | f3440499fa7a0b0dbc2e0f5ad1669e3d45f6f165 /src/ajax.js | |
parent | 3a0a35288304ab5289a1815055623b18de4dc9f6 (diff) | |
download | jquery-ba9e0fc177841bd74cc5ea4e52f09cd87d747bf5.tar.gz jquery-ba9e0fc177841bd74cc5ea4e52f09cd87d747bf5.zip |
Use a different workaround for detecting when Opera finds a status 304 page. Fixes #6060.
Diffstat (limited to 'src/ajax.js')
-rw-r--r-- | src/ajax.js | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/ajax.js b/src/ajax.js index 78d9b24f1..aa4e3ec45 100644 --- a/src/ajax.js +++ b/src/ajax.js @@ -629,9 +629,11 @@ jQuery.extend( jQuery.ajax, { try { // IE error sometimes returns 1223 when it should be 204 so treat it as success, see #1450 return !xhr.status && location.protocol === "file:" || - // Opera returns 0 when status is 304 ( xhr.status >= 200 && xhr.status < 300 ) || - xhr.status === 304 || xhr.status === 1223 || xhr.status === 0; + xhr.status === 304 || xhr.status === 1223 || + // Opera returns a status of 0 for redirects - + // We can detect this by the fact that Opera also doesn't return any headers + xhr.status === 0 && !xhr.getAllResponseHeaders(); } catch(e) {} return false; @@ -651,7 +653,7 @@ jQuery.extend( jQuery.ajax, { } // Opera returns 0 when status is 304 - return xhr.status === 304 || xhr.status === 0; + return xhr.status === 304 || xhr.status === 0 && !xhr.getAllResponseHeaders(); }, httpData: function( xhr, type, s ) { |