Merge pull request #402 from KxSystems/remove-lbls-when-remove-conn

Remove labels when remove conn
KxSystems · Aug 15, 2024 · db77f14 · db77f14
2 parents 62a1deb + 6ab6302
commit db77f14
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/commands/serverCommand.ts b/src/commands/serverCommand.ts
@@ -192,8 +192,8 @@ export async function editInsightsConnection(
       } else {
         const oldKey = getKeyForServerName(oldAlias);
         const newKey = insightsData.alias;
+        removeConnFromLabels(oldAlias);
         if (insights[oldKey] && oldAlias !== insightsData.alias) {
-          removeConnFromLabels(oldAlias);
           const uInsights = Object.keys(insights).filter((insight) => {
             return insight !== oldKey;
           });
@@ -452,6 +452,7 @@ export async function editKdbConnection(
         return;
       } else {
         const oldKey = getKeyForServerName(oldAlias);
+        removeConnFromLabels(oldKey);
         const newKey = kdbData.serverAlias;
         const removedAuth =
           editAuth && (kdbData.username === "" || kdbData.password === "");
@@ -526,6 +527,11 @@ export async function editKdbConnection(
 
 export async function removeConnection(viewItem: KdbNode | InsightsNode) {
   const connMngService = new ConnectionManagementService();
+  removeConnFromLabels(
+    viewItem instanceof KdbNode
+      ? viewItem.details.serverAlias
+      : viewItem.details.alias,
+  );
   await connMngService.removeConnection(viewItem);
 }