diff options
author | Azul <azul@riseup.net> | 2017-03-23 16:35:42 +0100 |
---|---|---|
committer | Azul <azul@riseup.net> | 2017-03-23 16:35:42 +0100 |
commit | c11e31acf3a592b8560459469ae4acf0e6e5dfd4 (patch) | |
tree | 9130225ec9af905aaad9ed7f8fa64dbc22d574da /app/assets/javascripts/users.js | |
parent | 94ba8eae189ef6d60ddfa194e73b672123e17d6f (diff) |
fix: display error js responses in dev environment
Sometimes the dev environment will send back a plain text response.
This causes the json parser to raise an exception and used to cause
the browser not to display any error message.
Now we dumpt the whole server response - which happes to also include the
backtrace. A lot more useful than doing nothing.
In production this should never happen as 500s get handled by the
ExceptionApplication / our ErrorsController there.
Diffstat (limited to 'app/assets/javascripts/users.js')
-rw-r--r-- | app/assets/javascripts/users.js | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/app/assets/javascripts/users.js b/app/assets/javascripts/users.js index 3dae526..4b5198f 100644 --- a/app/assets/javascripts/users.js +++ b/app/assets/javascripts/users.js @@ -151,11 +151,11 @@ } function extractErrors(message) { - if ($.isPlainObject(message) && message.errors) { - return message.errors; + if ($.isPlainObject(message)) { + return message.errors || { base: message.error }; } else { return { - base: (message.error || JSON.stringify(message)) + base: message }; } } |