From 932a5462134584d6c70f22dc97331e73dab3876f Mon Sep 17 00:00:00 2001
From: Yohann D'ANELLO <yohann.danello@gmail.com>
Date: Fri, 31 Jul 2020 19:43:03 +0200
Subject: [PATCH] Better error messages

---
 static/js/base.js     |  2 +-
 static/js/consos.js   |  4 ++--
 static/js/transfer.js | 15 +++++----------
 3 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/static/js/base.js b/static/js/base.js
index 77057103..fdad58d7 100644
--- a/static/js/base.js
+++ b/static/js/base.js
@@ -369,7 +369,7 @@ function de_validate(id, validated) {
         },
         error: function (err) {
             addMsg("Une erreur est survenue lors de la validation/dévalidation " +
-                "de cette transaction : " + err.responseText, "danger");
+                "de cette transaction : " + JSON.parse(err.responseText)["detail"], "danger", 10000);
 
             refreshBalance();
             // error if this method doesn't exist. Please define it.
diff --git a/static/js/consos.js b/static/js/consos.js
index 2fa77249..268ddc95 100644
--- a/static/js/consos.js
+++ b/static/js/consos.js
@@ -207,10 +207,10 @@ function consume(source, source_alias, dest, quantity, amount, reason, type, cat
                 "template": template
             }).done(function() {
                 reset();
-                addMsg("La transaction n'a pas pu être validée pour cause de solde insuffisant.", "danger");
+                addMsg("La transaction n'a pas pu être validée pour cause de solde insuffisant.", "danger", 10000);
             }).fail(function () {
                 reset();
-                errMsg(e.responseJSON);
+                errMsg(e.responseJSON, 10000);
             });
     });
 }
diff --git a/static/js/transfer.js b/static/js/transfer.js
index ed651640..4234e775 100644
--- a/static/js/transfer.js
+++ b/static/js/transfer.js
@@ -189,7 +189,7 @@ $("#btn_transfer").click(function() {
                     }).done(function () {
                         addMsg("Le transfert de "
                             + pretty_money(source.quantity * dest.quantity * 100 * $("#amount").val()) + " de la note " + source.name
-                            + " vers la note " + dest.name + " a été fait avec succès !", "success");
+                            + " vers la note " + dest.name + " a été fait avec succès !", "success", 10000);
 
                         reset();
                     }).fail(function (err) { // do it again but valid = false
@@ -210,15 +210,11 @@ $("#btn_transfer").click(function() {
                         }).done(function () {
                             addMsg("Le transfert de "
                                 + pretty_money(source.quantity * dest.quantity * 100 * $("#amount").val()) + " de la note " + source.name
-                                + " vers la note " + dest.name + " a échoué : Solde insuffisant", "danger");
-
-                            reset();
+                                + " vers la note " + dest.name + " a échoué : Solde insuffisant", "danger", 10000);
                         }).fail(function (err) {
                             addMsg("Le transfert de "
                                 + pretty_money(source.quantity * dest.quantity * 100 * $("#amount").val()) + " de la note " + source.name
-                                + " vers la note " + dest.name + " a échoué : " + err.responseText, "danger");
-
-                            reset();
+                                + " vers la note " + dest.name + " a échoué : " + JSON.parse(err.responseText)["detail"], "danger", 10000);
                     });
                 });
             });
@@ -261,11 +257,10 @@ $("#btn_transfer").click(function() {
                 "first_name": $("#first_name").val(),
                 "bank": $("#bank").val()
             }).done(function () {
-                addMsg("Le crédit/retrait a bien été effectué !", "success");
+                addMsg("Le crédit/retrait a bien été effectué !", "success", 10000);
                 reset();
             }).fail(function (err) {
-                addMsg("Le crédit/retrait a échoué : " + err.responseText, "danger");
-                reset();
+                addMsg("Le crédit/retrait a échoué : " + JSON.parse(err.responseText)["detail"], "danger", 10000);
         });
     }
 });
-- 
GitLab