Merge pull request #1135 from nextcloud-libraries/fix/fileListAction-api

__tests__/fileListAction.spec.ts

@@ -156,7 +156,7 @@ describe('FileListAction creation', () => {
 		expect(testAction.displayName({} as unknown as View)).toBe('Test')
 		expect(testAction.iconSvgInline({} as unknown as View)).toBe('<svg></svg>')
 		expect(testAction.order).toBe(0)
-		expect(testAction.enabled?.({} as unknown as View, [], { folder: {} as Folder })).toBe(true)
-		await expect(testAction.exec({} as unknown as View, [], { folder: {} as Folder })).resolves.toBe(undefined)
+		expect(testAction.enabled?.({} as unknown as View, [], {} as Folder)).toBe(true)
+		await expect(testAction.exec({} as unknown as View, [], {} as Folder)).resolves.toBe(undefined)
 	})
 })

lib/fileListAction.ts

@@ -3,15 +3,11 @@
  * SPDX-License-Identifier: AGPL-3.0-or-later
  */
 
-import { Node } from './files/node.ts'
-import { Folder } from './files/folder.ts'
-import { View } from './navigation/view.ts'
+import type { Folder } from './files/folder.ts'
+import type { Node } from './files/node.ts'
+import type { View } from './navigation/view.ts'
 import logger from './utils/logger.ts'
 
-interface ActionContext {
-	folder: Folder,
-}
-
 interface FileListActionData {
 	/** Unique ID */
 	id: string
@@ -28,20 +24,20 @@ interface FileListActionData {
 	/**
 	 * Returns true if this action shoud be shown
 	 *
+	 * @param view The current view
 	 * @param nodes The nodes in the current directory
-	 * @param context The context
-	 * @param context.folder The current folder
+	 * @param folder The current folder
 	 */
-	enabled?: (view: View, nodes: Node[], context: ActionContext) => boolean
+	enabled?: (view: View, nodes: Node[], folder: Folder) => boolean
 
 	/**
 	 * Function to execute
 	 *
+	 * @param view The current view
 	 * @param nodes The nodes in the current directory
-	 * @param context The context
-	 * @param context.folder The current folder
+	 * @param folder The current folder
 	 */
-	exec: (view: View, nodes: Node[], context: ActionContext) => Promise<void>
+	exec: (view: View, nodes: Node[], folder: Folder) => Promise<void>
 }
 
 export class FileListAction {