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.

1 files changed, 11 insertions, 4 deletions

diff --git a/app/assets/javascripts/srp/src/jqueryRest.js b/app/assets/javascripts/srp/src/jqueryRest.js
index 0c58eb2..437bd65 100644
--- a/app/assets/javascripts/srp/src/jqueryRest.js
+++ b/app/assets/javascripts/srp/src/jqueryRest.js
@@ -88,10 +88,17 @@ srp.remote = (function(){
   // the http error response.
   function error(xhr, text, thrown)
   {
-    if (xhr.responseText && xhr.responseText != "")
-      srp.error($.parseJSON(xhr.responseText));
-    else
-      srp.error("Server did not respond.");
+    var message;
+    if (xhr.responseText && xhr.responseText != "") {
+      try {
+        message = $.parseJSON(xhr.responseText);
+      } catch (err) {
+        message = xhr.responseText;
+      }
+    } else {
+      message = "Server did not respond.";
+    }
+    srp.error(message);
   };
 
   return {