Merge branch '17018-readonly-file-actions-fix'
[arvados-workbench2.git] / src / views-components / context-menu / actions / download-collection-file-action.tsx
index aadc1d11a14c26c4aa89fccdf52fdf1d39d7f001..e1986d3cd338454551cc46b9563405d097257ab2 100644 (file)
@@ -6,19 +6,20 @@ import { connect } from "react-redux";
 import { RootState } from "../../../store/store";
 import { DownloadAction } from "./download-action";
 import { getNodeValue } from "../../../models/tree";
-import { CollectionFileType } from "../../../models/collection-file";
 import { ContextMenuKind } from '../context-menu';
 import { filterCollectionFilesBySelection } from "~/store/collection-panel/collection-panel-files/collection-panel-files-state";
+import { sanitizeToken } from "./helpers";
 
 const mapStateToProps = (state: RootState) => {
     const { resource } = state.contextMenu;
     const currentCollectionUuid = state.collectionPanel.item ? state.collectionPanel.item.uuid : '';
-    if (resource && resource.menuKind === ContextMenuKind.COLLECTION_FILES_ITEM) {
+    if (resource && (
+        resource.menuKind === ContextMenuKind.COLLECTION_FILES_ITEM ||
+        resource.menuKind === ContextMenuKind.READONLY_COLLECTION_FILES_ITEM)) {
         const file = getNodeValue(resource.uuid)(state.collectionPanelFiles);
         if (file) {
             return {
-                href: file.url,
-                download: file.type === CollectionFileType.DIRECTORY ? undefined : file.name,
+                href: sanitizeToken(file.url, true),
                 kind: 'file',
                 currentCollectionUuid
             };
@@ -27,7 +28,6 @@ const mapStateToProps = (state: RootState) => {
         const files = filterCollectionFilesBySelection(state.collectionPanelFiles, true);
         return {
             href: files.map(file => file.url),
-            download: files.map(file => file.name),
             kind: 'files',
             currentCollectionUuid
         };