tidy up

dkandalov · Oct 2, 2024 · 52871f3 · 52871f3
1 parent 6ec9398
commit 52871f3
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 9 deletions.
diff --git a/resources/META-INF/plugin.xml b/resources/META-INF/plugin.xml
@@ -53,7 +53,7 @@
         <postStartupActivity implementation="limitedwip.autorevert.components.AutoRevertComponentStartup"/>
         <postStartupActivity implementation="limitedwip.tcr.components.TcrComponentStartup"/>
 
-        <projectConfigurable instance="limitedwip.common.settings.LimitedWipConfigurable"/>
+        <projectConfigurable instance="limitedwip.common.settings.LimitedWipConfigurable" displayName="Limited WIP"/>
         <projectService serviceImplementation="limitedwip.common.settings.LimitedWipSettings"/>
         <checkinHandlerFactory implementation="limitedwip.common.vcs.SuccessfulCheckin"/>
         <checkinHandlerFactory implementation="limitedwip.common.vcs.AllowCommit"/>

diff --git a/src/limitedwip/common/settings/LimitedWipConfigurable.kt b/src/limitedwip/common/settings/LimitedWipConfigurable.kt
@@ -18,13 +18,11 @@ class LimitedWipConfigurable(val project: Project): SearchableConfigurable {
         settingsForm.applyChanges()
     }
 
-    override fun reset() {
-        settingsForm.resetChanges()
-    }
+    override fun reset() = settingsForm.resetChanges()
 
     override fun isModified() = settingsForm.isModified()
 
     override fun getDisplayName() = pluginDisplayName
 
-    override fun getId(): String = pluginId
+    override fun getId() = pluginId
 }
diff --git a/src/limitedwip/common/vcs/SuccessfulCheckin.kt b/src/limitedwip/common/vcs/SuccessfulCheckin.kt
@@ -8,17 +8,16 @@ import com.intellij.openapi.vcs.checkin.CheckinHandler
 import com.intellij.openapi.vcs.checkin.CheckinHandlerFactory
 import limitedwip.common.pluginId
 
-class SuccessfulCheckin: CheckinHandlerFactory() {
+class SuccessfulCheckin : CheckinHandlerFactory() {
 
-    override fun createHandler(panel: CheckinProjectPanel, commitContext: CommitContext): CheckinHandler {
-        return object: CheckinHandler() {
+    override fun createHandler(panel: CheckinProjectPanel, commitContext: CommitContext) =
+        object : CheckinHandler() {
             override fun checkinSuccessful() {
                 val changeList = panel.project.defaultChangeList() ?: return
                 val uncommittedFileCount = changeList.changes.size - panel.selectedChanges.size
                 notifySettingsListeners(allChangesAreCommitted = uncommittedFileCount == 0)
             }
         }
-    }
 
     private fun notifySettingsListeners(allChangesAreCommitted: Boolean) {
         Extensions.getRootArea().getExtensionPoint<Listener>(extensionPointName).extensions.forEach { listener ->