From f0c37a74c1800cb4a3c992cf335cb7db57fdedaa Mon Sep 17 00:00:00 2001
From: Knut Ahlers <knut@ahlers.me>
Date: Thu, 18 Jan 2024 12:09:32 +0100
Subject: [PATCH] Fix: Do not display / send category in tracking accounts

& refetch transactions when creating transfers
---
 frontend/components/accountOverview.vue | 16 +++++++++++++---
 frontend/components/budgetDashboard.vue |  1 +
 2 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/frontend/components/accountOverview.vue b/frontend/components/accountOverview.vue
index 5334d90..294f555 100644
--- a/frontend/components/accountOverview.vue
+++ b/frontend/components/accountOverview.vue
@@ -53,7 +53,9 @@
                   Date
                 </th>
                 <th>Payee</th>
-                <th>Category</th>
+                <th v-if="account.type !== 'tracking'">
+                  Category
+                </th>
                 <th>Description</th>
                 <th class="minimized-column text-end">
                   Amount
@@ -80,7 +82,7 @@
                     type="text"
                   >
                 </td>
-                <td>
+                <td v-if="account.type !== 'tracking'">
                   <select
                     v-model="form.category"
                     class="form-select form-select-sm"
@@ -132,7 +134,9 @@
                   {{ new Date(tx.time).toLocaleDateString() }}
                 </td>
                 <td>{{ tx.payee }}</td>
-                <td>{{ accountIdToName[tx.category] }}</td>
+                <td v-if="account.type !== 'tracking'">
+                  {{ accountIdToName[tx.category] }}
+                </td>
                 <td>{{ tx.description }}</td>
                 <td :class="{'minimized-column text-end': true, 'text-danger': tx.amount < 0}">
                   {{ formatNumber(tx.amount) }} €
@@ -285,6 +289,10 @@ export default {
   components: { rangeSelector },
 
   computed: {
+    account() {
+      return this.accounts.filter(acc => acc.id === this.accountId)[0] || {}
+    },
+
     accountIdToName() {
       return Object.fromEntries(this.accounts.map(acc => [acc.id, acc.name]))
     },
@@ -379,6 +387,7 @@ export default {
         body: JSON.stringify({
           ...this.form,
           account: this.accountId,
+          category: this.form.category || null,
           time: new Date(this.form.date),
         }),
         headers: {
@@ -451,6 +460,7 @@ export default {
       })
         .then(() => {
           this.$emit('update-accounts')
+          this.fetchTransactions()
           Modal.getInstance(this.$refs.transferMoneyModal).toggle()
 
           this.modals.createTransfer = {
diff --git a/frontend/components/budgetDashboard.vue b/frontend/components/budgetDashboard.vue
index 26bde43..55a6ae8 100644
--- a/frontend/components/budgetDashboard.vue
+++ b/frontend/components/budgetDashboard.vue
@@ -283,6 +283,7 @@ export default {
       })
         .then(() => {
           this.$emit('update-accounts')
+          this.fetchTransactions()
           Modal.getInstance(this.$refs.transferMoneyModal).toggle()
 
           this.modals.createTransfer = {