textStatus unavailable from ajaxError

[john.firebaugh]

The jQuery.ajax error callback is passed XMLHttpRequest, textStatus, and errorThrown parameters.

The ajaxError callback is passed event, XMLHttpRequest, ajaxOptions, and thrownError parameters. The "textStatus" parameter should also be made passed, so that global ajaxError callbacks have access to the same state as error callbacks.

Reported on forums here: ​http://forum.jquery.com/topic/textstatus-unavailable-from-ajaxerror

[Rick Waldron]

Thanks for taking the time to contribute to the jQuery project! Please provide a reduced jsFiddle test case to help us assess your ticket!

Additionally, test against the jQuery 0 GIT version to ensure the issue still exists. Be Excellent to eachother!

[danheberden]

Congratulations! You've placed the 8000th bug ticket!

[tianon]

The following diff implements the (backwards compatible!) fix found in one of the posts on the forum (created from latest "jQuery 0 GIT" code):

speed/jquery-basis.js

@@ -5306 +5306 @@
 
                 // Fire the global callback
                 if ( s.global ) {
-                        (s.context ? jQuery(s.context) : jQuery.event).trigger( "ajaxError", [xhr, s, e] );
+                        (s.context ? jQuery(s.context) : jQuery.event).trigger( "ajaxError", [xhr, s, e, status] );
                 }
         },
 

(I would attach a .patch, but Trac says no. :P)

[jitter]

This could be a valid enhancement but needs some discussion.

The main problem I have with this is, that it would create yet another different signature (speaking of parameter order). Where the situation in that regard is already bad enough with the different ajax callbacks. But maybe it can't be made worse by adding another parameter. Marking for discussion for a future release

[john]

As mentioned in another, similar, ticket - as soon as we can land this API change (1.7) we'll probably be removing, or significantly changing, global Ajax events.