terminal: fix 'new terminal' handling

The commit fixes a regression with `new terminal` and `new terminal (with profile)` which did not open successfully when no workspace root was present. The code fixes the issue and also refactors the logic to remove unnecessary duplication. Signed-off-by: vince-fugnitto <[email protected]>
eclipse-theia · Mar 17, 2023 · 5439636 · 5439636
1 parent 16be9ac
commit 5439636
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 28 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -9,6 +9,7 @@
 <a name="breaking_changes_1.36.0">[Breaking Changes:](#breaking_changes_1.36.0)</a>
 
 - [plugin] renamed `TreeViewExtImpl#toTreeItem()` to `TreeViewExtImpl#toTreeElement()`
+- [terminal] removed `openTerminalFromProfile` method from `TerminalFrontendContribution` [#12322](https://github.com/eclipse-theia/theia/pull/12322)
 
 ## v1.35.0 - 02/23/2023
 

diff --git a/packages/terminal/src/browser/terminal-frontend-contribution.ts b/packages/terminal/src/browser/terminal-frontend-contribution.ts
@@ -513,7 +513,13 @@ export class TerminalFrontendContribution implements FrontendApplicationContribu
         });
 
         commands.registerCommand(TerminalCommands.PROFILE_NEW, {
-            execute: () => this.openTerminalFromProfile()
+            execute: async () => {
+                const profile = await this.selectTerminalProfile(nls.localize('theia/terminal/selectProfile', 'Select a profile for the new terminal'));
+                if (!profile) {
+                    return;
+                }
+                this.openTerminal(undefined, profile[1]);
+            }
         });
 
         commands.registerCommand(TerminalCommands.PROFILE_DEFAULT, {
@@ -933,40 +939,29 @@ export class TerminalFrontendContribution implements FrontendApplicationContribu
         return ref instanceof TerminalWidget ? ref : undefined;
     }
 
-    protected async openTerminal(options?: ApplicationShell.WidgetOptions): Promise<void> {
-        let profile = this.profileService.defaultProfile;
-
-        if (!profile) {
-            throw new Error('There are not profiles registered');
+    protected async openTerminal(options?: ApplicationShell.WidgetOptions, terminalProfile?: TerminalProfile): Promise<void> {
+        let profile = terminalProfile;
+        if (!terminalProfile) {
+            profile = this.profileService.defaultProfile;
+            if (!profile) {
+                throw new Error('There are not profiles registered');
+            }
         }
+
         if (profile instanceof ShellTerminalProfile) {
-            const cwd = await this.selectTerminalCwd();
-            if (!cwd) {
-                return;
+            if (this.workspaceService.workspace) {
+                const cwd = await this.selectTerminalCwd();
+                if (!cwd) {
+                    return;
+                }
+                profile = profile.modify({ cwd });
             }
-            profile = profile.modify({ cwd });
         }
 
         const termWidget = await profile?.start();
-
-        this.open(termWidget, { widgetOptions: options });
-    }
-
-    protected async openTerminalFromProfile(options?: ApplicationShell.WidgetOptions): Promise<void> {
-        const result = await this.selectTerminalProfile(nls.localize('theia/terminal/selectProfile', 'Select a profile for the new terminal'));
-        if (!result) {
-            return;
+        if (!!termWidget) {
+            this.open(termWidget, { widgetOptions: options });
         }
-        let profile = result[1];
-        if (profile instanceof ShellTerminalProfile) {
-            const cwd = await this.selectTerminalCwd();
-            if (!cwd) {
-                return;
-            }
-            profile = profile.modify({ cwd });
-        }
-        const termWidget = await profile.start();
-        this.open(termWidget, { widgetOptions: options });
     }
 
     protected async chooseDefaultProfile(): Promise<void> {