Restore typed options after unchecking check box

Author: vol0n

clion-plugin/src/main/kotlin/org/utbot/cpp/clion/plugin/ui/ObservableValue.kt

@@ -5,9 +5,10 @@ import kotlin.properties.Delegates
 // allows attaching multiple listeners for value change
 class ObservableValue<T>(initialValue: T) {
     private val changeListeners: MutableList<(T) -> Unit> = mutableListOf()
-    var value: T by Delegates.observable(initialValue) { _, _, newVal ->
-        changeListeners.forEach {
-            it(newVal)
+    var value: T by Delegates.observable(initialValue) { _, oldVal, newVal ->
+        changeListeners.forEach {callback ->
+            if (oldVal != newVal)
+                callback(newVal)
         }
     }
 

clion-plugin/src/main/kotlin/org/utbot/cpp/clion/plugin/ui/wizard/steps/ConnectionStep.kt

@@ -56,13 +56,33 @@ class ConnectionStep(
     private val connectionStatus = ObservableValue(ConnectionStatus.Failed)
     private val useConnectionDefaults = ObservableValue(false)
 
+    inner class ConnectionInfo(val port: Int, val host: String, val remotePath: String) {
+        constructor(): this(portComponent.number, hostTextField.text, remotePathTextField.text)
+        fun apply() {
+            portComponent.number = port
+            hostTextField.text = host
+            remotePathTextField.text = remotePath
+        }
+    }
+
+    private val defaultConnectionInfo = ConnectionInfo(
+        UTBotAllProjectSettings.DEFAULT_PORT,
+        UTBotAllProjectSettings.DEFAULT_HOST,
+        if (isWindows) project.path.toWslFormatIfNeeded()
+                else UTBotProjectStoredSettings.REMOTE_PATH_VALUE_FOR_LOCAL_SCENARIO
+    )
+
+    private var beforeCheckingBoxConnectionInfo: ConnectionInfo? = null
+
     init {
         useConnectionDefaults.addOnChangeListener { newValue ->
             if (newValue) {
-                portComponent.number = UTBotAllProjectSettings.DEFAULT_PORT
-                hostTextField.text = UTBotAllProjectSettings.DEFAULT_HOST
-                remotePathTextField.text = if (isWindows) project.path.toWslFormatIfNeeded()
-                    else UTBotProjectStoredSettings.REMOTE_PATH_VALUE_FOR_LOCAL_SCENARIO
+                beforeCheckingBoxConnectionInfo = ConnectionInfo()
+                println("remembered before check info: ${beforeCheckingBoxConnectionInfo?.remotePath ?: ""}")
+                defaultConnectionInfo.apply()
+            } else {
+                println("Apply before check info: ${beforeCheckingBoxConnectionInfo?.remotePath ?: "empty"}")
+                beforeCheckingBoxConnectionInfo?.apply()
             }
         }
     }
@@ -88,6 +108,7 @@ class ConnectionStep(
                     .bindSelected(getter = { useConnectionDefaults.value }, setter = { newValue ->
                         useConnectionDefaults.value = newValue
                     }).selected.addListener { newValue ->
+                        println("New value was set to checkbox: $newValue")
                         useConnectionDefaults.value = newValue
                     }
             }
@@ -143,7 +164,7 @@ class ConnectionStep(
                 })
 
                 val warningMessage: () -> String = {
-                    "⚠️ Warning! Versions are different or not defined:" +
+                    "⚠️ Warning! Versions are different or not defined: " +
                             "Client: ${ourPluginVersion} Server: ${serverVersion ?: "not defined"}"
                 }
                 label(warningMessage()).visibleIf(
@@ -172,11 +193,11 @@ class ConnectionStep(
                     .applyToComponent {
                         remotePathTextField = this
                     }.enabledIf(object : ComponentPredicate() {
-                    override fun invoke() = !useConnectionDefaults.value
-                    override fun addListener(listener: (Boolean) -> Unit) {
-                        useConnectionDefaults.addOnChangeListener { newValue -> listener(!newValue) }
-                    }
-                })
+                        override fun invoke() = !useConnectionDefaults.value
+                        override fun addListener(listener: (Boolean) -> Unit) {
+                            useConnectionDefaults.addOnChangeListener { newValue -> listener(!newValue) }
+                        }
+                    })
             }
         }.addToUI()
     }