diff options
Diffstat (limited to 'apps')
-rw-r--r-- | apps/files/src/actions/deleteAction.spec.ts | 29 | ||||
-rw-r--r-- | apps/files/src/actions/deleteAction.ts | 25 |
2 files changed, 16 insertions, 38 deletions
diff --git a/apps/files/src/actions/deleteAction.spec.ts b/apps/files/src/actions/deleteAction.spec.ts index f1aee5bdcba..c568ec59d9d 100644 --- a/apps/files/src/actions/deleteAction.spec.ts +++ b/apps/files/src/actions/deleteAction.spec.ts @@ -50,13 +50,13 @@ describe('Delete action conditions tests', () => { permissions: Permission.ALL, }) - const file2 = new File({ - id: 1, - source: 'https://cloud.domain.com/remote.php/dav/files/admin/foobar.txt', - owner: 'admin', - mime: 'text/plain', - permissions: Permission.ALL, - }) + // const file2 = new File({ + // id: 1, + // source: 'https://cloud.domain.com/remote.php/dav/files/admin/foobar.txt', + // owner: 'admin', + // mime: 'text/plain', + // permissions: Permission.ALL, + // }) test('Default values', () => { expect(action).toBeInstanceOf(FileAction) @@ -71,14 +71,15 @@ describe('Delete action conditions tests', () => { expect(action.displayName([file], trashbinView)).toBe('Delete permanently') }) - test('Shared node values', () => { - jest.spyOn(auth, 'getCurrentUser').mockReturnValue(null) - expect(action.displayName([file2], view)).toBe('Unshare') - }) + // TODO: Fix this test + // test('Shared node values', () => { + // jest.spyOn(auth, 'getCurrentUser').mockReturnValue(null) + // expect(action.displayName([file2], view)).toBe('Unshare') + // }) - test('Shared and owned nodes values', () => { - expect(action.displayName([file, file2], view)).toBe('Delete and unshare') - }) + // test('Shared and owned nodes values', () => { + // expect(action.displayName([file, file2], view)).toBe('Delete and unshare') + // }) }) describe('Delete action enabled tests', () => { diff --git a/apps/files/src/actions/deleteAction.ts b/apps/files/src/actions/deleteAction.ts index 29f1cf86c9b..1bc07aaa6f9 100644 --- a/apps/files/src/actions/deleteAction.ts +++ b/apps/files/src/actions/deleteAction.ts @@ -24,40 +24,17 @@ import { Permission, Node, View, FileAction } from '@nextcloud/files' import { translate as t } from '@nextcloud/l10n' import axios from '@nextcloud/axios' import TrashCanSvg from '@mdi/svg/svg/trash-can.svg?raw' -import CloseSvg from '@mdi/svg/svg/close.svg?raw' import logger from '../logger.js' -import { getCurrentUser } from '@nextcloud/auth' - -const isAllUnshare = (nodes: Node[]) => { - return !nodes.some(node => node.owner === getCurrentUser()?.uid) -} - -const isMixedUnshareAndDelete = (nodes: Node[]) => { - const hasUnshareItems = nodes.some(node => node.owner !== getCurrentUser()?.uid) - const hasDeleteItems = nodes.some(node => node.owner === getCurrentUser()?.uid) - return hasUnshareItems && hasDeleteItems -} export const action = new FileAction({ id: 'delete', displayName(nodes: Node[], view: View) { - if (isMixedUnshareAndDelete(nodes)) { - return t('files', 'Delete and unshare') - } - - if (isAllUnshare(nodes)) { - return t('files', 'Unshare') - } - return view.id === 'trashbin' ? t('files', 'Delete permanently') : t('files', 'Delete') }, - iconSvgInline: (nodes: Node[]) => { - if (isAllUnshare(nodes)) { - return CloseSvg - } + iconSvgInline: () => { return TrashCanSvg }, |