From aae013d4987dc3c986d5b1e290a4ffa7513871d2 Mon Sep 17 00:00:00 2001 From: Anthony Dresser Date: Wed, 15 Jul 2020 15:06:46 -0700 Subject: [PATCH] add option to open excel file externally (#11324) --- .../services/query/common/resultSerializer.ts | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/src/sql/workbench/services/query/common/resultSerializer.ts b/src/sql/workbench/services/query/common/resultSerializer.ts index b870d8f2a1..927102470e 100644 --- a/src/sql/workbench/services/query/common/resultSerializer.ts +++ b/src/sql/workbench/services/query/common/resultSerializer.ts @@ -17,6 +17,7 @@ import { IEditorService } from 'vs/workbench/services/editor/common/editorServic import { getRootPath, resolveCurrentDirectory } from 'sql/platform/common/pathUtilities'; import { IConfigurationService } from 'vs/platform/configuration/common/configuration'; import { IFileDialogService, FileFilter } from 'vs/platform/dialogs/common/dialogs'; +import { IOpenerService } from 'vs/platform/opener/common/opener'; import { IQueryEditorConfiguration } from 'sql/platform/query/common/query'; let prevSavePath: URI; @@ -54,7 +55,8 @@ export class ResultSerializer { @IEditorService private _editorService: IEditorService, @IWorkspaceContextService private _contextService: IWorkspaceContextService, @IFileDialogService private readonly fileDialogService: IFileDialogService, - @INotificationService private _notificationService: INotificationService + @INotificationService private _notificationService: INotificationService, + @IOpenerService private readonly openerService: IOpenerService ) { } /** @@ -321,6 +323,17 @@ export class ResultSerializer { message: error }); }); + } else { + this._notificationService.prompt( + Severity.Info, + nls.localize('msgSaveSucceeded', "Successfully saved results to {0}", filePath.path), + [{ + label: nls.localize('openFile', "Open file"), + run: () => { + this.openerService.open(filePath, { openExternal: true }); + } + }] + ); } } }