Treat a 304 HTTP status as a successful response. [#331 state:resolve…

…d] (Kenneth Kin Lum, Andrew Dupont)
prototypejs · Nov 2, 2010 · e65e921 · e65e921
1 parent 5dc12bd
commit e65e921
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,3 +1,5 @@
+* Treat a 304 HTTP status as a successful response. [#331 state:resolved] (Kenneth Kin Lum, Andrew Dupont)
+
 * Handle sparse arrays properly in `Array#_each` to match behavior with browsers' built-in `Array#forEach` (and ES5). [#790 state:resolved] (Andriy Tyurnikov, Yaffle, Andrew Dupont)
 
 * Make `Event.extend` work with legacy IE events in IE 9. (Andrew Dupont)

diff --git a/src/ajax/request.js b/src/ajax/request.js
@@ -270,7 +270,7 @@ Ajax.Request = Class.create(Ajax.Base, {
   **/
   success: function() {
     var status = this.getStatus();
-    return !status || (status >= 200 && status < 300);
+    return !status || (status >= 200 && status < 300) || status == 304;
   },
 
   getStatus: function() {