From: Robert Kaiser <kairo@kairo.at>
Date: Mon, 20 Nov 2017 18:26:09 +0000 (+0100)
Subject: improve error handling, actually print the error message if one is sent
X-Git-Tag: production~23
X-Git-Url: https://git-public.kairo.at/?p=lantea.git;a=commitdiff_plain;h=5e9f4a24c890ea34a762380ac01d76648a8547e5;ds=sidebyside

improve error handling, actually print the error message if one is sent
---

diff --git a/js/ui.js b/js/ui.js
index f451fbe..263a139 100644
--- a/js/ui.js
+++ b/js/ui.js
@@ -527,21 +527,31 @@ function uploadTrack() {
       if (aStatusCode >= 400) {
         reportUploadStatus(false, aResult);
       }
-      else {
+      else if (aResult["id"]) {
         reportUploadStatus(true);
       }
+      else { // If no ID is returned, we assume a general error.
+        reportUploadStatus(false);
+      }
     }
   );
 }
 
-function reportUploadStatus(aSuccess, aMessage) {
+function reportUploadStatus(aSuccess, aResponse) {
   document.getElementById("uploadStatusCloseButton").disabled = false;
   document.getElementById("uploadInProgress").style.display = "none";
   if (aSuccess) {
     document.getElementById("uploadSuccess").style.display = "block";
   }
-  else if (aMessage) {
-    document.getElementById("uploadErrorMsg").textContent = aMessage;
+  else if (aResponse["message"]) {
+    document.getElementById("uploadErrorMsg").textContent = aResponse["message"];
+    if (aResponse["errortype"]) {
+      document.getElementById("uploadErrorMsg").textContent += " (" + aResponse["errortype"] + ")";
+    }
+    document.getElementById("uploadError").style.display = "block";
+  }
+  else if (aResponse) {
+    document.getElementById("uploadErrorMsg").textContent = aResponse;
     document.getElementById("uploadError").style.display = "block";
   }
   else {