Fixed showOpenDialog fallback (#10573)

The showOpenDialog and other file dialog extension APIs now correctly fall back to the workspace root when a defaultUri is not provided. Signed-off-by: Matthew Gordon <[email protected]>
eclipse-theia · Dec 22, 2021 · 17b419f · 17b419f
1 parent fad4231
commit 17b419f
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/packages/plugin-ext/src/main/browser/dialogs-main.ts b/packages/plugin-ext/src/main/browser/dialogs-main.ts
@@ -52,14 +52,14 @@ export class DialogsMainImpl implements DialogsMain {
             } catch {
                 rootStat = undefined;
             }
-        }
 
-        // Try to use as root the parent folder of existing file URI/non existing URI
-        if (rootStat && !rootStat.isDirectory || !rootStat) {
-            try {
-                rootStat = await this.fileService.resolve(new URI(defaultUri).parent);
-            } catch {
-                rootStat = undefined;
+            // Try to use as root the parent folder of existing file URI/non existing URI
+            if (rootStat && !rootStat.isDirectory || !rootStat) {
+                try {
+                    rootStat = await this.fileService.resolve(new URI(defaultUri).parent);
+                } catch {
+                    rootStat = undefined;
+                }
             }
         }