Side navigation
#7201 closed bug ()
Opened October 15, 2010 12:30PM UTC
Closed November 11, 2010 11:09PM UTC
Last modified January 31, 2011 05:55PM UTC
Exception thrown by $.ajax.error callback cannot be caught by outer try...catch (for a synchronous request)
Reported by: | cipinistor@gmail.com | Owned by: | cipinistor@gmail.com |
---|---|---|---|
Priority: | low | Milestone: | 1.5 |
Component: | ajax | Version: | 1.5 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
Making a synchronous $.ajax request that completes with a server exception, an exception thrown inside the "error" callback cannot be caught by an outer try...catch.
That is because in 1.4.2 the result of the request is handled in an asynchronous manner (by attaching a handler to the xhr.onreadystatechange event, even for synchronous requests) thus making it impossible to catch an exception from the error callback.
A possible fix for this could be modifying this line:
var onreadystatechange = xhr.onreadystatechange = function( isTimeout ) {
into:
var onreadystatechange = function( isTimeout ) {
and after the onreadystatechange code block, this should be added:
if (s.async)
{
xhr.onreadystatechange = onreadystatechange;
}
Attachments (0)
Change History (5)
Changed October 15, 2010 03:45PM UTC by comment:1
component: | unfiled → ajax |
---|---|
owner: | → cipinistor@gmail.com |
priority: | undecided → low |
status: | new → pending |
Changed October 21, 2010 10:57PM UTC by comment:2
milestone: | 1.4.4 → 1.4.5 |
---|
Retargeting due to severity of regressions in 1.4.3.
Changed November 11, 2010 11:09PM UTC by comment:3
status: | pending → closed |
---|
Automatically closed due to 14 days of inactivity.
Changed January 31, 2011 05:53PM UTC by comment:4
version: | 1.4.2 → 1.5 |
---|
There wasn't a 1.4.5 release, was actually 1.5.
Changed January 31, 2011 05:55PM UTC by comment:5
milestone: | 1.4.5 → 1.5 |
---|
There was no 1.4.5 release, was actually 1.5.
Please provide a valid test case that we can use to verify the behavior described in your ticket.