Merge pull request #1392 from nextcloud-libraries/fix/missing-view-actions-contents

Author: skjnldsv

__tests__/actions/fileListAction.spec.ts

@@ -40,8 +40,8 @@ describe('FileListActions init', () => {
 		const testAction = mockAction('test')
 
 		expect(testAction.id).toBe('test')
-		expect(testAction.displayName({ view, folder })).toBe('Test')
-		expect(testAction.iconSvgInline!({ view, folder })).toBe('<svg></svg>')
+		expect(testAction.displayName({ view, folder, contents: [] })).toBe('Test')
+		expect(testAction.iconSvgInline!({ view, folder, contents: [] })).toBe('<svg></svg>')
 
 		registerFileListAction(testAction)
 		expect(actions).toHaveLength(1)
@@ -158,10 +158,10 @@ describe('FileListAction creation', () => {
 		})
 
 		expect(testAction.id).toBe('test')
-		expect(testAction.displayName({ view, folder })).toBe('Test')
-		expect(testAction.iconSvgInline!({ view, folder })).toBe('<svg></svg>')
+		expect(testAction.displayName({ view, folder, contents: [] })).toBe('Test')
+		expect(testAction.iconSvgInline!({ view, folder, contents: [] })).toBe('<svg></svg>')
 		expect(testAction.order).toBe(0)
-		expect(testAction.enabled?.({ view, folder })).toBe(true)
-		await expect(testAction.exec({ view, folder, nodes: [], content: [] })).resolves.toBe(undefined)
+		expect(testAction.enabled?.({ view, folder, contents: [] })).toBe(true)
+		await expect(testAction.exec({ view, folder, contents: [] })).resolves.toBe(undefined)
 	})
 })

lib/actions/fileListAction.ts

@@ -2,7 +2,7 @@
  * SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
  * SPDX-License-Identifier: AGPL-3.0-or-later
  */
-import type { ActionContext, ViewActionContext } from '../types.ts'
+import type { ViewActionContext } from '../types.ts'
 
 import logger from '../utils/logger.ts'
 
@@ -30,7 +30,7 @@ interface FileListActionData {
 	 * false otherwise and null if the action is silent/undefined.
 	 * @throws Error if the action failed
 	 */
-	exec: (context: ActionContext) => Promise<boolean|null>,
+	exec: (context: ViewActionContext) => Promise<boolean|null>,
 }
 
 export class FileListAction {

lib/types.ts

@@ -23,4 +23,5 @@ export type ActionContext = {
 export type ViewActionContext = {
 	view: View,
 	folder: Folder,
+	contents: Node[],
 }