search

ivan-novakov / extjs-upload-widget

File upload widget for ExtJS v4
81 stars 42 forks source link

Ext.ux.upload.uploader.AbstractXhrUploader onUploadSuccess #22

Closed imaximix closed 10 years ago

imaximix commented 10 years ago

This method checks for reponse.responseText . Shouldn't it check for response.status?

ivan-novakov commented 10 years ago

Good point. It should. I'll change it.

imaximix commented 10 years ago

If you don't have time, i can send a PR.

ivan-novakov commented 10 years ago

Of course, that would help, thanks!