From 9353d82648d922f9ef018483c0c4da78741587e9 Mon Sep 17 00:00:00 2001
From: Jan Bader <jan@javil.eu>
Date: Mon, 31 Jan 2022 21:45:02 +0000
Subject: [PATCH] Fix diverse errors

---
 web/src/dialogs/NewBudget.vue |  5 ++--
 web/src/pages/Settings.vue    |  3 +-
 web/src/store/index.ts        | 52 ++++++++++++++++++-----------------
 3 files changed, 32 insertions(+), 28 deletions(-)

diff --git a/web/src/dialogs/NewBudget.vue b/web/src/dialogs/NewBudget.vue
index 65869f4..07caffc 100644
--- a/web/src/dialogs/NewBudget.vue
+++ b/web/src/dialogs/NewBudget.vue
@@ -1,4 +1,5 @@
-<script>
+<script>import { NEW_BUDGET } from "../store/action-types";
+
   export default {
     data () {
       return {
@@ -8,7 +9,7 @@
     },
     methods: {
       saveBudget () {
-        this.$store.dispatch("newBudget", this.$data.budgetName);
+        this.$store.dispatch(NEW_BUDGET, this.$data.budgetName);
         this.$data.dialog = false;
       },
       newBudget () {
diff --git a/web/src/pages/Settings.vue b/web/src/pages/Settings.vue
index c4006a4..0ca0fce 100644
--- a/web/src/pages/Settings.vue
+++ b/web/src/pages/Settings.vue
@@ -1,5 +1,6 @@
 <script lang="ts">
 import { defineComponent } from "vue"
+import { IMPORT_YNAB } from "../store/action-types";
 import { TITLE } from "../store/mutation-types"
 
 export default defineComponent({
@@ -49,7 +50,7 @@ export default defineComponent({
             let formData = new FormData();
             formData.append("transactions", this.$data.transactionsFile);
             formData.append("assignments", this.$data.assignmentsFile);
-            this.$store.dispatch("YNAB", formData);
+            this.$store.dispatch(IMPORT_YNAB, formData);
         }
     }
 })
diff --git a/web/src/store/index.ts b/web/src/store/index.ts
index 89eac63..e361db1 100644
--- a/web/src/store/index.ts
+++ b/web/src/store/index.ts
@@ -1,7 +1,7 @@
 import { InjectionKey } from 'vue'
 import { createStore, Store, createLogger } from 'vuex'
 import { LOGIN_SUCCESS, LOGOUT, TITLE } from './mutation-types'
-import { FETCH_ACCOUNT, FETCH_BUDGET, GET, REGISTER, IMPORT_YNAB, LOGIN, NEW_BUDGET, POST, SET_CURRENT_ACCOUNT, SET_CURRENT_BUDGET} from './action-types'
+import { FETCH_ACCOUNT, FETCH_BUDGET, GET, REGISTER, IMPORT_YNAB, LOGIN, NEW_BUDGET, POST, SET_CURRENT_ACCOUNT, SET_CURRENT_BUDGET } from './action-types'
 
 export interface State {
     Session: {
@@ -32,21 +32,21 @@ export const key: InjectionKey<Store<State>> = Symbol()
 
 export const store = createStore<State>({
     state: {
-            Session: {
-                Token: undefined,
-                User: undefined
-            },
-            ShowMenu: undefined,
-            Budgets: new Map<string, Budget>(),
-            CurrentBudgetID: undefined,
-            Accounts: new Map<string, Account>(),
-            CurrentAccountID: undefined,
-            Categories: [],
-            Transactions: [],
-            Assignments: []
+        Session: {
+            Token: undefined,
+            User: undefined
+        },
+        ShowMenu: undefined,
+        Budgets: new Map<string, Budget>(),
+        CurrentBudgetID: undefined,
+        Accounts: new Map<string, Account>(),
+        CurrentAccountID: undefined,
+        Categories: [],
+        Transactions: [],
+        Assignments: []
     },
     mutations: {
-        deleteBudget(state : State, budgetid : string) {
+        deleteBudget(state: State, budgetid: string) {
             state.Budgets.delete(budgetid)
         },
         toggleMenu(state) {
@@ -58,20 +58,20 @@ export const store = createStore<State>({
                 return;
 
             const restoredState = JSON.parse(store);
-            if(!restoredState)
+            if (!restoredState)
                 return;
-            
+
             state.Session = restoredState.Session;
-            state.CurrentBudgetID= restoredState.CurrentBudgetID;
-            state.CurrentAccountID= restoredState.CurrentAccountID;
+            state.CurrentBudgetID = restoredState.CurrentBudgetID;
+            state.CurrentAccountID = restoredState.CurrentAccountID;
             state.ShowMenu = restoredState.ShowMenu;
-                
+
             for (const budget of restoredState.Budgets) {
                 state.Budgets.set(budget[0], budget[1]);
-            }        
+            }
             for (const account of restoredState.Accounts) {
                 state.Accounts.set(account[0], account[1]);
-            }        
+            }
         },
         [TITLE](state, title) {
             document.title = "Budgeteer - " + title;
@@ -88,6 +88,9 @@ export const store = createStore<State>({
         addBudget(state, budget) {
             state.Budgets.set(budget.ID, budget);
         },
+        addAccount(state, account) {
+            state.Accounts.set(account.ID, account);
+        },
         [LOGOUT](state, token) {
             state.Session = { Token: undefined, User: undefined };
             state.Budgets.clear();
@@ -102,9 +105,6 @@ export const store = createStore<State>({
         setCurrentAccountID(state, accountid) {
             state.CurrentAccountID = accountid;
         },
-        setAccounts(state, accounts) {
-            state.Accounts = accounts;
-        },
         setTransactions(state, transactions) {
             state.Transactions = transactions;
         }
@@ -163,7 +163,9 @@ export const store = createStore<State>({
         async [FETCH_BUDGET]({ state, commit, dispatch, rootState }, budgetid) {
             const result = await dispatch("GET", { path: "/budget/" + budgetid });
             const response = await result.json();
-            return commit("setAccounts", response.Accounts);
+            for (const account of response.Accounts || []) {
+                commit("addAccount", account);
+            }
         },
         async [FETCH_ACCOUNT]({ state, commit, rootState }, accountid) {
             const result = await fetch("/api/v1/account/" + accountid + "/transactions", {