Merge pull request #405 from AllenInstitute/bugfix/ZEN-wrong-path

bugfix/ZEN-wrong-path

Author: BrianWhitneyAI

packages/core/hooks/useOpenWithMenuItems/index.tsx

@@ -222,7 +222,9 @@ export default (fileDetails?: FileDetail, filters?: FileFilter[]): IContextualMe
                 key: `open-with-${name}`,
                 text: name,
                 title: `Open files with ${name}`,
-                disabled: !filters && !fileDetails,
+                disabled:
+                    (!filters && !fileDetails) ||
+                    (app.filePath.toLowerCase().includes("zen") && !fileDetails?.localPath),
                 onClick() {
                     if (filters) {
                         dispatch(interaction.actions.openWith(app, filters));

packages/core/state/interaction/logics.ts

@@ -489,8 +489,13 @@ const openWithLogic = createLogic({
         } else {
             filesToOpen = await fileSelection.fetchAllDetails();
         }
+
         const filePaths = await Promise.all(
-            filesToOpen.map((file) => executionEnvService.formatPathForHost(file.path))
+            // Default to local path for desktop apps
+            filesToOpen.map((file) => {
+                const filePath = file.localPath ?? file.path;
+                return executionEnvService.formatPathForHost(filePath);
+            })
         );
 
         // Open the files in the specified executable