ckaczor/azuredatastudio
1
0
Fork 0
mirror of https://github.com/ckaczor/azuredatastudio.git synced 2026-01-19 09:35:36 -05:00
Code Issues Packages Projects Releases Wiki Activity

Fix contentManager undefined when builtin manager used (#3710)

Browse Source 
* Fix for contentManager undefined for builtin manager

* Clean up code some more
This commit is contained in:
Chris LaFreniere
2019-01-11 10:36:36 -08:00
committed by GitHub
parent 71c1ed6c49
commit d0acb51fd7
1 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/sql/parts/notebook/models/notebookModel.ts
View File

@@ -76,7 +76,11 @@ export class NotebookModel extends Disposable implements INotebookModel {
}
public get notebookManagers(): INotebookManager[] {
return this.notebookOptions.notebookManagers.filter(manager => manager.providerId !== DEFAULT_NOTEBOOK_PROVIDER);
let notebookManagers = this.notebookOptions.notebookManagers.filter(manager => manager.providerId !== DEFAULT_NOTEBOOK_PROVIDER);
if (!notebookManagers.length) {
return this.notebookOptions.notebookManagers;
}
return notebookManagers;
}
public get notebookManager(): INotebookManager {
@@ -424,6 +428,9 @@ export class NotebookModel extends Disposable implements INotebookModel {
await this.loadActiveContexts(e);
});
});
if (!this.notebookManager) {
return;
}
try {
let sessionManager = this.notebookManager.sessionManager;
if (sessionManager) {