Remove console spam after reload

Author: MrMicky-FR

README.md

@@ -12,15 +12,17 @@ This plugin fix this problem, so 1.11 and higher players will able to send longe
 
 ## Downloads
 
-You can download releases and find more information on SpigotMC: https://www.spigotmc.org/resources/viachatfixer.61955/
+You can download releases and find more information on [SpigotMC](https://www.spigotmc.org/resources/viachatfixer.61955/).
 
 ## Installation
+
 Just put the ViaChatFixer jar in your plugins folder.
 Currently this plugin only works on Spigot, but Sponge support should be added soon.
 This plugin can work with BungeeCord as long as ViaVersion and this plugin are on the Spigot servers.
 
 **All players online during a reload will not be able to have longer chat messages until they reconnect**
 
 ## Planned features
+
 * Sponge support
-* BungeeCord support
+* BungeeCord support _(not sure)_

bukkit/src/main/java/fr/mrmicky/viachatfixer/bukkit/ViaChatFixerBukkit.java

@@ -2,7 +2,7 @@
 
 import fr.mrmicky.viachatfixer.ViaChatFixerPlatform;
 import fr.mrmicky.viachatfixer.handlers.ChatHandler;
-import fr.mrmicky.viachatfixer.handlers.via.ViaVersionChatHandler;
+import fr.mrmicky.viachatfixer.handlers.via.ViaChatHandler;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.EventPriority;
 import org.bukkit.event.Listener;
@@ -12,9 +12,6 @@
 
 import java.util.logging.Level;
 
-/**
- * @author MrMicky
- */
 public final class ViaChatFixerBukkit extends JavaPlugin implements ViaChatFixerPlatform, Listener {
 
     private ChatHandler chatHandler;
@@ -27,14 +24,14 @@ public void onEnable() {
             return;
         }
 
-        chatHandler = new ViaVersionChatHandler(this);
+        chatHandler = new ViaChatHandler(this);
 
         // Only load when ViaVersion is loaded
         getServer().getScheduler().runTask(this, () -> {
             try {
                 chatHandler.init();
             } catch (Exception e) {
-                getLogger().log(Level.SEVERE, "An error occurred during init", e);
+                getLogger().log(Level.SEVERE, "An error occurred while enabling", e);
                 getServer().getPluginManager().disablePlugin(this);
                 return;
             }

common/src/main/java/fr/mrmicky/viachatfixer/ViaChatFixerPlatform.java

@@ -2,9 +2,6 @@
 
 import java.util.logging.Logger;
 
-/**
- * @author MrMicky
- */
 public interface ViaChatFixerPlatform {
 
     Logger getLogger();

common/src/main/java/fr/mrmicky/viachatfixer/handlers/ChatHandler.java

@@ -2,9 +2,6 @@
 
 import java.util.UUID;
 
-/**
- * @author MrMicky
- */
 public interface ChatHandler {
 
     void init() throws Exception;

common/src/main/java/fr/mrmicky/viachatfixer/handlers/via/ChatTracker.java

@@ -3,9 +3,6 @@
 import us.myles.ViaVersion.api.data.StoredObject;
 import us.myles.ViaVersion.api.data.UserConnection;
 
-/**
- * @author MrMicky
- */
 public class ChatTracker extends StoredObject {
 
     private String lastMessage;
@@ -19,15 +16,12 @@ public String getLastMessage() {
         return lastMessage;
     }
 
-    public void setLastMessage(String lastMessage) {
-        this.lastMessage = lastMessage;
-    }
-
     public boolean isValid(int time) {
         return (System.currentTimeMillis() - lastMessageTime) < time;
     }
 
-    public void updateLastMessageTime() {
+    public void updateLastMessage(String message) {
+        lastMessage = message;
         lastMessageTime = System.currentTimeMillis();
     }
 

…/handlers/via/ViaVersionChatHandler.java …atfixer/handlers/via/ViaChatHandler.javacommon/src/main/java/fr/mrmicky/viachatfixer/handlers/via/ViaVersionChatHandler.java renamed to common/src/main/java/fr/mrmicky/viachatfixer/handlers/via/ViaChatHandler.java common/src/main/java/fr/mrmicky/viachatfixer/handlers/via/ViaVersionChatHandler.java renamed to common/src/main/java/fr/mrmicky/viachatfixer/handlers/via/ViaChatHandler.java

@@ -15,17 +15,18 @@
 import us.myles.ViaVersion.protocols.protocol1_11to1_10.Protocol1_11To1_10;
 
 import java.lang.reflect.Field;
+import java.util.HashSet;
 import java.util.Map;
+import java.util.Set;
 import java.util.UUID;
 
-/**
- * @author MrMicky
- */
-public class ViaVersionChatHandler implements ChatHandler {
+public class ViaChatHandler implements ChatHandler {
+
+    private final Set<UUID> unknownPlayers = new HashSet<>();
 
     private ViaChatFixerPlatform platform;
 
-    public ViaVersionChatHandler(ViaChatFixerPlatform platform) {
+    public ViaChatHandler(ViaChatFixerPlatform platform) {
         this.platform = platform;
     }
 
@@ -71,16 +72,16 @@ public void handle(PacketWrapper wrapper) throws Exception {
                             ChatTracker chatTracker = connection.get(ChatTracker.class);
 
                             if (chatTracker == null) {
-                                connection.put(chatTracker = new ChatTracker(connection));
+                                chatTracker = new ChatTracker(connection);
+                                connection.put(chatTracker);
                             }
 
                             // don't allow messages longer than 256 characters
                             if (msg.length() > 256) {
                                 msg = msg.substring(0, 256);
                             }
 
-                            chatTracker.setLastMessage(msg);
-                            chatTracker.updateLastMessageTime();
+                            chatTracker.updateLastMessage(msg);
                         }
                     }
                 });
@@ -93,7 +94,9 @@ public String handle(UUID uuid) {
         UserConnection connection = Via.getManager().getConnection(uuid);
 
         if (connection == null) {
-            platform.getLogger().warning("Unknown connection for player with UUID " + uuid);
+            if (unknownPlayers.add(uuid)) {
+                platform.getLogger().warning("Unknown connection for player with UUID " + uuid);
+            }
             return null;
         }
 

pom.xml

@@ -40,21 +40,15 @@
     <repositories>
         <repository>
             <id>viaversion-repo</id>
-            <url>https://repo.viaversion.com</url>
+            <url>https://repo.viaversion.com/</url>
         </repository>
     </repositories>
 
     <dependencies>
         <dependency>
             <groupId>us.myles</groupId>
             <artifactId>viaversion</artifactId>
-            <version>2.0.0-SNAPSHOT</version>
-            <scope>provided</scope>
-        </dependency>
-        <dependency>
-            <groupId>io.netty</groupId>
-            <artifactId>netty-all</artifactId>
-            <version>4.0.20.Final</version>
+            <version>2.1.3</version>
             <scope>provided</scope>
         </dependency>
     </dependencies>

universal/pom.xml

@@ -5,8 +5,8 @@
     <modelVersion>4.0.0</modelVersion>
 
     <parent>
-        <artifactId>viachatfixer</artifactId>
         <groupId>fr.mrmicky</groupId>
+        <artifactId>viachatfixer</artifactId>
         <version>0.2.0</version>
     </parent>