feat: improve Settings UI

idea-plugin/src/main/kotlin/com/itangcent/idea/plugin/configurable/AbstractEasyApiConfigurable.kt

@@ -35,7 +35,7 @@ abstract class AbstractEasyApiConfigurable(private var myProject: Project?) : Se
             return false
         }
         if (throttle.acquire(5000L)) {
-            LOG.info("settings is modified:\npre:----\n${pre.toJson()}\ninUI:----\n${inUI.toJson()}\n----\n")
+            LOG.info("settings is modified at ${this::class.qualifiedName}:\npre:----\n${pre.toJson()}\ninUI:----\n${inUI.toJson()}\n----\n")
         }
         return true
     }
@@ -72,33 +72,11 @@ abstract class AbstractEasyApiConfigurable(private var myProject: Project?) : Se
         easyApiConfigurableGUI = createGUI()
 
         context.init(easyApiConfigurableGUI)
-        context.runAsync {
-            context.runInSwingUI {
-                easyApiConfigurableGUI.onCreate()
-                easyApiConfigurableGUI.setSettings(settingBinder.read().copy())
-                checkUI()
-            }
-        }
+        easyApiConfigurableGUI.onCreate()
+        easyApiConfigurableGUI.setSettings(settingBinder.read().copy())
         return easyApiConfigurableGUI.getRootPanel()
     }
 
-    private fun checkUI() {
-        context!!.runAsync {
-            for (i in 0..8) {
-                Thread.sleep(250)
-                if (easyApiConfigurableGUI.checkUI()) {
-                    continue
-                }
-                LOG.error("checkUI failed.Try refresh ui again.")
-                context!!.runInSwingUI {
-                    easyApiConfigurableGUI.setSettings(easyApiConfigurableGUI.getSettings())
-                    checkUI()
-                }
-                break
-            }
-        }
-    }
-
     open fun afterBuildActionContext(builder: ActionContext.ActionContextBuilder) {}
 
     protected abstract fun createGUI(): EasyApiSettingGUI

idea-plugin/src/main/kotlin/com/itangcent/idea/plugin/configurable/AbstractEasyApiSettingGUI.kt

@@ -1,29 +1,23 @@
 package com.itangcent.idea.plugin.configurable
 
 import com.itangcent.idea.plugin.settings.Settings
-import com.itangcent.intellij.extend.rx.AutoComputer
 
 abstract class AbstractEasyApiSettingGUI : EasyApiSettingGUI {
 
     protected var settingsInstance: Settings? = null
 
-    protected var autoComputer: AutoComputer = AutoComputer()
-
-    override fun checkUI(): Boolean {
-        return true
+    override fun onCreate() {
+        //NOP
     }
 
     override fun getSettings(): Settings {
-        return (settingsInstance ?: Settings()).copy()
+        return (settingsInstance ?: Settings()).copy().also { readSettings((it)) }
     }
 
     override fun setSettings(settings: Settings) {
-        autoComputer.value(this::settingsInstance, settings.copy())
+        this.settingsInstance = settings
     }
 
     override fun readSettings(settings: Settings) {
-        readSettings(settings, getSettings())
     }
-
-    protected abstract fun readSettings(settings: Settings, from: Settings)
 }

idea-plugin/src/main/kotlin/com/itangcent/idea/plugin/configurable/EasyApiSettingGUI.kt

@@ -4,15 +4,23 @@ import com.itangcent.idea.plugin.settings.Settings
 import javax.swing.JComponent
 
 interface EasyApiSettingGUI {
+
     fun getRootPanel(): JComponent?
 
     fun onCreate()
 
-    fun checkUI(): Boolean
-
+    /**
+     * get settings in UI
+     */
     fun getSettings(): Settings
 
+    /**
+     * set settings to UI
+     */
     fun setSettings(settings: Settings)
 
+    /**
+     * read settings from UI
+     */
     fun readSettings(settings: Settings)
 }

idea-plugin/src/main/kotlin/com/itangcent/idea/plugin/dialog/EasyApiSettingBuiltInConfigGUI.kt

@@ -17,17 +17,12 @@ class EasyApiSettingBuiltInConfigGUI : AbstractEasyApiSettingGUI() {
         return rootPanel
     }
 
-    override fun onCreate() {
-        autoComputer.bind(this.builtInConfigTextArea!!)
-            .with<String?>(this, "settingsInstance.builtInConfig")
-            .eval { it.takeIf { it.notNullOrBlank() } ?: EasyApiSettingGUI.DEFAULT_BUILT_IN_CONFIG }
-
-        autoComputer.bind<String?>(this, "settingsInstance.builtInConfig")
-            .with(builtInConfigTextArea!!)
-            .eval { it.takeIf { it != EasyApiSettingGUI.DEFAULT_BUILT_IN_CONFIG } ?: "" }
+    override fun setSettings(settings: Settings) {
+        super.setSettings(settings)
+        this.builtInConfigTextArea!!.text = settings.builtInConfig.takeIf { it.notNullOrBlank() } ?: EasyApiSettingGUI.DEFAULT_BUILT_IN_CONFIG
     }
 
-    override fun readSettings(settings: Settings, from: Settings) {
-        settings.builtInConfig = from.builtInConfig
+    override fun readSettings(settings: Settings) {
+        settings.builtInConfig =  this.builtInConfigTextArea!!.text.takeIf { it != EasyApiSettingGUI.DEFAULT_BUILT_IN_CONFIG } ?: ""
     }
 }