Fix debug message for LuckPermsHook

jonagamerpro1234 · Nov 24, 2022 · 306b1e0 · 306b1e0
1 parent 7b95c6a
commit 306b1e0
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 7 deletions.
diff --git a/...ndQuitMessagesOld/src/main/java/jss/customjoinandquitmessages/config/PreConfigLoader.java b/...ndQuitMessagesOld/src/main/java/jss/customjoinandquitmessages/config/PreConfigLoader.java
@@ -23,6 +23,7 @@ public void loadConfig() {
         try {
 
             //Other
+            Settings.settings_debug = config.getBoolean("Config.Debug");
             Settings.update = Objects.equals(config.getString("Config.Update"), "true");
             Settings.c_type = config.getString("Config.Type");
             Settings.is_Group_Display = Objects.requireNonNull(config.getString("Config.Type")).equalsIgnoreCase("group");

diff --git a/CustomJoinAndQuitMessagesOld/src/main/java/jss/customjoinandquitmessages/gui/DisplayGui.java b/CustomJoinAndQuitMessagesOld/src/main/java/jss/customjoinandquitmessages/gui/DisplayGui.java
@@ -14,8 +14,8 @@ public class DisplayGui {
     private Inventory inv;
     private ItemStack item;
     private ItemMeta meta;
-    private Player player;
-    private CustomJoinAndQuitMessages plugin;
+    private final Player player;
+    private final CustomJoinAndQuitMessages plugin;
 
     public DisplayGui(Player player, CustomJoinAndQuitMessages plugin) {
         this.player = player;
@@ -28,17 +28,16 @@ public void create() {
 
         for (int i = 0; i < 54; i++) {
             inv.setItem(i, setDecoration());
-            if (i == 54) {
-                break;
-            }
         }
         setItems();
     }
 
 
     public ItemStack setDecoration() {
         item = XMaterial.BLACK_STAINED_GLASS_PANE.parseItem();
+        assert item != null;
         meta = item.getItemMeta();
+        assert meta != null;
         meta.setDisplayName(null);
         item.setItemMeta(meta);
         return item;

diff --git a/...AndQuitMessagesOld/src/main/java/jss/customjoinandquitmessages/listener/JoinListener.java b/...AndQuitMessagesOld/src/main/java/jss/customjoinandquitmessages/listener/JoinListener.java
@@ -54,7 +54,9 @@ public void onJoinListener(@NotNull PlayerJoinEvent e) {
         PlayerManager playerManager = new PlayerManager();
         playerManager.createPlayer(p, tempGroup);
 
-        Util.sendColorMessage(p,   Util.getPrefix() + " &eLuckPermHook is: " + luckPermsHook.isEnabled());
+        if(Settings.settings_debug){
+            Logger.debug(" &eLuckPermHook is: " + luckPermsHook.isEnabled());
+        }
 
         if (Settings.c_type.equalsIgnoreCase("group")){
             if (luckPermsHook.isEnabled()) {

diff --git a/CustomJoinAndQuitMessagesOld/src/main/java/jss/customjoinandquitmessages/utils/Settings.java b/CustomJoinAndQuitMessagesOld/src/main/java/jss/customjoinandquitmessages/utils/Settings.java
@@ -10,6 +10,7 @@ public class Settings {
     public static boolean is_Group_Display;
     public static boolean update;
     public static String c_type;
+    public static boolean settings_debug;
 
     //Join section
     public static String join_type;

diff --git a/CustomJoinAndQuitMessagesOld/src/main/resources/config.yml b/CustomJoinAndQuitMessagesOld/src/main/resources/config.yml
@@ -20,7 +20,7 @@ Config:
   Update: true
 
   # |Debug| shows you important information about the plugin loading 
-  Debug: false
+  Debug: true
 
   # |Type| the type is a way of knowing what is being used whether in the [normal] | [group] | [none] format
   Type: normal