feat: Hook should be disabled by default. (#49)

It should be manually enabled.
SchweizerischeBundesbahnen · Jun 20, 2024 · 30409f7 · 30409f7
1 parent c71877a
commit 30409f7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -26,6 +26,7 @@ Use Administration -> Interceptor -> Settings page to enable/disable or set spec
 
 ## Hooks installation
 Copy hook jar to hooks folder (`<polarion_home>/polarion/extensions/ch.sbb.polarion.extension.interceptor/eclipse/plugins/hooks`) and enforce hooks reloading from the settings page or restart Polarion.
+By default, newly added hooks are disabled and must be enabled manually.
 
 ## Upgrade
 

diff --git a/src/main/java/ch/sbb/polarion/extension/interceptor/settings/HookSettings.java b/src/main/java/ch/sbb/polarion/extension/interceptor/settings/HookSettings.java
@@ -24,6 +24,10 @@ public void afterSave(@NotNull HookModel what) {
 
     @Override
     public @NotNull HookModel defaultValues() {
-        return HookModel.builder().enabled(true).hookVersion(hook.getVersion()).properties(hook.getDefaultSettings()).build();
+        return HookModel.builder()
+                .enabled(false)
+                .hookVersion(hook.getVersion())
+                .properties(hook.getDefaultSettings())
+                .build();
     }
 }