Fix reload duplicate event

The asyncManager was not clear

Author: Anarchick

build.gradle

@@ -97,6 +97,10 @@ task export() {
             from jar // here it automatically reads jar file produced from jar task
             into "C:/Users/aeim/Documents/minecraft/btooom/server/plugins"
         }
+        copy {
+            from jar // here it automatically reads jar file produced from jar task
+            into "C:/Users/aeim/Documents/minecraft/law/server/survie/plugins"
+        }
     }
 }
 

src/main/java/fr/anarchick/skriptpacket/elements/Types.java

@@ -1,5 +1,6 @@
 package fr.anarchick.skriptpacket.elements;
 
+import fr.anarchick.skriptpacket.SkriptPacket;
 import org.eclipse.jdt.annotation.Nullable;
 
 import com.comphenix.protocol.PacketType;
@@ -73,8 +74,13 @@ public String toVariableNameString(PacketContainer packet) {
 
                     @Override
                     public String toString(PacketContainer packet, int flags) {
-                        packet.getAttributeCollectionModifier().getValues();
-                        return packet.toString();
+                        String str;
+                        try {
+                            str = packet.toString();
+                        } catch (Exception e) {
+                            str = "PacketContainer[type=" + packet.getType() + ", structureModifier=INVALID_DATA]";
+                        }
+                        return str;
                     }
 
                 })

src/main/java/fr/anarchick/skriptpacket/packets/PacketManager.java

@@ -87,20 +87,17 @@ public static String getPacketName(PacketType packettype) {
     }
 
     public static void onPacketEvent(PacketType packetType, ListenerPriority priority, Mode mode) {
+        final SPPacketAdapter SPPacketAdapter = new SPPacketAdapter(priority, packetType, mode);
         switch (mode) {
-            case ASYNC -> PROTOCOL_MANAGER.getAsynchronousManager().registerAsyncHandler(new SPPacketAdapter(priority, packetType, mode)).start();
-            case SYNC -> PROTOCOL_MANAGER.addPacketListener(new SPPacketAdapter(priority, packetType, mode));
-            default -> PROTOCOL_MANAGER.getAsynchronousManager().registerAsyncHandler(new SPPacketAdapter(priority, packetType, mode)).syncStart();
+            case ASYNC -> PROTOCOL_MANAGER.getAsynchronousManager().registerAsyncHandler(SPPacketAdapter).start();
+            case SYNC -> PROTOCOL_MANAGER.addPacketListener(SPPacketAdapter);
+            default -> PROTOCOL_MANAGER.getAsynchronousManager().registerAsyncHandler(SPPacketAdapter).syncStart();
         }
     }
 
     public static void removeListeners() {
-        for (PacketListener listener : PROTOCOL_MANAGER.getPacketListeners()) {
-            if (listener.getPlugin().equals(PLUGIN)) {
-                PROTOCOL_MANAGER.removePacketListeners(PLUGIN);
-                return;
-            }
-        }
+        PROTOCOL_MANAGER.removePacketListeners(PLUGIN);
+        PROTOCOL_MANAGER.getAsynchronousManager().unregisterAsyncHandlers(PLUGIN);
     }
 
     public static void sendPacket(PacketContainer packet, Player[] players) {

src/main/java/fr/anarchick/skriptpacket/packets/SPPacketAdapter.java

@@ -40,4 +40,5 @@ public void onPacketReceiving(PacketEvent event) {
                 SkriptPacket.pluginManager.callEvent(new BukkitPacketEvent(event, priority, mode, isAsync));
             }
     }
+
 }

src/main/java/fr/anarchick/skriptpacket/packets/SkriptPacketEventListener.java

@@ -23,29 +23,29 @@ private boolean equals(final Manager manager) {
         }
     }
 
-    private static final List<Manager> listeners = new ArrayList<>();
-    private static final List<Manager> simpleListeners = new ArrayList<>();
+    private static final List<Manager> LISTENERS = new ArrayList<>();
+    private static final List<Manager> SIMPLE_LISTENERS = new ArrayList<>();
 
     public static void addPacketTypes(PacketType[] packetTypes, ListenerPriority priority, Mode mode, String scriptName) {
         for (PacketType packetType : packetTypes) {
             Manager manager = new Manager(scriptName, mode, packetType, priority);
-            add(listeners, manager);
+            add(LISTENERS, manager);
         }
         update();
     }
 
     public static void update() {
-        simpleListeners.clear();
+        SIMPLE_LISTENERS.clear();
         // Create simpleListeners
-        for (Manager listener : listeners) {
+        for (Manager listener : LISTENERS) {
             Mode mode = listener.mode();
             PacketType packetType = listener.packetType();
             ListenerPriority priority = listener.priority();
             Manager manager = new Manager("", mode, packetType, priority);
-            add(simpleListeners, manager);
+            add(SIMPLE_LISTENERS, manager);
         }
 
-        for (Manager listener : simpleListeners) {
+        for (Manager listener : SIMPLE_LISTENERS) {
             Mode mode = listener.mode();
             PacketType packetType = listener.packetType();
             ListenerPriority priority = listener.priority();
@@ -71,12 +71,12 @@ public static void onReload(PreScriptLoadEvent e) {
         List<Manager> removes = new ArrayList<>();
         for (Config config : e.getScripts()) {
             String scriptName = config.getFileName();
-            for (Manager listener : listeners) {
+            for (Manager listener : LISTENERS) {
                 if (listener.scriptName().equals(scriptName))
                         removes.add(listener);
             }
         }
-        listeners.removeAll(removes);
+        LISTENERS.removeAll(removes);
         update();
     }