Fix envs panel duplication + fix EDT issue on Envs panel rebuild

Author: MykolaYaremchuk

src/main/kotlin/org/digma/intellij/plugin/ui/common/EnvironmentsPanel.kt

@@ -170,65 +170,79 @@ class EnvironmentsPanel(
     }
 
     private fun rebuild() {
-
-        if (components.isNotEmpty()) {
-            this.components.forEach {
-                this.remove(it)
-            }
-        }
-
+        val buttonsInfo: MutableMap<String, MutableMap<String,Any>> = HashMap()
         val usageStatusResult = model.getUsageStatus()
-
         val envsThatHaveUsageSet: Set<String> = buildEnvironmentWithUsages(usageStatusResult)
         val hasUsageFunction = fun(env: String): Boolean { return envsThatHaveUsageSet.contains(env) }
-
         val relevantEnvs = buildRelevantSortedEnvironments(environmentsSupplier, hasUsageFunction)
-
-
         for (currEnv in relevantEnvs) {
-            val isSelectedEnv = currEnv.contentEquals(environmentsSupplier.getCurrent())
+            val currentButtonInfo: MutableMap<String,Any> = HashMap()
+
+            val isSelectedEnv: Boolean = currEnv.contentEquals(environmentsSupplier.getCurrent())
             val toolTip = buildToolTip(usageStatusResult, currEnv)
             val linkText = buildLinkText(currEnv, isSelectedEnv)
 
-            SwingUtilities.invokeLater {
-                buildEnvironmentsPanelButtons(currEnv, linkText, isSelectedEnv, toolTip, hasUsageFunction)
-            }
+            currentButtonInfo["isSelectedEnv"] = isSelectedEnv
+            currentButtonInfo["toolTip"] = toolTip
+            currentButtonInfo["linkText"] = linkText
+            buttonsInfo[currEnv] = currentButtonInfo
+
+        }
+        SwingUtilities.invokeLater{
+            removeExistingEnvironmentsPanelIfPresent()
+            buildEnvironmentsPanelButtons(buttonsInfo, hasUsageFunction)
+            revalidate()
         }
-        revalidate()
     }
 
-    private fun buildEnvironmentsPanelButtons(currEnv: String, linkText: String, isSelectedEnv: Boolean,
-                                              toolTip: String, hasUsageFunction: (String) -> Boolean) {
-        val envLink = EnvLink(currEnv, linkText, isSelectedEnv)
-        envLink.toolTipText = toolTip
+    private fun buildEnvironmentsPanelButtons(buttonsInfo: MutableMap<String, MutableMap<String, Any>>,
+                                              hasUsageFunction: (String) -> Boolean) {
+        buttonsInfo.forEach { button ->
+            val buttonData = button.value
+            val currEnv = button.key
+            val linkText = buttonData.getValue("linkText").toString()
+            val toolTip = buttonData.getValue("toolTip").toString()
+            val isSelectedEnv = buttonData.getValue("isSelectedEnv") as Boolean
 
-        envLink.addActionListener { event ->
+            val envLink = EnvLink(currEnv, linkText, isSelectedEnv)
+            envLink.toolTipText = toolTip
 
-            val currentSelected: EnvLink? = getSelected()
+            envLink.addActionListener { event ->
 
-            if (currentSelected === event.source) {
-                return@addActionListener
-            }
+                val currentSelected: EnvLink? = getSelected()
 
-            currentSelected?.deselect { buildLinkText(it, false) }
+                if (currentSelected === event.source) {
+                    return@addActionListener
+                }
 
-            val clickedLink: EnvLink = event.source as EnvLink
-            clickedLink.select { buildLinkText(it, true) }
+                currentSelected?.deselect { buildLinkText(it, false) }
 
-            changeEnvAlarm.cancelAllRequests()
-            changeEnvAlarm.addRequest({
-                environmentsSupplier.setCurrent(clickedLink.env)
-            }, 100)
+                val clickedLink: EnvLink = event.source as EnvLink
+                clickedLink.select { buildLinkText(it, true) }
 
-        }
+                changeEnvAlarm.cancelAllRequests()
+                changeEnvAlarm.addRequest({
+                    environmentsSupplier.setCurrent(clickedLink.env)
+                }, 100)
+
+            }
+
+            val icon: Icon = if (hasUsageFunction(currEnv)) ENVIRONMENT_HAS_USAGE else ENVIRONMENT_HAS_NO_USAGE
+            val iconComponent = JBLabel(icon)
 
-        val icon: Icon = if (hasUsageFunction(currEnv)) ENVIRONMENT_HAS_USAGE else ENVIRONMENT_HAS_NO_USAGE
-        val iconComponent = JBLabel(icon)
+            val singlePanel = SingleEnvPanel(envLink, iconComponent)
+            singlePanel.toolTipText = toolTip
 
-        val singlePanel = SingleEnvPanel(envLink, iconComponent)
-        singlePanel.toolTipText = toolTip
+            this.add(singlePanel)
+        }
+    }
 
-        this.add(singlePanel)
+    private fun removeExistingEnvironmentsPanelIfPresent() {
+        if (components.isNotEmpty()) {
+            this.components.forEach {
+                this.remove(it)
+            }
+        }
     }
 
     private fun getSelected(): EnvLink? {