ckaczor/azuredatastudio
1
0
Fork 0
mirror of https://github.com/ckaczor/azuredatastudio.git synced 2026-01-13 17:22:15 -05:00
Code Issues Packages Projects Releases Wiki Activity

Fix crash when opening SQL diff input (#15183)

Browse Source 
* Fix crash when opening SQL diff input

* fix compile

* Fix tests
This commit is contained in:
Charles Gagnon
2021-04-19 13:49:02 -07:00
committed by GitHub
parent 1914546553
commit 2f04adb510
1 changed files with 21 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
src/sql/workbench/contrib/editorReplacement/common/editorReplacerContribution.ts
View File

@@ -19,6 +19,7 @@ import { UntitledTextEditorInput } from 'vs/workbench/services/untitled/common/u
import { isThenable } from 'vs/base/common/async';
import { withNullAsUndefined } from 'vs/base/common/types';
import { DiffEditorInput } from 'vs/workbench/common/editor/diffEditorInput';
import { mixin } from 'vs/base/common/objects';
const languageAssociationRegistry = Registry.as<ILanguageAssociationRegistry>(LanguageAssociationExtensions.LanguageAssociations);
@@ -69,7 +70,16 @@ export class EditorReplacementContribution implements IWorkbenchContribution {
editor.setMode(defaultInputCreator[0]);
const newInput = defaultInputCreator[1].convertInput(editor);
if (newInput) {
return { override: isThenable(newInput) ? newInput.then(input => this.editorService.openEditor(input ?? editor, options, group)) : this.editorService.openEditor(newInput, options, group) };
return {
// If the new input is a thenable which resolves to undefined (no input to convert)
// then don't allow further overriding since otherwise we could get in an infinite loop
// (openEditor calls onEditorOpening handler and we keep repeating this process). This
// is replicating the behavior for the non-Thenable returns where we just let the openEditor
// continue on
override: isThenable(newInput) ?
newInput.then(input => this.editorService.openEditor(input ?? editor, mixin(options, { override: input ? options?.override : false }), group)) :
this.editorService.openEditor(newInput, options, group)
};
}
}
} else {
@@ -77,7 +87,16 @@ export class EditorReplacementContribution implements IWorkbenchContribution {
if (inputCreator) {
const newInput = inputCreator.convertInput(editor);
if (newInput) {
return { override: isThenable(newInput) ? newInput.then(input => this.editorService.openEditor(input ?? editor, options, group)) : this.editorService.openEditor(newInput, options, group) };
return {
// If the new input is a thenable which resolves to undefined (no input to convert)
// then don't allow further overriding since otherwise we could get in an infinite loop
// (openEditor calls onEditorOpening handler and we keep repeating this process). This
// is replicating the behavior for the non-Thenable returns where we just let the openEditor
// continue on
override: isThenable(newInput) ?
newInput.then(input => this.editorService.openEditor(input ?? editor, mixin(options, { override: input ? options?.override : false }), group))
: this.editorService.openEditor(newInput, options, group)
};
}
}
}