ckaczor/azuredatastudio
1
0
Fork 0
mirror of https://github.com/ckaczor/azuredatastudio.git synced 2026-02-16 10:58:30 -05:00
Code Issues Packages Projects Releases Wiki Activity

Fixed cancel connectionDialog from attach to shows dup "select connection" (#4865)

Browse Source
This commit is contained in:
Yurong He
2019-04-08 14:06:46 -07:00
committed by GitHub
parent 0975e6834e
commit 88161cc37d
2 changed files with 10 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -668,7 +668,6 @@ export class NotebookModel extends Disposable implements INotebookModel {
});
} else {
this._onValidConnectionSelected.fire(false);
throw new Error('No valid connection');
}
} catch (err) {
let msg = notebookUtils.getErrorMessage(err);

14
src/sql/parts/notebook/notebookActions.ts
View File

@@ -387,7 +387,9 @@ export class AttachToDropdown extends SelectBox {
this.selectWithOptionName(msgSelectConnection);
}
else {
connections.push(msgAddNewConnection);
if (!connections.includes(msgAddNewConnection)) {
connections.push(msgAddNewConnection);
}
}
this.setOptions(connections);
}
@@ -396,9 +398,13 @@ export class AttachToDropdown extends SelectBox {
private loadWithSelectConnection(connections: string[]): string[] {
if (connections && connections.length > 0) {
connections.unshift(msgSelectConnection);
if (!connections.includes(msgSelectConnection)) {
connections.unshift(msgSelectConnection);
}
this.selectWithOptionName(msgSelectConnection);
connections.push(msgAddNewConnection);
if (!connections.includes(msgAddNewConnection)) {
connections.push(msgAddNewConnection);
}
this.setOptions(connections);
}
return connections;
@@ -470,7 +476,7 @@ export class AttachToDropdown extends SelectBox {
}
let connectionUri = this._connectionManagementService.getConnectionUri(connection);
let connectionProfile = new ConnectionProfile(this._capabilitiesService, connection);
let connectedServer = connectionProfile.title? connectionProfile.title : connectionProfile.serverName;
let connectedServer = connectionProfile.title ? connectionProfile.title : connectionProfile.serverName;
//Check to see if the same server is already there in dropdown. We only have server names in dropdown
if (attachToConnections.some(val => val === connectedServer)) {
this.loadAttachToDropdown(this.model, this.getKernelDisplayName());