Fix issue where an Ajax request in IE sometimes returns 1223 instead …

…of 204 as the status code. [#129 state:resolved] (adevadeh, gordyt, Andrew Dupont)
prototypejs · Oct 17, 2010 · 8b2ed97 · 8b2ed97
1 parent 05e0ebc
commit 8b2ed97
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,3 +1,5 @@
+* Fix issue where an Ajax request in IE sometimes returns 1223 instead of 204 as the status code. [#129 state:resolved] (adevadeh, gordyt, Andrew Dupont)
+
 * Add Object.isDate. [#443 state:resolved] (Nesterenko Dmitry, kangax, Samuel Lebeau, Andrew Dupont)
 
 * Handle cases where `document` or `document.documentElement` is passed into Element#getOffsetParent. Fixes IE errors with many layout/positioning methods. [#90 state:resolved] (Padraig Kennedy, Andrew Dupont)

diff --git a/src/ajax/request.js b/src/ajax/request.js
@@ -278,6 +278,8 @@ Ajax.Request = Class.create(Ajax.Base, {
 
   getStatus: function() {
     try {
+      // IE sometimes returns 1223 for a 204 response.
+      if (this.transport.status === 1223) return 204;
       return this.transport.status || 0;
     } catch (e) { return 0 }
   },