Extract LuckPermsHook

Author: Jakubk15

chatformatter-core/src/main/java/com/eternalcode/formatter/ChatFormatterPlugin.java

@@ -2,6 +2,7 @@
 
 import com.eternalcode.formatter.config.ConfigManager;
 import com.eternalcode.formatter.config.PluginConfig;
+import com.eternalcode.formatter.mention.LuckPermsHook;
 import com.eternalcode.formatter.mention.MentionListener;
 import com.eternalcode.formatter.mention.MentionPlayerSettings;
 import com.eternalcode.formatter.mention.MentionService;
@@ -56,7 +57,7 @@ public ChatFormatterPlugin(Plugin plugin) {
         new Metrics(plugin, 15199);
 
         this.chatHandler = new ChatHandlerImpl(miniMessage, pluginConfig, this.rankProvider, this.placeholderRegistry, this.templateService);
-        MentionPlayerSettings mentionPlayerSettings = new MentionPlayerSettings(server, plugin.getLogger(), configManager.getPluginConfig().mentions);
+        MentionPlayerSettings mentionPlayerSettings = new MentionPlayerSettings(plugin.getLogger(), configManager.getPluginConfig().mentions, LuckPermsHook.initialize(server, plugin.getLogger()));
 
         server.getPluginCommand("chatformatter").setExecutor(new ChatFormatterCommand(configManager, audienceProvider, miniMessage, mentionPlayerSettings));
 

chatformatter-core/src/main/java/com/eternalcode/formatter/mention/LuckPermsHook.java

@@ -0,0 +1,61 @@
+package com.eternalcode.formatter.mention;
+
+import net.luckperms.api.LuckPerms;
+import net.luckperms.api.LuckPermsProvider;
+import net.luckperms.api.model.user.User;
+import net.luckperms.api.node.types.MetaNode;
+import org.bukkit.Server;
+
+import java.util.Optional;
+import java.util.UUID;
+import java.util.logging.Logger;
+
+public class LuckPermsHook {
+
+    private static final String MENTION_SOUND_META_KEY = "chatformatter-mention-sound";
+
+    private final LuckPerms luckPerms;
+
+    private LuckPermsHook(LuckPerms luckPerms) {
+        this.luckPerms = luckPerms;
+    }
+
+    public static Optional<LuckPermsHook> initialize(Server server, Logger logger) {
+        if (!server.getPluginManager().isPluginEnabled("LuckPerms")) {
+            return Optional.empty();
+        }
+
+        try {
+            return Optional.of(new LuckPermsHook(LuckPermsProvider.get()));
+        } catch (IllegalStateException e) {
+            logger.warning("Failed to initialize LuckPerms API: " + e.getMessage());
+            return Optional.empty();
+        }
+    }
+
+    public Optional<Boolean> getMentionSoundEnabled(UUID uuid) {
+        User user = this.luckPerms.getUserManager().getUser(uuid);
+        if (user == null) {
+            return Optional.empty();
+        }
+
+        String metaValue = user.getCachedData().getMetaData().getMetaValue(MENTION_SOUND_META_KEY);
+        if (metaValue == null) {
+            return Optional.empty();
+        }
+
+        return Optional.of(Boolean.parseBoolean(metaValue));
+    }
+
+    public void setMentionSoundEnabled(UUID uuid, boolean enabled) {
+        this.luckPerms.getUserManager().modifyUser(uuid, user -> {
+            user.data().clear(node ->
+                node instanceof MetaNode metaNode && metaNode.getMetaKey().equals(MENTION_SOUND_META_KEY)
+            );
+
+            MetaNode metaNode = MetaNode.builder(MENTION_SOUND_META_KEY, Boolean.toString(enabled)).build();
+            user.data().add(metaNode);
+        });
+    }
+}
+

chatformatter-core/src/main/java/com/eternalcode/formatter/mention/MentionPlayerSettings.java

@@ -1,75 +1,34 @@
 package com.eternalcode.formatter.mention;
 
-import net.luckperms.api.LuckPerms;
-import net.luckperms.api.LuckPermsProvider;
-import net.luckperms.api.model.user.User;
-import net.luckperms.api.node.types.MetaNode;
-import org.bukkit.Server;
-
+import java.util.Optional;
 import java.util.UUID;
 import java.util.logging.Logger;
 
 public class MentionPlayerSettings {
 
-    private static final String MENTION_SOUND_META_KEY = "chatformatter-mention-sound";
-
     private final Logger logger;
     private final MentionConfig config;
-    private final Server server;
-    private LuckPerms luckPerms;
+    private final Optional<LuckPermsHook> luckPermsHook;
 
-    public MentionPlayerSettings(Server server, Logger logger, MentionConfig config) {
-        this.server = server;
+    public MentionPlayerSettings(Logger logger, MentionConfig config, Optional<LuckPermsHook> luckPermsHook) {
         this.logger = logger;
         this.config = config;
-        this.initializeLuckPerms();
-    }
-
-    private void initializeLuckPerms() {
-        if (!this.server.getPluginManager().isPluginEnabled("LuckPerms")) {
-            this.logger.warning("LuckPerms is not installed! Mention sound toggle feature will not work.");
-            return;
-        }
-
-        try {
-            this.luckPerms = LuckPermsProvider.get();
-        } catch (IllegalStateException e) {
-            this.logger.warning("Failed to initialize LuckPerms API: " + e.getMessage());
-        }
+        this.luckPermsHook = luckPermsHook;
     }
 
     public boolean isMentionSoundEnabled(UUID uuid) {
-        if (this.luckPerms == null) {
-            return this.config.enabled;
-        }
-
-        User user = this.luckPerms.getUserManager().getUser(uuid);
-        if (user == null) {
-            return this.config.enabled;
-        }
-
-        String metaValue = user.getCachedData().getMetaData().getMetaValue(MENTION_SOUND_META_KEY);
-        if (metaValue == null) {
-            return this.config.enabled;
-        }
-
-        return Boolean.parseBoolean(metaValue);
+        return this.luckPermsHook
+            .flatMap(hook -> hook.getMentionSoundEnabled(uuid))
+            .orElse(this.config.enabled);
     }
 
     public void setMentionSoundEnabled(UUID uuid, boolean enabled) {
-        if (this.luckPerms == null) {
+        if (this.luckPermsHook.isEmpty()) {
             this.logger.warning("Cannot set mention sound preference - LuckPerms is not available!");
             return;
         }
 
-        this.luckPerms.getUserManager().modifyUser(uuid, user -> {
-            user.data().clear(node ->
-                node instanceof MetaNode metaNode && metaNode.getMetaKey().equals(MENTION_SOUND_META_KEY)
-            );
-
-            MetaNode metaNode = MetaNode.builder(MENTION_SOUND_META_KEY, Boolean.toString(enabled)).build();
-            user.data().add(metaNode);
-        });
+        this.luckPermsHook.get().setMentionSoundEnabled(uuid, enabled);
     }
 
     public boolean toggleMentionSound(UUID uuid) {