From a895bde6612e7a2b22a84b6ca7c583bafc9ebc9e Mon Sep 17 00:00:00 2001
From: kolaente <k@knt.li>
Date: Fri, 7 Oct 2022 17:18:15 +0200
Subject: [PATCH] fix: make sure subscriptions are properly inherited between
 lists and namespaces

---
 .../namespace/namespace-settings-dropdown.vue | 20 +++++++++++++++----
 src/stores/namespaces.ts                      | 11 ++++++++--
 2 files changed, 25 insertions(+), 6 deletions(-)

diff --git a/src/components/namespace/namespace-settings-dropdown.vue b/src/components/namespace/namespace-settings-dropdown.vue
index 778616215..fb71ba66a 100644
--- a/src/components/namespace/namespace-settings-dropdown.vue
+++ b/src/components/namespace/namespace-settings-dropdown.vue
@@ -33,14 +33,13 @@
 			>
 				{{ $t('menu.archive') }}
 			</dropdown-item>
-			<task-subscription
-				v-if="subscription"
+			<Subscription
 				class="has-no-shadow"
 				:is-button="false"
 				entity="namespace"
 				:entity-id="namespace.id"
 				:model-value="subscription"
-				@update:model-value="sub => subscription = sub"
+				@update:model-value="setSubscriptionInStore"
 				type="dropdown"
 			/>
 			<dropdown-item
@@ -59,9 +58,10 @@ import {ref, onMounted, type PropType} from 'vue'
 
 import Dropdown from '@/components/misc/dropdown.vue'
 import DropdownItem from '@/components/misc/dropdown-item.vue'
-import TaskSubscription from '@/components/misc/subscription.vue'
+import Subscription from '@/components/misc/subscription.vue'
 import type {INamespace} from '@/modelTypes/INamespace'
 import type {ISubscription} from '@/modelTypes/ISubscription'
+import {useNamespaceStore} from '@/stores/namespaces'
 
 const props = defineProps({
 	namespace: {
@@ -70,8 +70,20 @@ const props = defineProps({
 	},
 })
 
+const namespaceStore = useNamespaceStore()
+
 const subscription = ref<ISubscription | null>(null)
 onMounted(() => {
 	subscription.value = props.namespace.subscription
 })
+
+function setSubscriptionInStore(sub: ISubscription) {
+	subscription.value = sub
+	namespaceStore.setNamespaces([
+		{
+			...props.namespace,
+			subscription: sub,
+		},
+	])
+}
 </script>
diff --git a/src/stores/namespaces.ts b/src/stores/namespaces.ts
index 86a7f0268..5d83948ba 100644
--- a/src/stores/namespaces.ts
+++ b/src/stores/namespaces.ts
@@ -43,7 +43,7 @@ export const useNamespaceStore = defineStore('namespace', {
 		getNamespaceById: state => (namespaceId: INamespace['id']) => {
 			return state.namespaces.find(({id}) => id == namespaceId) || null
 		},
-	
+
 		searchNamespace() {
 			return (query: string) => (
 				search(query)
@@ -64,6 +64,13 @@ export const useNamespaceStore = defineStore('namespace', {
 			this.namespaces = namespaces
 			namespaces.forEach(n => {
 				add(n)
+
+				// Check for each list in that namespace if it has a subscription and set it if not
+				n.lists.forEach(l => {
+					if (l.subscription === null || l.subscription.entity !== 'list') {
+						l.subscription = n.subscription
+					}
+				})
 			})
 		},
 
@@ -203,5 +210,5 @@ export const useNamespaceStore = defineStore('namespace', {
 
 // support hot reloading
 if (import.meta.hot) {
-  import.meta.hot.accept(acceptHMRUpdate(useNamespaceStore, import.meta.hot))
+	import.meta.hot.accept(acceptHMRUpdate(useNamespaceStore, import.meta.hot))
 }
\ No newline at end of file