fixed quest objective doubling issue

src/main/java/net/Indyuce/mmocore/api/player/PlayerData.java

@@ -130,6 +130,12 @@ public class PlayerData extends OfflinePlayerData implements Closable {
 
     @Override
     public void close() {
+
+        // Remove from party
+        if (hasParty())
+            getParty().removeMember(this);
+
+        // Close quest data
         questData.close();
     }
 

src/main/java/net/Indyuce/mmocore/listener/PlayerListener.java

@@ -67,10 +67,7 @@ public class PlayerListener implements Listener {
     @EventHandler
     public void e(PlayerQuitEvent event) {
         PlayerData playerData = PlayerData.get(event.getPlayer());
-        if (playerData.hasParty())
-            playerData.getParty().removeMember(playerData);
-
-        MMOCore.plugin.dataProvider.getDataManager().remove(playerData);
+        MMOCore.plugin.dataProvider.getDataManager().unregisterSafe(playerData);
     }
 
     /**

src/main/java/net/Indyuce/mmocore/manager/data/PlayerDataManager.java

@@ -35,12 +35,18 @@ public abstract class PlayerDataManager {
     }
 
     /**
-     * Clears the data map from a certain key
+     * Safely unregisters the player data from the map.
+     * This saves the player data either through SQL or YAML,
+     * then closes the player data and clears it from the data map.
      *
-     * @param uuid Player UUID to clear
+     * @param playerData PLayer data to unregister
      */
-    protected void unregisterData(UUID uuid) {
-        data.remove(uuid);
+    public void unregisterSafe(PlayerData playerData) {
+        if (playerData.isFullyLoaded())
+            saveData(playerData);
+
+        playerData.close();
+        this.data.remove(playerData.getUniqueId());
     }
 
     /**
@@ -119,8 +125,6 @@ public abstract class PlayerDataManager {
      */
     public abstract void saveData(PlayerData data);
 
-    public abstract void remove(PlayerData data);
-
     public static class DefaultPlayerData {
         private final int level, classPoints, skillPoints, attributePoints, attrReallocPoints;
 

src/main/java/net/Indyuce/mmocore/manager/data/mysql/MySQLPlayerDataManager.java

@@ -235,13 +235,4 @@ public class MySQLPlayerDataManager extends PlayerDataManager {
 			return lastLogin;
 		}
 	}
-
-	@Override
-	public void remove(PlayerData data) {
-		if (data.isFullyLoaded())
-			saveData(data);
-
-		data.close();
-		unregisterData(data.getUniqueId());
-	}
 }

src/main/java/net/Indyuce/mmocore/manager/data/yaml/YAMLPlayerDataManager.java

@@ -127,10 +127,4 @@ public class YAMLPlayerDataManager extends PlayerDataManager {
 	public OfflinePlayerData getOffline(UUID uuid) {
 		return isLoaded(uuid) ? get(uuid) : new YAMLOfflinePlayerData(uuid);
 	}
-
-	@Override
-	public void remove(PlayerData data) {
-		data.close();
-		unregisterData(data.getUniqueId());
-	}
 }