Ext.Ajax.Request executes the success handler when it fails

[yoh.suzuki]

I'm using Sencha Touch 2.0 (release).

Ext.Ajax.request({
    url: 'https://somewhere.com/api/404', 
    jsonData: {}, 
    success: function() {
        console.log('success')
    }
}); // <-- the success handler is executed when the request fails

[yoh.suzuki]

(Oops. The title should read "Ext.Ajax.request".)

[yoh.suzuki]

Inside the onComplete and parseStatus functions of Ext.data.Connection, request.xhr.status is interpreted as success when it is 0, which I believe is incorrect according to the W3C specification.

http://www.w3.org/TR/XMLHttpRequest/#the-status-attribute

[yoh.suzuki]

I'm sure there is a reason this was added sometime between PR3 and 2.0 Release, but for now, I'm using this workaround:

Ext.define('Ext.data.Connection.fixed', {
    override: 'Ext.data.Connection',
    parseStatus: function(status) {
        status = status == 1223 ? 204 : status;
        var success = (status >= 200 && status < 300) || status == 304 /*|| status == 0*/, // <-- 0 is bad!
            isException = false;
        if (!success) {
            switch (status) {
                case 12002:
                case 12029:
                case 12030:
                case 12031:
                case 12152:
                case 13030:
                    isException = true;
                    break;
            }
        }
        return {
            success: success,
            isException: isException
        };
    }
});

[Jamie Avins]

Thank you for the report.

[edspencer]

Ah I think we did this to support the microloader when using native packaging. I'll confer with Jacky to see if we can come up with a better solution

[Jacky Nguyen]

This has been addressed for 2.0.1 release