From 29a93358446dbf293f3c08e2f2857bec4e5fc9d7 Mon Sep 17 00:00:00 2001
From: Dominik Pschenitschni <mail@celement.de>
Date: Sun, 14 Nov 2021 21:34:34 +0100
Subject: [PATCH] feat: save current list view just once

---
 src/views/list/ListGantt.vue   | 8 --------
 src/views/list/ListKanban.vue  | 6 ------
 src/views/list/ListList.vue    | 7 -------
 src/views/list/ListTable.vue   | 8 +-------
 src/views/list/ListWrapper.vue | 5 +++++
 5 files changed, 6 insertions(+), 28 deletions(-)

diff --git a/src/views/list/ListGantt.vue b/src/views/list/ListGantt.vue
index 224f0539b..b01855b37 100644
--- a/src/views/list/ListGantt.vue
+++ b/src/views/list/ListGantt.vue
@@ -66,7 +66,6 @@
 
 <script setup>
 import { ref, computed } from 'vue'
-import { useRoute } from 'vue-router'
 import flatPickr from 'vue-flatpickr-component'
 
 import { i18n } from '@/i18n'
@@ -76,13 +75,6 @@ import ListWrapper from './ListWrapper'
 import GanttChart from '@/components/tasks/gantt-component'
 import Fancycheckbox from '@/components/input/fancycheckbox'
 
-import {saveListView} from '@/helpers/saveListView'
-
-const route = useRoute()
-// Save the current list view to local storage
-// We use local storage and not vuex here to make it persistent across reloads.
-saveListView(route.params.listId, route.name)
-
 const showTaskswithoutDates = ref(false)
 const dayWidth = ref(35)
 const dateFrom = ref(new Date((new Date()).setDate((new Date()).getDate() - 15)))
diff --git a/src/views/list/ListKanban.vue b/src/views/list/ListKanban.vue
index 5def5b15e..1836db35e 100644
--- a/src/views/list/ListKanban.vue
+++ b/src/views/list/ListKanban.vue
@@ -234,7 +234,6 @@ import cloneDeep from 'lodash.clonedeep'
 
 import BucketModel from '../../models/bucket'
 import {mapState} from 'vuex'
-import {saveListView} from '@/helpers/saveListView'
 import Rights from '../../models/constants/rights.json'
 import {LOADING, LOADING_MODULE} from '@/store/mutation-types'
 import ListWrapper from './ListWrapper'
@@ -299,11 +298,6 @@ export default {
 		}
 	},
 
-	created() {
-		// Save the current list view to local storage
-		// We use local storage and not vuex here to make it persistent across reloads.
-		saveListView(this.$route.params.listId, this.$route.name)
-	},
 	watch: {
 		loadBucketParameter: {
 			handler: 'loadBuckets',
diff --git a/src/views/list/ListList.vue b/src/views/list/ListList.vue
index d1a84710f..063caf577 100644
--- a/src/views/list/ListList.vue
+++ b/src/views/list/ListList.vue
@@ -133,14 +133,12 @@
 
 <script>
 import { ref } from 'vue'
-import { useRoute } from 'vue-router'
 
 import ListWrapper from './ListWrapper'
 import EditTask from '@/components/tasks/edit-task'
 import AddTask from '@/components/tasks/add-task'
 import SingleTaskInList from '@/components/tasks/partials/singleTaskInList'
 import { useTaskList } from '@/composables/taskList'
-import {saveListView} from '@/helpers/saveListView'
 import Rights from '../../models/constants/rights.json'
 import FilterPopup from '@/components/list/partials/filter-popup.vue'
 import {HAS_TASKS} from '@/store/mutation-types'
@@ -206,11 +204,6 @@ export default {
 
 		const taskList = useTaskList(beforeLoad)
 
-		// Save the current list view to local storage
-		// We use local storage and not vuex here to make it persistent across reloads.
-		const route = useRoute()
-		saveListView(route.params.listId, route.name)
-
 		taskList.initTaskList()
 
 		return {
diff --git a/src/views/list/ListTable.vue b/src/views/list/ListTable.vue
index fab090668..70cc7d910 100644
--- a/src/views/list/ListTable.vue
+++ b/src/views/list/ListTable.vue
@@ -184,7 +184,7 @@
 
 <script setup>
 import { ref, reactive, computed, toRaw } from 'vue'
-import { useRouter, useRoute } from 'vue-router'
+import { useRouter } from 'vue-router'
 
 import ListWrapper from './ListWrapper'
 import Done from '@/components/misc/Done.vue'
@@ -199,7 +199,6 @@ import Pagination from '@/components/misc/pagination.vue'
 import Popup from '@/components/misc/popup'
 
 import { useTaskList } from '@/composables/taskList'
-import {saveListView} from '@/helpers/saveListView'
 
 const ACTIVE_COLUMNS_DEFAULT = {
 	id: true,
@@ -288,11 +287,6 @@ const taskDetailRoutes = computed(() => Object.fromEntries(
 	])),
 ))
 
-// Save the current list view to local storage
-// We use local storage and not vuex here to make it persistent across reloads.
-const route = useRoute()
-saveListView(route.params.listId, route.name)
-
 function sort(property) {
 	const order = sortBy.value[property]
 	if (typeof order === 'undefined' || order === 'none') {
diff --git a/src/views/list/ListWrapper.vue b/src/views/list/ListWrapper.vue
index 04b82d6c2..131761907 100644
--- a/src/views/list/ListWrapper.vue
+++ b/src/views/list/ListWrapper.vue
@@ -59,11 +59,16 @@ import {store} from '@/store'
 import {CURRENT_LIST} from '@/store/mutation-types'
 
 import {getListTitle} from '@/helpers/getListTitle'
+import {saveListView} from '@/helpers/saveListView'
 import {saveListToHistory} from '@/modules/listHistory'
 import { useTitle } from '@/composables/useTitle'
 
 const route = useRoute()
 
+// Save the current list view to local storage
+// We use local storage and not vuex here to make it persistent across reloads.
+saveListView(route.params.listId, route.name)
+
 const listService = shallowRef(new ListService())
 const loadedListId = ref(0)