79 felmeddelande vid utbetalning om kontouppgifter saknas

expenses/views.py

@@ -329,6 +329,12 @@ def new_payment(request):
             return HttpResponseBadRequest("Alla kvitton måste ha samma ägare")
 
     if expense_owner.bank_name == "" or expense_owner.bank_account == "" or expense_owner.sorting_number == "":
+        for expense in expenses:
+            models.Comment(
+                author=request.user.profile,
+                expense=expense,
+                content="Utbetalning misslyckades: användaren saknar fullständiga bankuppgifter"
+            ).save()
         return HttpResponseBadRequest("Användaren har inte angett alla sina bankuppgifter")
 
     payment = models.Payment(
@@ -369,6 +375,12 @@ def api_new_payment(request):
             return HttpResponseBadRequest("Alla kvitton måste ha samma ägare")
 
     if expense_owner.bank_name == "" or expense_owner.bank_account == "" or expense_owner.sorting_number == "":
+        for expense in expenses:
+            models.Comment(
+                author=request.user.profile,
+                expense=expense,
+                content="Utbetalning misslyckades: användaren saknar fullständiga bankuppgifter"
+            ).save()
         return HttpResponseBadRequest("Användaren har inte angett alla sina bankuppgifter")
 
     payment = models.Payment(

templates/admin/pay/overview.html

@@ -5,6 +5,9 @@
 {% block content %}
 <div id="app">
     <p>Här kan du hitta alla kvitton som är redo att betalas ut ordnade efter användare.</p>
+    <ul class="messages" v-if="error_message">
+        <li v-text="error_message"></li>
+    </ul>
     <div class="select" style="margin-left: 10px;float:right">
         <select v-model="cost_centre">
             <option :value="false">Filtrera på resultatställe</option>
@@ -142,10 +145,12 @@ <h1>Genomför nedanstående betalning</h1>
                 expenses: {% autoescape off %}{{ expenses }}{% endautoescape %},
                 invoices: {% autoescape off %}{{ invoices }}{% endautoescape %},
                 payment: null,
+                error_message: '',
             }
         },
         methods: {
             save: function (user, e) {
+                this.error_message = ''
                 let exps = this.expenses.filter(x => x.checked && x.owner == user && (this.cost_centre == false || x.cost_centres.indexOf(this.cost_centre) > -1))
                 if (exps.length < 1) {
                     console.log('Not enough expenses')
@@ -163,7 +168,25 @@ <h1>Genomför nedanstående betalning</h1>
                     credentials: 'same-origin',
                     body: form
                 })
-                .then(res => res.json())
+                .then(async res => {
+                    const body = await res.text()
+                    let parsed = null
+
+                    if (body) {
+                        try {
+                            parsed = JSON.parse(body)
+                        } catch (err) {
+                            parsed = null
+                        }
+                    }
+
+                    if (!res.ok) {
+                        const message = (parsed && parsed.error) || body || 'Utbetalningen misslyckades.'
+                        throw new Error(message)
+                    }
+
+                    return parsed || {}
+                })
                 .then(res => {
                     // Done
                     console.log(res)
@@ -176,6 +199,9 @@ <h1>Genomför nedanstående betalning</h1>
                     })
                     this.sortExpensesByOwnerTotal()
                 })
+                .catch(err => {
+                    this.error_message = err.message || 'Utbetalningen misslyckades.'
+                })
                 if (e) e.preventDefault()
             },
             trim(id, event) {