From b238c5cec390860b3265238e07b41d62349f70ce Mon Sep 17 00:00:00 2001 From: Lucas Di Pentima Date: Fri, 20 Dec 2019 20:14:58 -0300 Subject: [PATCH] 15672: Uses process context menu on subprocess panel. Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima --- .../subprocess-panel-root.tsx | 7 ++++--- .../subprocess-panel/subprocess-panel.tsx | 20 ++++++------------- 2 files changed, 10 insertions(+), 17 deletions(-) diff --git a/src/views/subprocess-panel/subprocess-panel-root.tsx b/src/views/subprocess-panel/subprocess-panel-root.tsx index 2d37ccb3..a8ce6075 100644 --- a/src/views/subprocess-panel/subprocess-panel-root.tsx +++ b/src/views/subprocess-panel/subprocess-panel-root.tsx @@ -16,6 +16,7 @@ import { SUBPROCESS_PANEL_ID } from '~/store/subprocess-panel/subprocess-panel-a import { DataTableDefaultView } from '~/components/data-table-default-view/data-table-default-view'; import { createTree } from '~/models/tree'; import { getInitialProcessStatusFilters } from '~/store/resource-type-filters/resource-type-filters'; +import { ResourcesState } from '~/store/resources/resources'; export enum SubprocessPanelColumnNames { NAME = "Name", @@ -63,12 +64,12 @@ export const subprocessPanelColumns: DataColumns = [ ]; export interface SubprocessPanelDataProps { - isAdmin: boolean; + resources: ResourcesState; } export interface SubprocessPanelActionProps { onItemClick: (item: string) => void; - onContextMenu: (event: React.MouseEvent, item: string, isAdmin: boolean) => void; + onContextMenu: (event: React.MouseEvent, item: string, resources: ResourcesState) => void; onItemDoubleClick: (item: string) => void; } @@ -84,7 +85,7 @@ export const SubprocessPanelRoot = (props: SubprocessPanelProps) => { id={SUBPROCESS_PANEL_ID} onRowClick={props.onItemClick} onRowDoubleClick={props.onItemDoubleClick} - onContextMenu={(event, item) => props.onContextMenu(event, item, props.isAdmin)} + onContextMenu={(event, item) => props.onContextMenu(event, item, props.resources)} contextMenuColumn={true} dataTableDefaultView={ ({ - onContextMenu: (event, resourceUuid, isAdmin) => { - const menuKind = resourceKindToContextMenuKind(resourceUuid, isAdmin); - const resource = getProcess(resourceUuid); - if (menuKind && resource) { - dispatch(openContextMenu(event, { - name: resource.name, - uuid: resourceUuid, - ownerUuid: '', - kind: ResourceKind.PROCESS, - menuKind - })); + onContextMenu: (event, resourceUuid, resources) => { + const process = getProcess(resourceUuid)(resources); + if (process) { + dispatch(openProcessContextMenu(event, process)); } }, onItemClick: (uuid: string) => { @@ -35,7 +27,7 @@ const mapDispatchToProps = (dispatch: Dispatch): SubprocessPanelActionProps => ( }); const mapStateToProps = (state: RootState): SubprocessPanelDataProps => ({ - isAdmin: state.auth.user ? state.auth.user.isAdmin : false + resources: state.resources }); export const SubprocessPanel = connect(mapStateToProps, mapDispatchToProps)(SubprocessPanelRoot); \ No newline at end of file -- 2.30.2