ckaczor/azuredatastudio
1
0
Fork 0
mirror of https://github.com/ckaczor/azuredatastudio.git synced 2026-02-02 01:25:39 -05:00
Code Issues Packages Projects Releases Wiki Activity

'Confirm save' implementation while closing untitled/existing notebooks (#4349)

Browse Source 
* #4326: 'Confirm save' while closing both notebook

* Adding comment
This commit is contained in:
Raj
2019-03-08 13:28:15 -08:00
committed by GitHub
parent c3900f6984
commit 4d6271c161
1 changed files with 32 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
src/sql/parts/notebook/notebookInput.ts
View File

@@ -8,7 +8,7 @@
import * as nls from 'vs/nls';
import { TPromise } from 'vs/base/common/winjs.base';
import { IEditorModel } from 'vs/platform/editor/common/editor';
import { EditorInput, EditorModel } from 'vs/workbench/common/editor';
import { EditorInput, EditorModel, ConfirmResult } from 'vs/workbench/common/editor';
import { Emitter, Event } from 'vs/base/common/event';
import { URI } from 'vs/base/common/uri';
import * as resources from 'vs/base/common/resources';
@@ -25,6 +25,7 @@ import { UntitledEditorModel } from 'vs/workbench/common/editor/untitledEditorMo
import { Schemas } from 'vs/base/common/network';
import { IUntitledEditorService } from 'vs/workbench/services/untitled/common/untitledEditorService';
import { notebookModeId } from 'sql/common/constants';
import { ITextFileService, ISaveOptions } from 'vs/workbench/services/textfile/common/textfiles';
import { LocalContentManager } from 'sql/workbench/services/notebook/node/localContentManager';
export type ModeViewSaveHandler = (handle: number) => Thenable<boolean>;
@@ -35,7 +36,8 @@ export class NotebookEditorModel extends EditorModel {
private readonly _onDidChangeDirty: Emitter<void> = this._register(new Emitter<void>());
constructor(public readonly notebookUri: URI,
private textEditorModel: TextFileEditorModel | UntitledEditorModel,
@INotebookService private notebookService: INotebookService
@INotebookService private notebookService: INotebookService,
@ITextFileService private textFileService: ITextFileService
) {
super();
this._register(this.notebookService.onNotebookEditorAdd(notebook => {
@@ -73,6 +75,24 @@ export class NotebookEditorModel extends EditorModel {
this._onDidChangeDirty.fire();
}
public confirmSave(): TPromise<ConfirmResult> {
return this.textFileService.confirmSave([this.notebookUri]);
}
/**
* UntitledEditor uses TextFileService to save data from UntitledEditorInput
* Titled editor uses TextFileEditorModel to save existing notebook
*/
save(options: ISaveOptions): TPromise<boolean> {
if (this.textEditorModel instanceof TextFileEditorModel) {
this.textEditorModel.save(options);
return TPromise.as(true);
}
else {
return this.textFileService.save(this.notebookUri, options);
}
}
public updateModel(): void {
let notebookModel = this.getNotebookModel();
if (notebookModel && this.textEditorModel && this.textEditorModel.textEditorModel) {
@@ -134,6 +154,10 @@ export class NotebookInput extends EditorInput {
this.assignProviders();
}
public confirmSave(): TPromise<ConfirmResult> {
return this._model.confirmSave();
}
public get notebookUri(): URI {
return this.resource;
}
@@ -188,6 +212,11 @@ export class NotebookInput extends EditorInput {
this._providers = value;
}
public save(): TPromise<boolean> {
let options: ISaveOptions = { force: false };
return this._model.save(options);
}
public set standardKernels(value: IStandardKernelWithProvider[]) {
value.forEach(kernel => {
this._standardKernels.push({
@@ -328,4 +357,5 @@ class NotebookEditorContentManager implements IContentManager {
let contents = await contentManager.loadFromContentString(notebookEditorModel.contentString);
return contents;
}
}