Skip to content

Commit 5e2f963

Browse files
Fix keys used to resolve terminal settings (#132868)
Co-authored-by: IllusionMH <[email protected]>
1 parent 20e7648 commit 5e2f963

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

src/vs/workbench/contrib/terminal/browser/terminalProfileResolverService.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -209,17 +209,17 @@ export abstract class BaseTerminalProfileResolverService implements ITerminalPro
209209
}
210210

211211
private _getUnresolvedRealDefaultProfile(os: OperatingSystem): ITerminalProfile | undefined {
212-
const defaultProfileName = this._configurationService.getValue(`${TerminalSettingPrefix.DefaultProfile}.${this._getOsKey(os)}`);
212+
const defaultProfileName = this._configurationService.getValue(`${TerminalSettingPrefix.DefaultProfile}${this._getOsKey(os)}`);
213213
if (defaultProfileName && typeof defaultProfileName === 'string') {
214214
return this._terminalService.availableProfiles.find(e => e.profileName === defaultProfileName);
215215
}
216216
return undefined;
217217
}
218218

219219
private async _getUnresolvedShellSettingDefaultProfile(options: IShellLaunchConfigResolveOptions): Promise<ITerminalProfile | undefined> {
220-
let executable = this._configurationService.getValue<string>(`${TerminalSettingPrefix.Shell}.${this._getOsKey(options.os)}`);
220+
let executable = this._configurationService.getValue<string>(`${TerminalSettingPrefix.Shell}${this._getOsKey(options.os)}`);
221221
if (!this._isValidShell(executable)) {
222-
const shellArgs = this._configurationService.inspect(`${TerminalSettingPrefix.ShellArgs}.${this._getOsKey(options.os)}`);
222+
const shellArgs = this._configurationService.inspect(`${TerminalSettingPrefix.ShellArgs}${this._getOsKey(options.os)}`);
223223
// && !this.getSafeConfigValue('shellArgs', options.os, false)) {
224224
if (!shellArgs.userValue && !shellArgs.workspaceValue) {
225225
return undefined;
@@ -231,7 +231,7 @@ export abstract class BaseTerminalProfileResolverService implements ITerminalPro
231231
}
232232

233233
let args: string | string[] | undefined;
234-
const shellArgsSetting = this._configurationService.getValue(`${TerminalSettingPrefix.ShellArgs}.${this._getOsKey(options.os)}`);
234+
const shellArgsSetting = this._configurationService.getValue(`${TerminalSettingPrefix.ShellArgs}${this._getOsKey(options.os)}`);
235235
if (this._isValidShellArgs(shellArgsSetting, options.os)) {
236236
args = shellArgsSetting;
237237
}

src/vs/workbench/contrib/terminal/common/terminalEnvironment.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -326,7 +326,7 @@ export function getDefaultShellArgs(
326326
}
327327

328328
const platformKey = platformOverride === Platform.Windows ? 'windows' : platformOverride === Platform.Mac ? 'osx' : 'linux';
329-
let args = fetchSetting(<TerminalShellArgsSetting>`${TerminalSettingPrefix.ShellArgs}.${platformKey}`);
329+
let args = fetchSetting(<TerminalShellArgsSetting>`${TerminalSettingPrefix.ShellArgs}${platformKey}`);
330330
if (!args) {
331331
return [];
332332
}
@@ -339,7 +339,7 @@ export function getDefaultShellArgs(
339339
try {
340340
resolvedArgs.push(variableResolver(arg));
341341
} catch (e) {
342-
logService.error(`Could not resolve ${TerminalSettingPrefix.ShellArgs}.${platformKey}`, e);
342+
logService.error(`Could not resolve ${TerminalSettingPrefix.ShellArgs}${platformKey}`, e);
343343
resolvedArgs.push(arg);
344344
}
345345
}

0 commit comments

Comments
 (0)