That's what we call a memory leak, pt2

Changed MessageHandling to use event's message content.

Registered MessageRegexListener and QuitListener event listeners.

Author: Peashooter101

src/main/java/adhdmc/simplepms/SimplePMs.java

@@ -6,6 +6,8 @@
 import adhdmc.simplepms.commands.SocialSpyCommand;
 import adhdmc.simplepms.config.LocaleConfig;
 import adhdmc.simplepms.listeners.LoginListener;
+import adhdmc.simplepms.listeners.MessageRegexListener;
+import adhdmc.simplepms.listeners.QuitListener;
 import net.kyori.adventure.text.minimessage.MiniMessage;
 import org.bukkit.entity.Player;
 import org.bukkit.plugin.Plugin;
@@ -25,6 +27,8 @@ public void onEnable() {
         instance = this;
         registerCommands();
         this.getServer().getPluginManager().registerEvents(new LoginListener(), this);
+        this.getServer().getPluginManager().registerEvents(new QuitListener(), this);
+        this.getServer().getPluginManager().registerEvents(new MessageRegexListener(), this);
         if (this.getServer().getPluginManager().getPlugin("PlaceholderAPI") != null) {
             papiEnabled = true;
         } else {

src/main/java/adhdmc/simplepms/handling/MessageHandling.java

@@ -45,12 +45,12 @@ public void playerSenderAndReceiver(Player initiator, Player recipient, String m
         if (event.isCancelled()) return;
         initiator.getPersistentDataContainer().set(lastMessaged, PersistentDataType.STRING, recipient.getName());
         recipient.getPersistentDataContainer().set(lastMessaged, PersistentDataType.STRING, initiator.getName());
-        initiator.sendMessage(Resolvers.getInstance().parseMessagePlayerToPlayer(Message.SENDING_FORMAT.getMessage(), initiator, recipient, messageContent));
-        recipient.sendMessage(Resolvers.getInstance().parseMessagePlayerToPlayer(Message.RECEIVING_FORMAT.getMessage(), initiator, recipient, messageContent));
+        initiator.sendMessage(Resolvers.getInstance().parseMessagePlayerToPlayer(Message.SENDING_FORMAT.getMessage(), initiator, recipient, event.getMessageContent()));
+        recipient.sendMessage(Resolvers.getInstance().parseMessagePlayerToPlayer(Message.RECEIVING_FORMAT.getMessage(), initiator, recipient, event.getMessageContent()));
         for (Player spy : spyingPlayers) {
             if (!spy.isOnline()) continue;
             if (spy.equals(initiator) || spy.equals(recipient)) continue;
-            spy.sendMessage(Resolvers.getInstance().parseMessagePlayerToPlayer(Message.SPY_FORMAT.getMessage(), initiator, recipient, messageContent));
+            spy.sendMessage(Resolvers.getInstance().parseMessagePlayerToPlayer(Message.SPY_FORMAT.getMessage(), initiator, recipient, event.getMessageContent()));
         }
     }
 
@@ -73,13 +73,13 @@ public void consoleSenderPlayerReceiver(CommandSender initiator, Player recipien
         Bukkit.getServer().getPluginManager().callEvent(event);
         if (event.isCancelled()) return;
         recipient.getPersistentDataContainer().set(lastMessaged, PersistentDataType.STRING, Message.PDC_CONSOLE.getMessage());
-        recipient.sendMessage(Resolvers.getInstance().parseMessageConsoleToPlayer(Message.RECEIVING_FORMAT.getMessage(), consoleChatComponent, recipient, messageContent));
-        initiator.sendMessage(Resolvers.getInstance().parseMessageConsoleToPlayer(Message.SENDING_FORMAT.getMessage(), consoleChatComponent, recipient, messageContent));
+        recipient.sendMessage(Resolvers.getInstance().parseMessageConsoleToPlayer(Message.RECEIVING_FORMAT.getMessage(), consoleChatComponent, recipient, event.getMessageContent()));
+        initiator.sendMessage(Resolvers.getInstance().parseMessageConsoleToPlayer(Message.SENDING_FORMAT.getMessage(), consoleChatComponent, recipient, event.getMessageContent()));
         for (Player spy : spyingPlayers) {
             if (!spy.isOnline()) continue;
             if (spy.equals(initiator) || spy.equals(recipient)) continue;
             if (!spy.hasPermission(Perm.CONSOLE_MESSAGE_SPY.getPerm())) continue;
-            spy.sendMessage(Resolvers.getInstance().parseMessageConsoleToPlayer(Message.SPY_FORMAT.getMessage(), consoleSpyComponent, recipient, messageContent));
+            spy.sendMessage(Resolvers.getInstance().parseMessageConsoleToPlayer(Message.SPY_FORMAT.getMessage(), consoleSpyComponent, recipient, event.getMessageContent()));
         }
     }
 
@@ -107,12 +107,12 @@ public void playerSenderConsoleReceiver(CommandSender initiator, String messageC
         Bukkit.getServer().getPluginManager().callEvent(event);
         if (event.isCancelled()) return;
         initiatingPlayer.getPersistentDataContainer().set(lastMessaged, PersistentDataType.STRING, Message.PDC_CONSOLE.getMessage());
-        initiatingPlayer.sendMessage(Resolvers.getInstance().parseMessagePlayerToConsole(Message.SENDING_FORMAT.getMessage(), initiatingPlayer, consoleChatComponent, messageContent));
+        initiatingPlayer.sendMessage(Resolvers.getInstance().parseMessagePlayerToConsole(Message.SENDING_FORMAT.getMessage(), initiatingPlayer, consoleChatComponent, event.getMessageContent()));
         for (Player spy : spyingPlayers) {
             if (!spy.isOnline()) continue;
             if (spy.equals(initiator)) continue;
             if (!spy.hasPermission(Perm.CONSOLE_MESSAGE_SPY.getPerm())) continue;
-            spy.sendMessage(Resolvers.getInstance().parseMessagePlayerToConsole(Message.SPY_FORMAT.getMessage(),initiatingPlayer, consoleSpyComponent, messageContent));
+            spy.sendMessage(Resolvers.getInstance().parseMessagePlayerToConsole(Message.SPY_FORMAT.getMessage(),initiatingPlayer, consoleSpyComponent, event.getMessageContent()));
         }
     }
 }