Suggestion: Ext.form.Action handleResponse w/try...catch

[MarkT]

The default form actions' handleResponse() doesn't handle the possible SyntaxError thrown by Ext.decode(). I'd suggest the following:

handleResponse : function(response){
if(this.form.errorReader){
var rs = this.form.errorReader.read(response);
var errors = [];
if(rs.records){
for(var i = 0, len = rs.records.length; i < len; i++) {
var r = rs.records[i];
errors[i] = r.data;
}
}
if(errors.length < 1){
errors = null;
}
return {
success : rs.success,
errors : errors
};
}
var result;
try {
result = Ext.decode(response.responseText);
} catch(err) {
result = false;
}
return result;
}