From 8e0050f2af027dbecd73ec57aae382ec54539ec4 Mon Sep 17 00:00:00 2001 From: Kim Santiago <31145923+kisantia@users.noreply.github.com> Date: Thu, 25 Jun 2020 17:18:41 -0700 Subject: [PATCH] Don't ask for db twice if connection was to a db (#11099) --- .../src/controllers/projectController.ts | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/extensions/sql-database-projects/src/controllers/projectController.ts b/extensions/sql-database-projects/src/controllers/projectController.ts index 17cac978a3..b0e73c507d 100644 --- a/extensions/sql-database-projects/src/controllers/projectController.ts +++ b/extensions/sql-database-projects/src/controllers/projectController.ts @@ -624,11 +624,18 @@ export class ProjectsController { } const connectionId = connection.connectionId; - const databaseList = await this.apiWrapper.listDatabases(connectionId); - const database = (await this.apiWrapper.showQuickPick(databaseList.map(dbName => { return { label: dbName }; })))?.label; + let database; - if (!database) { - throw new Error(constants.databaseSelectionRequired); + // use database that was connected to if it isn't master + if (connection.options['database'] && connection.options['database'] !== constants.master) { + database = connection.options['database']; + } else { + const databaseList = await this.apiWrapper.listDatabases(connectionId); + database = (await this.apiWrapper.showQuickPick(databaseList.map(dbName => { return { label: dbName }; })))?.label; + + if (!database) { + throw new Error(constants.databaseSelectionRequired); + } } model.serverId = connectionId;