From ddaf647d8716c458a992136d72968028cdc9ec77 Mon Sep 17 00:00:00 2001
From: Jan Bader <jan@javil.eu>
Date: Sat, 5 Feb 2022 14:26:56 +0000
Subject: [PATCH] Add getters for current budget ID and Name

---
 web/src/App.vue                 |  2 +-
 web/src/pages/Account.vue       |  2 +-
 web/src/pages/BudgetSidebar.vue |  8 ++++----
 web/src/store/index.ts          | 22 +++++++++++++++++-----
 4 files changed, 23 insertions(+), 11 deletions(-)

diff --git a/web/src/App.vue b/web/src/App.vue
index d6c5013..d6cea96 100644
--- a/web/src/App.vue
+++ b/web/src/App.vue
@@ -28,7 +28,7 @@ export default defineComponent({
     <div class="flex bg-gray-400 p-4 m-2 rounded-lg">
       <span class="flex-1 font-bold text-5xl -my-3" @click="toggleMenu">≡</span>
 
-      <span class="flex-1">{{$store.getters.CurrentBudget.Name}}</span>
+      <span class="flex-1">{{$store.getters.CurrentBudgetName}}</span>
 
       <div class="flex flex-1 flex-row justify-end -mx-4">
         <router-link class="mx-4" v-if="loggedIn" to="/dashboard">Dashboard</router-link>
diff --git a/web/src/pages/Account.vue b/web/src/pages/Account.vue
index aa4537b..963054c 100644
--- a/web/src/pages/Account.vue
+++ b/web/src/pages/Account.vue
@@ -82,7 +82,7 @@ export default defineComponent({
                 >{{ transaction.CategoryGroup ? transaction.CategoryGroup + " : " + transaction.Category : "" }}</td>
                 <td>
                     <a
-                        :href="'/budget/' + $store.getters.CurrentBudget.ID + '/transaction/' + transaction.ID"
+                        :href="'/budget/' + $store.getters.CurrentBudgetID + '/transaction/' + transaction.ID"
                     >{{ transaction.Memo }}</a>
                 </td>
                 <td
diff --git a/web/src/pages/BudgetSidebar.vue b/web/src/pages/BudgetSidebar.vue
index 39d7e84..ad35a40 100644
--- a/web/src/pages/BudgetSidebar.vue
+++ b/web/src/pages/BudgetSidebar.vue
@@ -10,12 +10,12 @@ export default defineComponent({
   <div class="flex flex-col">
     <span class="m-1 p-1 px-3 text-xl">
       <router-link to="/dashboard">⌂</router-link>
-      {{$store.getters.CurrentBudget.Name}}
+      {{$store.getters.CurrentBudgetName}}
     </span>
     <span class="bg-orange-200 rounded-lg m-1 p-1 px-3 flex flex-col">
       <router-link :to="'/budget/'+budgetid">Budget</router-link><br />
       <span>Reports (Coming Soon)</span>
-      <router-link :to="'/budget/'+$store.getters.CurrentBudget.ID+'/all-accounts'">All Accounts</router-link>
+      <router-link :to="'/budget/'+$store.getters.CurrentBudgetID+'/all-accounts'">All Accounts</router-link>
     </span>
     <li class="bg-orange-200 rounded-lg m-1 p-1 px-3">
       On-Budget Accounts
@@ -35,13 +35,13 @@ export default defineComponent({
       Closed Accounts
     </li>
     <!--<li>
-      <router-link :to="'/budget/'+$store.getters.CurrentBudget.ID+'/accounts'">Edit accounts</router-link>
+      <router-link :to="'/budget/'+$store.getters.CurrentBudgetID+'/accounts'">Edit accounts</router-link>
     </li>-->
     <li class="bg-red-200 rounded-lg m-1 p-1 px-3">
       + Add Account
     </li>
     <li class="bg-red-200 rounded-lg m-1 p-1 px-3">
-      <router-link :to="'/budget/'+$store.getters.CurrentBudget.ID+'/settings'">Budget-Settings</router-link>
+      <router-link :to="'/budget/'+$store.getters.CurrentBudgetID+'/settings'">Budget-Settings</router-link>
     </li>
     <!--<li><router-link to="/admin">Admin</router-link></li>-->
   </div>        
diff --git a/web/src/store/index.ts b/web/src/store/index.ts
index c835abd..fe50897 100644
--- a/web/src/store/index.ts
+++ b/web/src/store/index.ts
@@ -201,15 +201,27 @@ export const store = createStore<State>({
                 'Authorization': 'Bearer ' + state.Session.Token
             }
         },
-        CurrentBudget(state) {
+        CurrentBudget(state) : Budget | undefined {
             if (state.CurrentBudgetID == null)
-                return {};
+                return undefined;
 
-            return state.Budgets.get(state.CurrentBudgetID) || {};
+            return state.Budgets.get(state.CurrentBudgetID);
         },
-        CurrentAccount(state) {
+        CurrentBudgetID(state) : string | undefined {
+            return state.CurrentBudgetID;
+        },
+        CurrentBudgetName(state) : string {
+            if (state.CurrentBudgetID == null)
+                return "";
+
+            const currentBudget = state.Budgets.get(state.CurrentBudgetID);
+            if(currentBudget != undefined)
+                return currentBudget.Name;
+            return "";
+        },
+        CurrentAccount(state) : Account | undefined {
             if (state.CurrentAccountID == null)
-                return { name: "Not found" };
+                return undefined;
             return state.Accounts.get(state.CurrentAccountID);
         },
         OnBudgetAccounts(state) {