1 files changed, 75 insertions, 75 deletions
diff --git a/apps/files/src/store/viewConfig.ts b/apps/files/src/store/viewConfig.ts
index 76786306d60..a902cedb6fa 100644
--- a/apps/files/src/store/viewConfig.ts
+++ b/apps/files/src/store/viewConfig.ts
@@ -2,95 +2,95 @@
  * SPDX-FileCopyrightText: 2023 Nextcloud GmbH and Nextcloud contributors
  * SPDX-License-Identifier: AGPL-3.0-or-later
  */
-import { defineStore } from 'pinia'
+import type { ViewConfigs, ViewId, ViewConfig } from '../types'
+
+import { getCurrentUser } from '@nextcloud/auth'
 import { emit, subscribe } from '@nextcloud/event-bus'
-import { generateUrl } from '@nextcloud/router'
 import { loadState } from '@nextcloud/initial-state'
+import { generateUrl } from '@nextcloud/router'
+import { defineStore } from 'pinia'
+import { ref, set } from 'vue'
 import axios from '@nextcloud/axios'
-import Vue from 'vue'
 
-import type { ViewConfigs, ViewConfigStore, ViewId, ViewConfig } from '../types'
+const initialViewConfig = loadState('files', 'viewConfigs', {}) as ViewConfigs
 
-const viewConfig = loadState('files', 'viewConfigs', {}) as ViewConfigs
+export const useViewConfigStore = defineStore('viewconfig', () => {
 
-export const useViewConfigStore = function(...args) {
-	const store = defineStore('viewconfig', {
-		state: () => ({
-			viewConfig,
-		} as ViewConfigStore),
+	const viewConfigs = ref({ ...initialViewConfig })
 
-		getters: {
-			getConfig: (state) => (view: ViewId): ViewConfig => state.viewConfig[view] || {},
+	/**
+	 * Get the config for a specific view
+	 * @param viewid Id of the view to fet the config for
+	 */
+	function getConfig(viewid: ViewId): ViewConfig {
+		return viewConfigs.value[viewid] || {}
+	}
 
-			getConfigs: (state) => (): ViewConfigs => state.viewConfig,
-		},
+	/**
+	 * Update the view config local store
+	 * @param viewId The id of the view to update
+	 * @param key The config key to update
+	 * @param value The new value
+	 */
+	function onUpdate(viewId: ViewId, key: string, value: string | number | boolean): void {
+		if (!(viewId in viewConfigs.value)) {
+			set(viewConfigs.value, viewId, {})
+		}
+		set(viewConfigs.value[viewId], key, value)
+	}
 
-		actions: {
-			/**
-			 * Update the view config local store
-			 * @param view
-			 * @param key
-			 * @param value
-			 */
-			onUpdate(view: ViewId, key: string, value: string | number | boolean) {
-				if (!this.viewConfig[view]) {
-					Vue.set(this.viewConfig, view, {})
-				}
-				Vue.set(this.viewConfig[view], key, value)
-			},
+	/**
+	 * Update the view config local store AND on server side
+	 * @param view Id of the view to update
+	 * @param key Config key to update
+	 * @param value New value
+	 */
+	async function update(view: ViewId, key: string, value: string | number | boolean): Promise<void> {
+		if (getCurrentUser() !== null) {
+			await axios.put(generateUrl('/apps/files/api/v1/views'), {
+				value,
+				view,
+				key,
+			})
+		}
 
-			/**
-			 * Update the view config local store AND on server side
-			 * @param view
-			 * @param key
-			 * @param value
-			 */
-			async update(view: ViewId, key: string, value: string | number | boolean) {
-				axios.put(generateUrl('/apps/files/api/v1/views'), {
-					value,
-					view,
-					key,
-				})
+		emit('files:view-config:updated', { view, key, value })
+	}
 
-				emit('files:viewconfig:updated', { view, key, value })
-			},
+	/**
+	 * Set the sorting key AND sort by ASC
+	 * The key param must be a valid key of a File object
+	 * If not found, will be searched within the File attributes
+	 * @param key Key to sort by
+	 * @param view View to set the sorting key for
+	 */
+	function setSortingBy(key = 'basename', view = 'files'): void {
+		// Save new config
+		update(view, 'sorting_mode', key)
+		update(view, 'sorting_direction', 'asc')
+	}
 
-			/**
-			 * Set the sorting key AND sort by ASC
-			 * The key param must be a valid key of a File object
-			 * If not found, will be searched within the File attributes
-			 * @param key Key to sort by
-			 * @param view View to set the sorting key for
-			 */
-			setSortingBy(key = 'basename', view = 'files') {
-				// Save new config
-				this.update(view, 'sorting_mode', key)
-				this.update(view, 'sorting_direction', 'asc')
-			},
+	/**
+	 * Toggle the sorting direction
+	 * @param viewId id of the view to set the sorting order for
+	 */
+	function toggleSortingDirection(viewId = 'files'): void {
+		const config = viewConfigs.value[viewId] || { sorting_direction: 'asc' }
+		const newDirection = config.sorting_direction === 'asc' ? 'desc' : 'asc'
 
-			/**
-			 * Toggle the sorting direction
-			 * @param view view to set the sorting order for
-			 */
-			toggleSortingDirection(view = 'files') {
-				const config = this.getConfig(view) || { sorting_direction: 'asc' }
-				const newDirection = config.sorting_direction === 'asc' ? 'desc' : 'asc'
+		// Save new config
+		update(viewId, 'sorting_direction', newDirection)
+	}
 
-				// Save new config
-				this.update(view, 'sorting_direction', newDirection)
-			},
-		},
-	})
+	// Initialize event listener
+	subscribe('files:view-config:updated', ({ view, key, value }) => onUpdate(view, key, value))
 
-	const viewConfigStore = store(...args)
+	return {
+		viewConfigs,
 
-	// Make sure we only register the listeners once
-	if (!viewConfigStore._initialized) {
-		subscribe('files:viewconfig:updated', function({ view, key, value }: { view: ViewId, key: string, value: boolean }) {
-			viewConfigStore.onUpdate(view, key, value)
-		})
-		viewConfigStore._initialized = true
+		getConfig,
+		setSortingBy,
+		toggleSortingDirection,
+		update,
 	}
-
-	return viewConfigStore
-}
+})