[ACA-1751] fix move/delete commands for shared preview (#598)

This commit is contained in:
Denys Vuika
2018-09-04 09:13:32 +01:00
committed by GitHub
parent e0d90b093c
commit 64319a7090
3 changed files with 20 additions and 3 deletions

View File

@@ -113,7 +113,8 @@ export class CoreExtensionsModule {
'app.navigation.isRecentFiles': nav.isRecentFiles, 'app.navigation.isRecentFiles': nav.isRecentFiles,
'app.navigation.isNotRecentFiles': nav.isNotRecentFiles, 'app.navigation.isNotRecentFiles': nav.isNotRecentFiles,
'app.navigation.isSearchResults': nav.isSearchResults, 'app.navigation.isSearchResults': nav.isSearchResults,
'app.navigation.isNotSearchResults': nav.isNotSearchResults 'app.navigation.isNotSearchResults': nav.isNotSearchResults,
'app.navigation.isPreview': nav.isPreview
}); });
} }
} }

View File

@@ -32,7 +32,8 @@ import {
isLibraries, isLibraries,
isTrashcan, isTrashcan,
isSharedFiles, isSharedFiles,
isNotSearchResults isNotSearchResults,
isPreview
} from './navigation.evaluators'; } from './navigation.evaluators';
export function canAddFavorite( export function canAddFavorite(
@@ -94,12 +95,19 @@ export function canDeleteSelection(
return true; return true;
} }
if (isPreview(context, ...args)) {
return context.permissions.check(context.selection.nodes, [
'delete'
]);
}
// workaround for Shared Files // workaround for Shared Files
if (isSharedFiles(context, ...args)) { if (isSharedFiles(context, ...args)) {
return context.permissions.check( return context.permissions.check(
context.selection.nodes, context.selection.nodes,
['delete'], ['delete'],
{ target: 'allowableOperationsOnTarget' }); { target: 'allowableOperationsOnTarget' }
);
} }
return context.permissions.check(context.selection.nodes, ['delete']); return context.permissions.check(context.selection.nodes, ['delete']);

View File

@@ -114,3 +114,11 @@ export function isNotSearchResults(
): boolean { ): boolean {
return !isSearchResults(context, ...args); return !isSearchResults(context, ...args);
} }
export function isPreview(
context: RuleContext,
...args: RuleParameter[]
): boolean {
const { url } = context.navigation;
return url && url.includes('/preview/');
}