X-Git-Url: https://git.arvados.org/arvados-workbench2.git/blobdiff_plain/7622261116e685cad4af9cf6a4d1dd9c58cd1605..1c7242e61e9d71c7a37483ec0583dd0256f8ee7e:/src/store/navigation/navigation-action.ts diff --git a/src/store/navigation/navigation-action.ts b/src/store/navigation/navigation-action.ts index 034cdacc..f8687ed7 100644 --- a/src/store/navigation/navigation-action.ts +++ b/src/store/navigation/navigation-action.ts @@ -3,24 +3,22 @@ // SPDX-License-Identifier: AGPL-3.0 import { Dispatch } from "redux"; -import projectActions, { getProjectList } from "../project/project-action"; +import { projectActions, getProjectList } from "../project/project-action"; import { push } from "react-router-redux"; import { TreeItemStatus } from "../../components/tree/tree"; import { findTreeItem } from "../project/project-reducer"; -import { Resource, ResourceKind as R } from "../../models/resource"; -import sidePanelActions from "../side-panel/side-panel-action"; -import dataExplorerActions from "../data-explorer/data-explorer-action"; +import { dataExplorerActions } from "../data-explorer/data-explorer-action"; import { PROJECT_PANEL_ID } from "../../views/project-panel/project-panel"; import { RootState } from "../store"; -import { sidePanelData } from "../side-panel/side-panel-reducer"; -import { loadDetails } from "../details-panel/details-panel-action"; -import { ResourceKind } from "../../models/kinds"; +import { Resource, ResourceKind } from "../../models/resource"; +import { getCollectionUrl } from "../../models/collection"; +import { getProjectUrl } from "../../models/project"; -export const getResourceUrl = (resource: Resource): string => { +export const getResourceUrl = (resource: T): string => { switch (resource.kind) { - case R.PROJECT: return `/projects/${resource.uuid}`; - case R.COLLECTION: return `/collections/${resource.uuid}`; - default: return ""; + case ResourceKind.PROJECT: return getProjectUrl(resource.uuid); + case ResourceKind.COLLECTION: return getCollectionUrl(resource.uuid); + default: return resource.href; } }; @@ -32,24 +30,12 @@ export enum ItemMode { export const setProjectItem = (itemId: string, itemMode: ItemMode) => (dispatch: Dispatch, getState: () => RootState) => { - const { projects, router, sidePanel } = getState(); + const { projects, router } = getState(); const treeItem = findTreeItem(projects.items, itemId); if (treeItem) { - // TODO: Get correct resource kind - dispatch(loadDetails(treeItem.data.uuid, ResourceKind.Project)); - dispatch(sidePanelActions.RESET_SIDE_PANEL_ACTIVITY()); - const projectsItem = sidePanelData[0]; - if(sidePanel.some(item => item.id === projectsItem.id && !item.open)){ - dispatch(sidePanelActions.TOGGLE_SIDE_PANEL_ITEM_OPEN(projectsItem.id)); - } - - if (itemMode === ItemMode.OPEN || itemMode === ItemMode.BOTH) { - dispatch(projectActions.TOGGLE_PROJECT_TREE_ITEM_OPEN(treeItem.data.uuid)); - } - - const resourceUrl = getResourceUrl({ ...treeItem.data }); + const resourceUrl = getResourceUrl(treeItem.data); if (itemMode === ItemMode.ACTIVE || itemMode === ItemMode.BOTH) { if (router.location && !router.location.pathname.includes(resourceUrl)) { @@ -58,15 +44,19 @@ export const setProjectItem = (itemId: string, itemMode: ItemMode) => dispatch(projectActions.TOGGLE_PROJECT_TREE_ITEM_ACTIVE(treeItem.data.uuid)); } - const promise = treeItem.status === TreeItemStatus.Loaded + const promise = treeItem.status === TreeItemStatus.LOADED ? Promise.resolve() : dispatch(getProjectList(itemId)); promise .then(() => dispatch(() => { + if (itemMode === ItemMode.OPEN || itemMode === ItemMode.BOTH) { + dispatch(projectActions.TOGGLE_PROJECT_TREE_ITEM_OPEN(treeItem.data.uuid)); + } dispatch(dataExplorerActions.RESET_PAGINATION({id: PROJECT_PANEL_ID})); dispatch(dataExplorerActions.REQUEST_ITEMS({id: PROJECT_PANEL_ID})); })); } }; +