chore(files): adjust code for breaking change of Navigation.setActive

Signed-off-by: Ferdinand Thiessen <[email protected]>

Author: susnux

apps/files/src/composables/useNavigation.spec.ts

@@ -41,7 +41,7 @@ describe('Composables: useNavigation', () => {
 		it('should return already active navigation', async () => {
 			const view = new nextcloudFiles.View({ getContents: () => Promise.reject(new Error()), icon: '<svg></svg>', id: 'view-1', name: 'My View 1', order: 0 })
 			navigation.register(view)
-			navigation.setActive(view)
+			navigation.setActive(view.id)
 			// Now the navigation is already set it should take the active navigation
 			const wrapper = mount(TestComponent)
 			expect((wrapper.vm as unknown as { currentView: View | null }).currentView).toBe(view)
@@ -55,7 +55,7 @@ describe('Composables: useNavigation', () => {
 			// no active navigation
 			expect((wrapper.vm as unknown as { currentView: View | null }).currentView).toBe(null)
 
-			navigation.setActive(view)
+			navigation.setActive(view.id)
 			// Now the navigation is set it should take the active navigation
 			expect((wrapper.vm as unknown as { currentView: View | null }).currentView).toBe(view)
 		})

apps/files/src/views/FilesNavigation.vue

@@ -178,7 +178,7 @@ export default defineComponent({
 		showView(view: View) {
 			// Closing any opened sidebar
 			window.OCA?.Files?.Sidebar?.close?.()
-			getNavigation().setActive(view)
+			getNavigation().setActive(view.id)
 			emit('files:navigation:changed', view)
 		},
 

apps/files_sharing/src/init-public.ts

@@ -1,8 +1,9 @@
-import type { Folder } from '@nextcloud/files'
-/**
+/*!
  * SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
  * SPDX-License-Identifier: AGPL-3.0-or-later
  */
+
+import type { Folder } from '@nextcloud/files'
 import type { ShareAttribute } from './sharing.d.ts'
 
 import { emit, subscribe, unsubscribe } from '@nextcloud/event-bus'
@@ -22,7 +23,12 @@ registerPublicFileShareView()
 // Get the current view from state and set it active
 const view = loadState<string>('files_sharing', 'view')
 const navigation = getNavigation()
-navigation.setActive(navigation.views.find(({ id }) => id === view) ?? null)
+try {
+	navigation.setActive(view)
+} catch {
+	// no such view
+	navigation.setActive(null)
+}
 
 // Force our own router
 window.OCP.Files = window.OCP.Files ?? {}