Less settings form logging.

antonio-petricca · Sep 28, 2024 · dbb2568 · dbb2568
1 parent 78ec7d3
commit dbb2568
Showing 1 changed file with 10 additions and 16 deletions.
diff --git a/src/main/kotlin/com/github/intellij/plugins/mt4ij/config/SettingsConfigurable.kt b/src/main/kotlin/com/github/intellij/plugins/mt4ij/config/SettingsConfigurable.kt
@@ -37,24 +37,18 @@ internal class SettingsConfigurable(private val project: Project) : Configurable
     }
 
     override fun isModified(): Boolean {
-        log.debug("Checking if settings are modified...")
+        val state = getState()
 
-        val settings = SettingsStorage
-            .getInstance(project)
-            .state
-
-        return settingsForm?.isModified(settings) ?: false
+        return settingsForm?.isModified(state) ?: false
     }
 
     override fun apply() {
         log.info("Applying settings...")
 
-        val settings = SettingsStorage
-            .getInstance(project)
-            .state
+        val state = getState()
 
         settingsForm?.apply {
-            settings.templatesPath = (getTemplatesPath() ?: "")
+            state.templatesPath = (getTemplatesPath() ?: "")
         }
 
         ApiHelpers.invokeLater(
@@ -66,18 +60,18 @@ internal class SettingsConfigurable(private val project: Project) : Configurable
     }
 
     override fun getDisplayName(): String {
-        log.debug("Getting display name...")
-
         return Bundle.message("mt4ij.settings.name")
     }
 
     override fun reset() {
-        log.info("Resetting settings...")
+        val state = getState()
+
+        settingsForm?.setTemplatesPath(state.templatesPath)
+    }
 
-        val settings = SettingsStorage
+    private fun getState(): SettingsState {
+        return SettingsStorage
             .getInstance(project)
             .state
-
-        settingsForm?.setTemplatesPath(settings.templatesPath)
     }
 }