fix(files): Make sure to add the `fileid` on favorite folders navigation entries

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

3 files changed, 40 insertions, 28 deletions

diff --git a/apps/files/src/views/Navigation.vue b/apps/files/src/views/Navigation.vue
index 5ec650569b2..38f7a980a91 100644
--- a/apps/files/src/views/Navigation.vue
+++ b/apps/files/src/views/Navigation.vue
@@ -25,7 +25,7 @@
 		<template #list>
 			<NcAppNavigationItem v-for="view in parentViews"
 				:key="view.id"
-				allow-collapse
+				:allow-collapse="true"
 				:data-cy-files-navigation-item="view.id"
 				:exact="useExactRouteMatching(view)"
 				:icon="view.iconClass"
@@ -179,7 +179,7 @@ export default {
 		 * Like for the 'files' view this does not work because of optional 'fileid' param so /files and /files/1234 are both in the 'files' view
 		 * @param view The view to check
 		 */
-		useExactRouteMatching(view: View) {
+		useExactRouteMatching(view: View): boolean {
 			return this.childViews[view.id]?.length > 0
 		},
 
@@ -221,8 +221,8 @@ export default {
 		 */
 		generateToNavigation(view: View) {
 			if (view.params) {
-				const { dir, fileid } = view.params
-				return { name: 'filelist', params: view.params, query: { dir, fileid } }
+				const { dir } = view.params
+				return { name: 'filelist', params: view.params, query: { dir } }
 			}
 			return { name: 'filelist', params: { view: view.id } }
 		},
diff --git a/apps/files/src/views/favorites.spec.ts b/apps/files/src/views/favorites.spec.ts
index 8fefb6d23b5..9497a7be1f9 100644
--- a/apps/files/src/views/favorites.spec.ts
+++ b/apps/files/src/views/favorites.spec.ts
@@ -82,9 +82,9 @@ describe('Favorites view definition', () => {
 
 	test('Default with favorites', () => {
 		const favoriteFolders = [
-			'/foo',
-			'/bar',
-			'/foo/bar',
+			{ fileid: 1, path: '/foo' },
+			{ fileid: 2, path: '/bar' },
+			{ fileid: 3, path: '/foo/bar' },
 		]
 		jest.spyOn(initialState, 'loadState').mockReturnValue(favoriteFolders)
 		jest.spyOn(favoritesService, 'getContents').mockReturnValue(Promise.resolve({ folder: {} as Folder, contents: [] }))
@@ -102,11 +102,12 @@ describe('Favorites view definition', () => {
 			const favoriteView = favoriteFoldersViews[index]
 			expect(favoriteView).toBeDefined()
 			expect(favoriteView?.id).toBeDefined()
-			expect(favoriteView?.name).toBe(basename(folder))
+			expect(favoriteView?.name).toBe(basename(folder.path))
 			expect(favoriteView?.icon).toBe('<svg>SvgMock</svg>')
 			expect(favoriteView?.order).toBe(index)
 			expect(favoriteView?.params).toStrictEqual({
-				dir: folder,
+				dir: folder.path,
+				fileid: folder.fileid.toString(),
 				view: 'favorites',
 			})
 			expect(favoriteView?.parent).toBe('favorites')
@@ -157,7 +158,7 @@ describe('Dynamic update of favourite folders', () => {
 	test('Remove a favorite folder remove the entry from the navigation column', async () => {
 		jest.spyOn(eventBus, 'emit')
 		jest.spyOn(eventBus, 'subscribe')
-		jest.spyOn(initialState, 'loadState').mockReturnValue(['/Foo/Bar'])
+		jest.spyOn(initialState, 'loadState').mockReturnValue([{ fileid: 42, path: '/Foo/Bar' }])
 		jest.spyOn(favoritesService, 'getContents').mockReturnValue(Promise.resolve({ folder: {} as Folder, contents: [] }))
 
 		registerFavoritesView()
diff --git a/apps/files/src/views/favorites.ts b/apps/files/src/views/favorites.ts
index 599aa191357..67c4fd58a86 100644
--- a/apps/files/src/views/favorites.ts
+++ b/apps/files/src/views/favorites.ts
@@ -19,11 +19,13 @@
  * along with this program. If not, see <http://www.gnu.org/licenses/>.
  *
  */
-import { basename } from 'path'
-import { getLanguage, translate as t } from '@nextcloud/l10n'
-import { loadState } from '@nextcloud/initial-state'
-import { Node, FileType, View, getNavigation } from '@nextcloud/files'
+import type { Folder, Node } from '@nextcloud/files'
+
 import { subscribe } from '@nextcloud/event-bus'
+import { FileType, View, getNavigation } from '@nextcloud/files'
+import { loadState } from '@nextcloud/initial-state'
+import { getLanguage, translate as t } from '@nextcloud/l10n'
+import { basename } from 'path'
 import FolderSvg from '@mdi/svg/svg/folder.svg?raw'
 import StarSvg from '@mdi/svg/svg/star.svg?raw'
 
@@ -31,15 +33,22 @@ import { getContents } from '../services/Favorites'
 import { hashCode } from '../utils/hashUtils'
 import logger from '../logger'
 
-export const generateFolderView = function(folder: string, index = 0): View {
+// The return type of the initial state
+interface IFavoriteFolder {
+	fileid: number
+	path: string
+}
+
+export const generateFavoriteFolderView = function(folder: IFavoriteFolder, index = 0): View {
 	return new View({
-		id: generateIdFromPath(folder),
-		name: basename(folder),
+		id: generateIdFromPath(folder.path),
+		name: basename(folder.path),
 
 		icon: FolderSvg,
 		order: index,
 		params: {
-			dir: folder,
+			dir: folder.path,
+			fileid: folder.fileid.toString(),
 			view: 'favorites',
 		},
 
@@ -57,8 +66,9 @@ export const generateIdFromPath = function(path: string): string {
 
 export default () => {
 	// Load state in function for mock testing purposes
-	const favoriteFolders = loadState<string[]>('files', 'favoriteFolders', [])
-	const favoriteFoldersViews = favoriteFolders.map((folder, index) => generateFolderView(folder, index)) as View[]
+	const favoriteFolders = loadState<IFavoriteFolder[]>('files', 'favoriteFolders', [])
+	const favoriteFoldersViews = favoriteFolders.map((folder, index) => generateFavoriteFolderView(folder, index)) as View[]
+	logger.debug('Generating favorites view', { favoriteFolders })
 
 	const Navigation = getNavigation()
 	Navigation.register(new View({
@@ -93,7 +103,7 @@ export default () => {
 			return
 		}
 
-		addPathToFavorites(node.path)
+		addToFavorites(node as Folder)
 	})
 
 	/**
@@ -118,9 +128,9 @@ export default () => {
 	 * update the order property of the existing views
 	 */
 	const updateAndSortViews = function() {
-		favoriteFolders.sort((a, b) => a.localeCompare(b, getLanguage(), { ignorePunctuation: true }))
+		favoriteFolders.sort((a, b) => a.path.localeCompare(b.path, getLanguage(), { ignorePunctuation: true }))
 		favoriteFolders.forEach((folder, index) => {
-			const view = favoriteFoldersViews.find(view => view.id === generateIdFromPath(folder))
+			const view = favoriteFoldersViews.find((view) => view.id === generateIdFromPath(folder.path))
 			if (view) {
 				view.order = index
 			}
@@ -128,16 +138,17 @@ export default () => {
 	}
 
 	// Add a folder to the favorites paths array and update the views
-	const addPathToFavorites = function(path: string) {
-		const view = generateFolderView(path)
+	const addToFavorites = function(node: Folder) {
+		const newFavoriteFolder: IFavoriteFolder = { path: node.path, fileid: node.fileid! }
+		const view = generateFavoriteFolderView(newFavoriteFolder)
 
 		// Skip if already exists
-		if (favoriteFolders.find(folder => folder === path)) {
+		if (favoriteFolders.find((folder) => folder.path === node.path)) {
 			return
 		}
 
 		// Update arrays
-		favoriteFolders.push(path)
+		favoriteFolders.push(newFavoriteFolder)
 		favoriteFoldersViews.push(view)
 
 		// Update and sort views
@@ -148,7 +159,7 @@ export default () => {
 	// Remove a folder from the favorites paths array and update the views
 	const removePathFromFavorites = function(path: string) {
 		const id = generateIdFromPath(path)
-		const index = favoriteFolders.findIndex(folder => folder === path)
+		const index = favoriteFolders.findIndex((folder) => folder.path === path)
 
 		// Skip if not exists
 		if (index === -1) {