New translations messages.yml (Lao)

.m2/settings.xml

@@ -0,0 +1,15 @@
+<settings>
+    <servers>
+        <server>
+            <id>nexus</id>
+            <username>${env.M2_REPO_USER}</username>
+            <password>${env.M2_REPO_PASS}</password>
+            <blocked>false</blocked>
+        </server>
+        <server>
+            <id>lumine</id>
+            <username>${env.M2_REPO_USER}</username>
+            <password>${env.M2_REPO_PASS}</password>
+        </server>
+    </servers>
+</settings>

LICENSE

@@ -1,18 +1,3 @@
-This project is owned and maintained by "Phoenix Development" and is distributed with "All Rights Reserved".
+This project is owned and maintained by PhoenixDvpt and is distributed with "All Right's Reserved".
-PhoenixDevt reserves the right to change these terms at any time, you have to comply with the most recent version.
-We do NOT provide support to servers involved with piracy in any form, or owners who have downloaded ANY of our plugins from an unofficial/illegal source.
 
-Things can you CANNOT do:
+You can fork and modify the source code of this project as you wish to meet your server's needs, and merge requests are accepted under the terms that you grant full rights to us using any pushed code. However, you may not distribute the plugin for any purpose other than providing cross-compatibility with other Minecraft plugins, and may not bypass any anti-piracy protection.
-- Issue a refund on PayPal without our explicit permission, as this is a digital good.
-- Redistribute, sell or give an official/modified version of the plugin (with or without any type of counterpart) to anyone else.
-- Modify and compile the project source code to bypass an anti-piracy protection.
-- Download, compile, decompile or use the plugin on any server without purchasing a license.
-
-Things can you CAN do when purchasing the plugin:
-- Download and decompile the plugin file.
-- Fork and modify the project source code to meet your production server's needs.
-- Use it on ONE production server or network (= proxy-connected servers) and one private test server at a time.
-
-You may propose a merge request, under the terms that you grant full rights to us using any pushed code.
-
-If you are a developer and have not purchased a license, you have the permission to download, fork, edit and compile the project source code, and sell code modifications to your client ONLY IF they have already purchased a license. This only applies to one-time comission works and does NOT apply to public sales.

MMOCore-API/pom.xml

@@ -5,18 +5,35 @@
     <parent>
         <artifactId>MMOCore</artifactId>
         <groupId>net.Indyuce</groupId>
-        <version>1.12.1-SNAPSHOT</version>
+        <version>${revision}</version>
     </parent>
     <modelVersion>4.0.0</modelVersion>
 
     <artifactId>MMOCore-API</artifactId>
 
+    <properties>
+        <maven.compiler.source>16</maven.compiler.source>
+        <maven.compiler.target>16</maven.compiler.target>
+    </properties>
+
     <build>
         <finalName>${project.name}-${project.version}</finalName>
         <sourceDirectory>src/main/java</sourceDirectory>
         <defaultGoal>clean package install</defaultGoal>
 
         <plugins>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-compiler-plugin</artifactId>
+                <version>3.8.1</version>
+                <configuration>
+                    <source>16</source>
+                    <target>16</target>
+                    <encoding>UTF-8</encoding>
+                    <compilerArgument>-proc:none</compilerArgument>
+                </configuration>
+            </plugin>
+
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-shade-plugin</artifactId>
@@ -80,13 +97,8 @@
         </repository>
 
         <repository>
-            <id>lumine</id>
+            <id>spigot-repo</id>
-            <url>https://mvn.lumine.io/repository/maven/</url>
+            <url>https://hub.spigotmc.org/nexus/content/repositories/snapshots/</url>
-        </repository>
-
-        <repository>
-            <id>jitpack.io</id>
-            <url>https://jitpack.io</url>
         </repository>
 
         <repository>
@@ -94,6 +106,11 @@
             <url>https://maven.enginehub.org/repo/</url>
         </repository>
 
+        <repository>
+            <id>papermc</id>
+            <url>https://papermc.io/repo/repository/maven-public/</url>
+        </repository>
+
         <repository>
             <id>dmulloy2-repo</id>
             <url>https://repo.dmulloy2.net/repository/public/</url>
@@ -104,39 +121,16 @@
             <id>simonsators Repo</id>
             <url>https://simonsator.de/repo</url>
         </repository>
-
-        <!-- Heroes Repository
-        <repository>
-            <id>herocraft</id>
-            <url>https://nexus.hc.to/content/repositories/pub_releases</url>
-        </repository> -->
-
     </repositories>
 
     <dependencies>
 
-        <!-- Paper API -->
+        <!-- Spigot API -->
         <dependency>
-            <groupId>io.papermc.paper</groupId>
+            <groupId>org.spigotmc</groupId>
-            <artifactId>paper-api</artifactId>
+            <artifactId>spigot-1.17.1</artifactId>
-            <version>1.20.6-R0.1-SNAPSHOT</version>
+            <version>dev</version>
             <scope>provided</scope>
-            <optional>true</optional>
-
-            <exclusions>
-                <exclusion>
-                    <groupId>com.google.code.gson</groupId>
-                    <artifactId>gson</artifactId>
-                </exclusion>
-            </exclusions>
-        </dependency>
-
-        <dependency>
-            <groupId>commons-lang</groupId>
-            <artifactId>commons-lang</artifactId>
-            <version>2.6</version>
-            <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <!-- Extra libs -->
@@ -152,32 +146,14 @@
             <artifactId>lombok</artifactId>
             <version>1.18.20</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <!-- Plugin dependencies -->
         <dependency>
             <groupId>io.lumine</groupId>
             <artifactId>Mythic-Dist</artifactId>
-            <version>5.7.1</version>
+            <version>5.0.1</version>
             <scope>provided</scope>
-            <optional>true</optional>
-        </dependency>
-
-        <dependency>
-            <groupId>net.playavalon</groupId>
-            <artifactId>MythicDungeons</artifactId>
-            <version>1.4.1-SNAPSHOT-CUSTOM</version>
-            <scope>provided</scope>
-            <optional>true</optional>
-        </dependency>
-
-        <dependency>
-            <groupId>fr.phoenixdevt</groupId>
-            <artifactId>Profile-API</artifactId>
-            <version>1.1</version>
-            <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <dependency>
@@ -185,7 +161,6 @@
             <artifactId>placeholderapi</artifactId>
             <version>2.9.2</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <dependency>
@@ -193,7 +168,6 @@
             <artifactId>VaultAPI</artifactId>
             <version>1.7.1</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <dependency>
@@ -201,18 +175,6 @@
             <artifactId>worldguard-bukkit</artifactId>
             <version>7.0.2-SNAPSHOT</version>
             <scope>provided</scope>
-            <optional>true</optional>
-
-            <exclusions>
-                <exclusion>
-                    <groupId>org.bukkit</groupId>
-                    <artifactId>bukkit</artifactId>
-                </exclusion>
-                <exclusion>
-                    <groupId>com.google.code.gson</groupId>
-                    <artifactId>gson</artifactId>
-                </exclusion>
-            </exclusions>
         </dependency>
 
         <dependency>
@@ -220,7 +182,6 @@
             <artifactId>ProtocolLib</artifactId>
             <version>4.8.0</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <dependency>
@@ -228,7 +189,6 @@
             <artifactId>Citizens</artifactId>
             <version>2.0.30</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <!-- Party plugins -->
@@ -237,7 +197,6 @@
             <artifactId>DevelopmentPAFSpigot</artifactId>
             <version>1.0.65</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <dependency>
@@ -245,7 +204,6 @@
             <artifactId>Party-and-Friends-MySQL-Edition-Spigot-API</artifactId>
             <version>1.5.4-RELEASE</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <dependency>
@@ -253,15 +211,6 @@
             <artifactId>spigot-party-api-for-party-and-friends</artifactId>
             <version>1.0.4-RELEASE</version>
             <scope>provided</scope>
-            <optional>true</optional>
-        </dependency>
-
-        <dependency>
-            <groupId>com.civious</groupId>
-            <artifactId>OBTeam</artifactId>
-            <version>1.1</version>
-            <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <dependency>
@@ -269,7 +218,6 @@
             <artifactId>Parties</artifactId>
             <version>3.1.14</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <dependency>
@@ -277,31 +225,13 @@
             <artifactId>mcMMO</artifactId>
             <version>2.1.209</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <dependency>
             <groupId>de.erethon</groupId>
             <artifactId>DungeonsXL</artifactId>
-            <version>0.18-SNAPSHOT-1149</version>
+            <version>0.18-PRE-02</version>
             <scope>provided</scope>
-            <optional>true</optional>
-        </dependency>
-
-        <dependency>
-            <groupId>com.herocraftonline</groupId>
-            <artifactId>Heroes</artifactId>
-            <version>1.9.22</version>
-            <scope>provided</scope>
-            <optional>true</optional>
-        </dependency>
-
-        <dependency>
-            <groupId>de.erethon</groupId>
-            <artifactId>Bedrock</artifactId>
-            <version>1.2.5</version>
-            <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <dependency>
@@ -309,7 +239,6 @@
             <artifactId>Dungeons</artifactId>
             <version>1.0</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <!-- Quest Plugins -->
@@ -318,45 +247,39 @@
             <artifactId>Quests</artifactId>
             <version>4.4.1-b340</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
         <dependency>
             <groupId>fr.skytasul.quests</groupId>
             <artifactId>BeautyQuests</artifactId>
             <version>0.19.5</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
         <dependency>
             <groupId>com.guillaumevdn</groupId>
             <artifactId>QuestCreator</artifactId>
             <version>6.39.0</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
         <dependency>
             <groupId>com.guillaumevdn</groupId>
             <artifactId>GCore</artifactId>
             <version>8.39.0</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <!-- Guild plugins -->
         <dependency>
-            <groupId>cc.javajobs</groupId>
+            <groupId>com.massivecraft</groupId>
-            <artifactId>FactionsBridge</artifactId>
+            <artifactId>Factions</artifactId>
-            <version>1.3.6</version>
+            <version>1.6.9.5-2.9.8-RC</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <dependency>
-            <groupId>com.github.UlrichBR</groupId>
+            <groupId>me.ulrich</groupId>
-            <artifactId>UClansV7-API</artifactId>
+            <artifactId>UltimateClans</artifactId>
-            <version>7.1.0</version>
+            <version>4.2.0</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <dependency>
@@ -364,15 +287,13 @@
             <artifactId>Guilds</artifactId>
             <version>3.5.6.0</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
         <dependency>
             <groupId>org.kingdoms.main</groupId>
             <artifactId>Kingdoms</artifactId>
-            <version>1.15.5</version>
+            <version>1.11.15.0.0.0.1.1</version>
             <scope>provided</scope>
-            <optional>true</optional>
         </dependency>
 
     </dependencies>

MMOCore-API/src/main/java/net/Indyuce/mmocore/MMOCore.java

@@ -2,15 +2,14 @@ package net.Indyuce.mmocore;
 
 import io.lumine.mythic.lib.MythicLib;
 import io.lumine.mythic.lib.UtilityMethods;
-import io.lumine.mythic.lib.data.sql.SQLDataSource;
-import io.lumine.mythic.lib.metrics.bukkit.Metrics;
-import io.lumine.mythic.lib.util.MMOPlugin;
 import io.lumine.mythic.lib.version.SpigotPlugin;
 import net.Indyuce.mmocore.api.player.PlayerData;
+import net.Indyuce.mmocore.manager.social.PartyManager;
+import net.Indyuce.mmocore.api.ConfigFile;
+import net.Indyuce.mmocore.manager.ActionBarManager;
 import net.Indyuce.mmocore.api.player.attribute.AttributeModifier;
 import net.Indyuce.mmocore.api.player.profess.resource.PlayerResource;
-import net.Indyuce.mmocore.command.MMOCoreCommandTreeRoot;
+import net.Indyuce.mmocore.command.*;
-import net.Indyuce.mmocore.command.api.ToggleableCommand;
 import net.Indyuce.mmocore.comp.citizens.CitizenInteractEventListener;
 import net.Indyuce.mmocore.comp.citizens.CitizensMMOLoader;
 import net.Indyuce.mmocore.comp.mythicmobs.MythicHook;
@@ -22,47 +21,45 @@ import net.Indyuce.mmocore.comp.region.DefaultRegionHandler;
 import net.Indyuce.mmocore.comp.region.RegionHandler;
 import net.Indyuce.mmocore.comp.region.WorldGuardMMOLoader;
 import net.Indyuce.mmocore.comp.region.WorldGuardRegionHandler;
-import net.Indyuce.mmocore.comp.region.pvpmode.PvPModeListener;
 import net.Indyuce.mmocore.comp.vault.VaultEconomy;
 import net.Indyuce.mmocore.comp.vault.VaultMMOLoader;
 import net.Indyuce.mmocore.guild.GuildModule;
 import net.Indyuce.mmocore.guild.GuildModuleType;
-import net.Indyuce.mmocore.guild.GuildRelationHandler;
 import net.Indyuce.mmocore.guild.provided.Guild;
 import net.Indyuce.mmocore.guild.provided.MMOCoreGuildModule;
-import net.Indyuce.mmocore.guild.provided.YAMLGuildDataManager;
 import net.Indyuce.mmocore.manager.*;
 import net.Indyuce.mmocore.manager.data.DataProvider;
-import net.Indyuce.mmocore.manager.data.GuildDataManager;
+import net.Indyuce.mmocore.manager.data.mysql.MySQLDataProvider;
-import net.Indyuce.mmocore.manager.data.LegacyDataProvider;
+import net.Indyuce.mmocore.manager.data.yaml.YAMLDataProvider;
-import net.Indyuce.mmocore.manager.data.PlayerDataManager;
-import net.Indyuce.mmocore.manager.data.sql.SQLDataHandler;
 import net.Indyuce.mmocore.manager.profession.*;
 import net.Indyuce.mmocore.manager.social.BoosterManager;
-import net.Indyuce.mmocore.manager.social.PartyManager;
 import net.Indyuce.mmocore.manager.social.RequestManager;
+import net.Indyuce.mmocore.party.MMOCoreTargetRestriction;
 import net.Indyuce.mmocore.party.PartyModule;
 import net.Indyuce.mmocore.party.PartyModuleType;
-import net.Indyuce.mmocore.party.PartyRelationHandler;
 import net.Indyuce.mmocore.party.provided.MMOCorePartyModule;
-import net.Indyuce.mmocore.script.mechanic.ExperienceMechanic;
-import net.Indyuce.mmocore.script.mechanic.ManaMechanic;
-import net.Indyuce.mmocore.script.mechanic.StaminaMechanic;
-import net.Indyuce.mmocore.script.mechanic.StelliumMechanic;
 import net.Indyuce.mmocore.skill.cast.SkillCastingMode;
-import net.Indyuce.mmocore.skill.trigger.MMOCoreTriggerType;
+import net.Indyuce.mmocore.script.mechanic.ExperienceMechanic;
+import net.Indyuce.mmocore.script.mechanic.StaminaMechanic;
+import net.Indyuce.mmocore.script.mechanic.ManaMechanic;
+import net.Indyuce.mmocore.script.mechanic.StelliumMechanic;
 import org.apache.commons.lang.Validate;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
-import org.bukkit.event.EventPriority;
+import io.lumine.mythic.lib.metrics.bukkit.Metrics;
+import org.bukkit.command.CommandMap;
+import org.bukkit.configuration.file.FileConfiguration;
+import org.bukkit.plugin.java.JavaPlugin;
 import org.bukkit.scheduler.BukkitRunnable;
 import org.jetbrains.annotations.NotNull;
 
 import java.io.File;
+import java.lang.reflect.Field;
 import java.util.logging.Level;
 
-public class MMOCore extends MMOPlugin {
+public class MMOCore extends JavaPlugin {
     public static MMOCore plugin;
+
     public final WaypointManager waypointManager = new WaypointManager();
     public final SoundManager soundManager = new SoundManager();
     public final RequestManager requestManager = new RequestManager();
@@ -82,10 +79,6 @@ public class MMOCore extends MMOPlugin {
     public final RestrictionManager restrictionManager = new RestrictionManager();
     public final SkillTreeManager skillTreeManager = new SkillTreeManager();
     public final StatManager statManager = new StatManager();
-    public final GuildDataManager nativeGuildManager = new YAMLGuildDataManager();
-    public final PlayerDataManager playerDataManager = new PlayerDataManager(this);
-    @Deprecated
-    public final DataProvider dataProvider = new LegacyDataProvider();
 
     // Profession managers
     public final CustomBlockManager mineManager = new CustomBlockManager();
@@ -99,11 +92,13 @@ public class MMOCore extends MMOPlugin {
     public VaultEconomy economy;
     public RegionHandler regionHandler = new DefaultRegionHandler();
     public PlaceholderParser placeholderParser = new DefaultParser();
+    public DataProvider dataProvider = new YAMLDataProvider();
 
     // Modules
     @NotNull
     public PartyModule partyModule;
     public GuildModule guildModule;
+    public boolean shouldDebugSQL = false;
 
     public MMOCore() {
         plugin = this;
@@ -111,13 +106,10 @@ public class MMOCore extends MMOPlugin {
 
     @Override
     public void onLoad() {
-        getLogger().log(Level.INFO, "Plugin file is called '" + getFile().getName() + "'");
 
         // Register MMOCore-specific objects
-        MythicLib.plugin.getEntities().registerRelationHandler(new PartyRelationHandler());
+        MythicLib.plugin.getEntities().registerRestriction(new MMOCoreTargetRestriction());
-        MythicLib.plugin.getEntities().registerRelationHandler(new GuildRelationHandler());
         MythicLib.plugin.getModifiers().registerModifierType("attribute", configObject -> new AttributeModifier(configObject));
-        MMOCoreTriggerType.registerAll();
 
         // Custom scripts
         MythicLib.plugin.getSkills().registerMechanic("mana", config -> new ManaMechanic(config));
@@ -151,13 +143,12 @@ public class MMOCore extends MMOPlugin {
             getLogger().warning("(Your config version: '" + configVersion + "' | Expected config version: '" + defConfigVersion + "')");
         }
 
-        if (getConfig().isConfigurationSection("mysql") && getConfig().getBoolean("mysql.enabled")) {
+        if (getConfig().isConfigurationSection("mysql") && getConfig().getBoolean("mysql.enabled"))
-            final SQLDataSource dataSource = new SQLDataSource(this);
+            dataProvider = new MySQLDataProvider(getConfig());
-            playerDataManager.setDataHandler(new SQLDataHandler(dataSource));
+        shouldDebugSQL = getConfig().getBoolean("mysql.debug");
-        }
 
         if (getConfig().isConfigurationSection("default-playerdata"))
-            playerDataManager.loadDefaultData(getConfig().getConfigurationSection("default-playerdata"));
+            dataProvider.getDataManager().loadDefaultData(getConfig().getConfigurationSection("default-playerdata"));
 
         if (Bukkit.getPluginManager().getPlugin("Vault") != null) economy = new VaultEconomy();
 
@@ -173,8 +164,6 @@ public class MMOCore extends MMOPlugin {
 
         if (Bukkit.getPluginManager().getPlugin("WorldGuard") != null) {
             regionHandler = new WorldGuardRegionHandler();
-            if (getConfig().getBoolean("pvp_mode.enabled"))
-                Bukkit.getPluginManager().registerEvents(new PvPModeListener(), this);
             getLogger().log(Level.INFO, "Hooked onto WorldGuard");
         }
 
@@ -184,8 +173,8 @@ public class MMOCore extends MMOPlugin {
         }
 
         /*
-         * Resource regeneration. Must check if entity is dead otherwise regen
+         * Resource regeneration. Must check if entity is dead otherwise regen will make
-         * will make the 'respawn' button glitched plus HURT entity effect bug
+         * the 'respawn' button glitched plus HURT entity effect bug
          */
         new BukkitRunnable() {
             public void run() {
@@ -207,6 +196,8 @@ public class MMOCore extends MMOPlugin {
         if (Bukkit.getPluginManager().getPlugin("MMOMana") != null) {
             getLogger().log(Level.SEVERE, ChatColor.DARK_RED + "MMOCore is not meant to be used with MMOItems ManaAndStamina");
             getLogger().log(Level.SEVERE, ChatColor.DARK_RED + "Please read the installation guide!");
+            Bukkit.broadcastMessage(ChatColor.DARK_RED + "[MMOCore] MMOCore is not meant to be used with MMOItems ManaAndStamina");
+            Bukkit.broadcastMessage(ChatColor.DARK_RED + "[MMOCore] Please read the installation guide!");
             return;
         }
 
@@ -216,10 +207,9 @@ public class MMOCore extends MMOPlugin {
         try {
             String partyPluginName = UtilityMethods.enumName(getConfig().getString("party-plugin"));
             PartyModuleType moduleType = PartyModuleType.valueOf(partyPluginName);
-            Validate.isTrue(moduleType.isValid(), "Plugin " + moduleType.getPluginName() + " is not installed");
+            Validate.isTrue(moduleType.isValid(), "Plugin '" + moduleType.name() + "' is not installed");
             partyModule = moduleType.provideModule();
-            getLogger().log(Level.INFO, "Hooked parties onto " + moduleType.getPluginName());
+        } catch (RuntimeException exception) {
-        } catch (Throwable exception) {
             getLogger().log(Level.WARNING, "Could not initialize party module: " + exception.getMessage());
             partyModule = new MMOCorePartyModule();
         }
@@ -230,7 +220,6 @@ public class MMOCore extends MMOPlugin {
             GuildModuleType moduleType = GuildModuleType.valueOf(pluginName);
             Validate.isTrue(moduleType.isValid(), "Plugin '" + moduleType.name() + "' is not installed");
             guildModule = moduleType.provideModule();
-            getLogger().log(Level.INFO, "Hooked guilds onto " + moduleType.getPluginName());
         } catch (RuntimeException exception) {
             getLogger().log(Level.WARNING, "Could not initialize guild module: " + exception.getMessage());
             guildModule = new MMOCoreGuildModule();
@@ -238,8 +227,8 @@ public class MMOCore extends MMOPlugin {
 
         // Skill casting
         try {
-            final SkillCastingMode mode = SkillCastingMode.valueOf(UtilityMethods.enumName(getConfig().getString("skill-casting.mode")));
+            SkillCastingMode mode = SkillCastingMode.valueOf(UtilityMethods.enumName(getConfig().getString("skill-casting.mode")));
-            mode.setCurrent(getConfig().getConfigurationSection("skill-casting"));
+            Bukkit.getPluginManager().registerEvents(mode.loadFromConfig(getConfig().getConfigurationSection("skill-casting")), this);
         } catch (RuntimeException exception) {
             getLogger().log(Level.WARNING, "Could not load skill casting: " + exception.getMessage());
         }
@@ -248,7 +237,6 @@ public class MMOCore extends MMOPlugin {
         try {
             Class.forName("net.Indyuce.mmocore.MMOCoreBukkit").getConstructor(MMOCore.class).newInstance(this);
         } catch (Throwable exception) {
-            exception.printStackTrace();
             throw new RuntimeException("Cannot run an API build on Spigot!");
         }
 
@@ -257,29 +245,95 @@ public class MMOCore extends MMOPlugin {
          * that after registering all the professses otherwise the player datas can't
          * recognize what profess the player has and professes will be lost
          */
-        playerDataManager.initialize(EventPriority.LOW, EventPriority.NORMAL);
+        Bukkit.getOnlinePlayers().forEach(player -> dataProvider.getDataManager().setup(player.getUniqueId()));
 
         // load guild data after loading player data
-        nativeGuildManager.load();
+        dataProvider.getGuildManager().load();
 
-        // Toggleable Commands
+        // Command
-        ToggleableCommand.register();
+        try {
+            final Field bukkitCommandMap = Bukkit.getServer().getClass().getDeclaredField("commandMap");
+
+            bukkitCommandMap.setAccessible(true);
+            CommandMap commandMap = (CommandMap) bukkitCommandMap.get(Bukkit.getServer());
+
+            FileConfiguration config = new ConfigFile("commands").getConfig();
+
+            if (config.contains("player"))
+                commandMap.register("mmocore", new PlayerStatsCommand(config.getConfigurationSection("player")));
+            if (config.contains("attributes"))
+                commandMap.register("mmocore", new AttributesCommand(config.getConfigurationSection("attributes")));
+            if (config.contains("class"))
+                commandMap.register("mmocore", new ClassCommand(config.getConfigurationSection("class")));
+            if (config.contains("waypoints"))
+                commandMap.register("mmocore", new WaypointsCommand(config.getConfigurationSection("waypoints")));
+            if (config.contains("quests"))
+                commandMap.register("mmocore", new QuestsCommand(config.getConfigurationSection("quests")));
+            if (config.contains("skills"))
+                commandMap.register("mmocore", new SkillsCommand(config.getConfigurationSection("skills")));
+            if (config.contains("friends"))
+                commandMap.register("mmocore", new FriendsCommand(config.getConfigurationSection("friends")));
+            if (config.contains("party"))
+                commandMap.register("mmocore", new PartyCommand(config.getConfigurationSection("party")));
+            if (config.contains("guild"))
+                commandMap.register("mmocore", new GuildCommand(config.getConfigurationSection("guild")));
+            if (config.contains("skill-trees"))
+                commandMap.register("mmocore", new SkillTreeCommand(config.getConfigurationSection("skill-trees")));
+            if (hasEconomy() && economy.isValid()) {
+                if (config.contains("withdraw"))
+                    commandMap.register("mmocore", new WithdrawCommand(config.getConfigurationSection("withdraw")));
+                if (config.contains("deposit"))
+                    commandMap.register("mmocore", new DepositCommand(config.getConfigurationSection("deposit")));
+            }
+        } catch (NoSuchFieldException | IllegalArgumentException | IllegalAccessException ex) {
+            ex.printStackTrace();
+        }
 
-        // Register MMOCore command what soever
         MMOCoreCommandTreeRoot mmoCoreCommand = new MMOCoreCommandTreeRoot();
         getCommand("mmocore").setExecutor(mmoCoreCommand);
         getCommand("mmocore").setTabCompleter(mmoCoreCommand);
+
+        if (getConfig().getBoolean("auto-save.enabled")) {
+            int autosave = getConfig().getInt("auto-save.interval") * 20;
+            new BukkitRunnable() {
+                public void run() {
+
+                    // Save player data
+                    for (PlayerData data : PlayerData.getAll())
+                        if (data.isFullyLoaded())
+                            dataProvider.getDataManager().saveData(data);
+
+                    // Save guild info
+                    for (Guild guild : dataProvider.getGuildManager().getAll())
+                        dataProvider.getGuildManager().save(guild);
+                }
+            }.runTaskTimerAsynchronously(MMOCore.plugin, autosave, autosave);
+        }
     }
 
     @Override
     public void onDisable() {
 
-        // Save guild info
+        //Executes all the pending asynchronous task (like saving the playerData)
-        for (Guild guild : nativeGuildManager.getAll())
+        Bukkit.getScheduler().getPendingTasks().forEach(worker -> {
-            nativeGuildManager.save(guild);
+            if (worker.getOwner().equals(this)) {
+                ((Runnable) worker).run();
+            }
+        });
+        // Save player data
+        for (PlayerData data : PlayerData.getAll())
+            if (data.isFullyLoaded()) {
+                data.close();
+                dataProvider.getDataManager().saveData(data);
+            }
 
-        // Close player data manager
+        // Save guild info
-        playerDataManager.close();
+        for (Guild guild : dataProvider.getGuildManager().getAll())
+            dataProvider.getGuildManager().save(guild);
+
+        // Close MySQL data provider (memory leaks)
+        if (dataProvider instanceof MySQLDataProvider)
+            ((MySQLDataProvider) dataProvider).close();
 
         // Reset active blocks
         mineManager.resetRemainingBlocks();
@@ -303,6 +357,8 @@ public class MMOCore extends MMOPlugin {
 
         configManager = new ConfigManager();
 
+        if (clearBefore)
+            MythicLib.plugin.getSkills().initialize(true);
         skillManager.initialize(clearBefore);
         mineManager.initialize(clearBefore);
         partyManager.initialize(clearBefore);
@@ -316,6 +372,7 @@ public class MMOCore extends MMOPlugin {
         statManager.initialize(clearBefore);
         professionManager.initialize(clearBefore);
 
+        InventoryManager.load();
         skillTreeManager.initialize(clearBefore);
         classManager.initialize(clearBefore);
         questManager.initialize(clearBefore);
@@ -325,14 +382,12 @@ public class MMOCore extends MMOPlugin {
         requestManager.initialize(clearBefore);
         soundManager.initialize(clearBefore);
         configItems.initialize(clearBefore);
-        //Needs to be loaded after the class manager.
-        InventoryManager.load();
 
         if (getConfig().isConfigurationSection("action-bar"))
             actionBarManager.reload(getConfig().getConfigurationSection("action-bar"));
 
         if (clearBefore)
-            PlayerData.getAll().forEach(PlayerData::reload);
+            PlayerData.getAll().forEach(PlayerData::update);
     }
 
     public static void log(String message) {
@@ -350,4 +405,9 @@ public class MMOCore extends MMOPlugin {
     public boolean hasEconomy() {
         return economy != null && economy.isValid();
     }
+
+    public static void sqlDebug(String s) {
+        if (!MMOCore.plugin.shouldDebugSQL) return;
+        MMOCore.plugin.getLogger().warning("- [SQL Debug] " + s);
+    }
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/ConfigMessage.java

@@ -2,143 +2,47 @@ package net.Indyuce.mmocore.api;
 
 import io.lumine.mythic.lib.MythicLib;
 import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.player.PlayerData;
+import org.bukkit.OfflinePlayer;
-import net.md_5.bungee.api.ChatMessageType;
+import org.bukkit.command.CommandSender;
-import net.md_5.bungee.api.chat.TextComponent;
-import org.apache.commons.lang.Validate;
 import org.bukkit.entity.Player;
-import org.jetbrains.annotations.NotNull;
-import org.jetbrains.annotations.Nullable;
 
-import java.util.ArrayList;
 import java.util.Collection;
 import java.util.List;
 
 public class ConfigMessage {
-    private final String key;
+    private final List<String> messages;
-    private final List<String> lines = new ArrayList<>();
-    private final boolean papiPlaceholders, actionbar, raw;
 
-    private ConfigMessage(@NotNull String key) {
+    public ConfigMessage(String key) {
-        this.key = key;
+        messages = MMOCore.plugin.configManager.getMessage(key);
-
-        // Initialize message list
-        final Object obj = MMOCore.plugin.configManager.getMessageObject(key);
-        if (obj == null) lines.add("<message_not_found:'" + key + "'>");
-        else if (obj instanceof List<?>) lines.addAll((List<String>) obj);
-        else {
-            final String tostr = obj.toString();
-            if (!tostr.isEmpty()) lines.add(tostr);
     }
 
-        // Does message include placeholders
+    public ConfigMessage addPlaceholders(String... placeholders) {
-        boolean hasPlaceholders = false;
+        for (int n = 0; n < messages.size(); n++) {
-        for (String str : lines)
+            String line = messages.get(n);
-            if (str.contains("%")) {
-                hasPlaceholders = true;
-                break;
-            }
-        this.papiPlaceholders = hasPlaceholders;
-
-        // Is message for action bar
-        actionbar = !lines.isEmpty() && lines.get(0).startsWith("%");
-        if (actionbar) lines.set(0, lines.get(0).substring(1));
-
-        // Are messages raw (JSON format)
-        raw = !lines.isEmpty() && (lines.get(0).startsWith("{") || lines.get(0).startsWith("["));
-    }
-
-    /**
-     * Useful for things like indicators or specific lore lines
-     * which are string tags not requiring more than one string
-     * object. An empty return value is accepted as some features
-     * do require the ability to fully remove text.
-     *
-     * @return First line of message, if it exists.
-     */
-    @NotNull
-    public String asLine() {
-        return lines.isEmpty() ? "" : lines.get(0);
-    }
-
-    @NotNull
-    public List<String> getLines() {
-        return lines;
-    }
-
-    @NotNull
-    public ConfigMessage addPlaceholders(@NotNull Object... placeholders) {
-
-        for (int n = 0; n < lines.size(); n++) {
-            String line = lines.get(n);
-
-            // Apply placeholders to string
             for (int j = 0; j < placeholders.length - 1; j += 2) {
-                final String placeholder = String.valueOf(placeholders[j]);
+                String placeholder = placeholders[j];
-                line = line.replace("{" + placeholder + "}", String.valueOf(placeholders[j + 1]));
+                if (line.contains("{" + placeholder + "}"))
+                    line = line.replace("{" + placeholder + "}", placeholders[j + 1]);
             }
-
+            messages.set(n, line);
-            lines.set(n, line);
         }
-
         return this;
     }
 
-    @Deprecated
+    public void send(CommandSender sender) {
-    public void sendAsJSon(Player player) {
+        messages.forEach(line -> sender.sendMessage(format(sender, line)));
-        send(player);
-    }
-
-    public void send(Player player) {
-        for (String line : lines) send(player, line);
     }
 
     public void send(Collection<? extends Player> players) {
-        for (Player player : players) for (String line : lines) send(player, line);
+        players.forEach(player -> messages.forEach(line -> player.sendMessage(format(player, line))));
     }
 
-    /**
+    public void sendAsJSon(Player player) {
-     * Sends a line of text to a target player
+        messages.forEach(line -> MythicLib.plugin.getVersion().getWrapper().sendJson(player, format(player, line)));
-     *
-     * @param player        Player to send message to. His player
-     *                      data is not necessarily fully loaded
-     * @param messageFormat Raw/normal message to send
-     */
-    private void send(@NotNull Player player, String messageFormat) {
-        Validate.notNull(player, "Player cannot be null");
-
-        final String rawMessage = format(player, messageFormat);
-        final PlayerData playerData = PlayerData.has(player) ? PlayerData.get(player) : null;
-
-        // Handle special case with player data + action bar
-        if (playerData != null && playerData.isOnline() && actionbar) {
-            playerData.displayActionBar(rawMessage, raw);
-            return;
     }
 
-        // Normal sender
+    private String format(CommandSender sender, String input) {
-        if (this.raw) {
+        String str = MythicLib.plugin.parseColors(input);
-            if (actionbar) MythicLib.plugin.getVersion().getWrapper().sendActionBarRaw(player, rawMessage);
+        return sender instanceof Player ? MMOCore.plugin.placeholderParser.parse((OfflinePlayer) sender, str) : str;
-            else MythicLib.plugin.getVersion().getWrapper().sendJson(player, rawMessage);
-        } else {
-            if (actionbar)
-                player.spigot().sendMessage(ChatMessageType.ACTION_BAR, TextComponent.fromLegacyText(rawMessage));
-            else player.sendMessage(rawMessage);
-        }
-    }
-
-    @NotNull
-    private String format(@Nullable Player player, String input) {
-        input = MythicLib.plugin.parseColors(input);
-        if (!papiPlaceholders || player == null) return input; // Optimization
-        return MMOCore.plugin.placeholderParser.parse(player, input);
-    }
-
-    @NotNull
-    public static ConfigMessage fromKey(@NotNull String key, Object... placeholders) {
-        Validate.notNull(MMOCore.plugin.configManager, "MMOCore has not finished enabling");
-        final ConfigMessage message = new ConfigMessage(key);
-        if (placeholders.length != 0) message.addPlaceholders(placeholders);
-        return message;
     }
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/MMOCoreAPI.java

@@ -25,7 +25,7 @@ public class MMOCoreAPI {
     }
 
     public PlayerData getPlayerData(OfflinePlayer player) {
-        return PlayerData.get(player);
+        return PlayerData.get(player.getUniqueId());
     }
 
     public boolean isInSameParty(Player player1, Player player2) {
@@ -65,7 +65,7 @@ public class MMOCoreAPI {
     public SkillResult cast(PlayerData playerData, ClassSkill skill) {
         PlayerMetadata casterMeta = playerData.getMMOPlayerData().getStatMap().cache(EquipmentSlot.MAIN_HAND);
         TriggerMetadata triggerMeta = new TriggerMetadata(casterMeta, null, null);
-        return new CastableSkill(skill, playerData).cast(triggerMeta);
+        return new CastableSkill(skill, playerData.getSkillLevel(skill.getSkill())).cast(triggerMeta);
     }
 
     /**

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/SoundEvent.java

@@ -6,14 +6,15 @@ public enum SoundEvent {
     WARP_CANCELLED,
     WARP_CHARGE,
     WARP_UNLOCK,
+    HOTBAR_SWAP,
     SPELL_CAST_BEGIN,
     SPELL_CAST_END,
     CANT_SELECT_CLASS,
     SELECT_CLASS,
     LEVEL_ATTRIBUTE,
     RESET_ATTRIBUTES,
-    RESET_SKILLS,
+    RESET_SKILLS
-    NOT_ENOUGH_POINTS,
+    ,NOT_ENOUGH_POINTS,
     CANCEL_QUEST,
     START_QUEST,
     CLOSE_LOOT_CHEST,

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/SoundObject.java

@@ -24,7 +24,6 @@ public class SoundObject {
     private final float volume;
     private final float pitch;
 
-    @Deprecated
     public SoundObject(String input) {
         String[] split = input.split(",");
 
@@ -37,14 +36,12 @@ public class SoundObject {
         }
 
         this.sound = sound;
-        this.key = key != null ? key.toLowerCase() : null;
+        this.key = key;
-
 
         volume = split.length > 1 ? Float.parseFloat(split[1]) : 1;
         pitch = split.length > 2 ? Float.parseFloat(split[2]) : 1;
     }
 
-    @Deprecated
     public SoundObject(ConfigurationSection config) {
         String input = config.getString("sound");
 
@@ -57,7 +54,7 @@ public class SoundObject {
         }
 
         this.sound = sound;
-        this.key = key != null ? key.toLowerCase() : null;
+        this.key = key;
 
         volume = (float) config.getDouble("volume", 1);
         pitch = (float) config.getDouble("pitch", 1);
@@ -67,39 +64,32 @@ public class SoundObject {
      * @return If this object is custom a custom sound, potentially
      *         from a resource pack
      */
-    @Deprecated
     public boolean isCustom() {
         return sound == null;
     }
 
     @Nullable
-    @Deprecated
     public Sound getSound() {
         return sound;
     }
 
     @Nullable
-    @Deprecated
     public String getKey() {
         return key;
     }
 
-    @Deprecated
     public float getVolume() {
         return volume;
     }
 
-    @Deprecated
     public float getPitch() {
         return pitch;
     }
 
-    @Deprecated
     public void playTo(Player player) {
         playTo(player, volume, pitch);
     }
 
-    @Deprecated
     public void playTo(Player player, float volume, float pitch) {
         if (isCustom())
             player.playSound(player.getLocation(), key, volume, pitch);
@@ -107,12 +97,10 @@ public class SoundObject {
             player.playSound(player.getLocation(), sound, volume, pitch);
     }
 
-    @Deprecated
     public void playAt(Location loc) {
         playAt(loc, volume, pitch);
     }
 
-    @Deprecated
     public void playAt(Location loc, float volume, float pitch) {
         if (isCustom())
             loc.getWorld().playSound(loc, key, volume, pitch);

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/block/BlockInfo.java

@@ -43,7 +43,7 @@ public class BlockInfo {
 					options.put(option, config.getBoolean("options." + key));
 				} catch (IllegalArgumentException exception) {
 					MMOCore.plugin.getLogger().log(Level.WARNING,
-							"Could not load option '" + key + "' from block info '" + block.display() + "': " + exception.getMessage());
+							"Could not load option '" + key + "' from block info '" + block.generateKey() + "': " + exception.getMessage());
 				}
 
 		if (config.contains("triggers")) {
@@ -55,7 +55,7 @@ public class BlockInfo {
 					triggers.add(MMOCore.plugin.loadManager.loadTrigger(new MMOLineConfig(key)));
 				} catch (IllegalArgumentException exception) {
 					MMOCore.plugin.getLogger().log(Level.WARNING,
-							"Could not load trigger '" + key + "' from block info '" + block.display() + "': " + exception.getMessage());
+							"Could not load trigger '" + key + "' from block info '" + block.generateKey() + "': " + exception.getMessage());
 				}
 		}
 
@@ -65,13 +65,13 @@ public class BlockInfo {
 				if (condition instanceof BlockCondition)
 					conditions.add((BlockCondition) condition);
 			}
+
 	}
 
 	public boolean getOption(BlockInfoOption option) {
 		return options.getOrDefault(option, option.getDefault());
 	}
 
-	@NotNull
 	public BlockType getBlock() {
 		return block;
 	}
@@ -85,7 +85,6 @@ public class BlockInfo {
 		return table != null;
 	}
 
-	@Deprecated
 	public List<ItemStack> collectDrops(LootBuilder builder) {
 		return table != null ? table.collect(builder) : new ArrayList<>();
 	}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/block/BlockType.java

@@ -1,31 +1,29 @@
 package net.Indyuce.mmocore.api.block;
 
 import org.bukkit.block.Block;
-import org.jetbrains.annotations.NotNull;
-import org.jetbrains.annotations.Nullable;
 
 public interface BlockType {
 
 	/**
 	 * Called when placing temporary blocks
 	 */
-    void place(@NotNull BlockInfo.RegeneratingBlock placed);
+	void place(BlockInfo.RegeneratingBlock placed);
 
 	/**
 	 * Called when regenerating an older block with block regen
 	 */
-    void regenerate(@NotNull BlockInfo.RegeneratingBlock regenerating);
+	void regenerate(BlockInfo.RegeneratingBlock regenerating);
 
-    @NotNull String display();
+	/**
+	 * Generates a key used to store the BlockInfo instance in the manager map,
+	 * the key depends on the block type to make sure there is no interference
+	 */
+	String generateKey();
 
 	/**
 	 * Applies some extra break restrictions; returns TRUE if the block can be
 	 * broken. This method is used to prevent non mature crops from being broken
 	 * for example
 	 */
-    boolean breakRestrictions(@NotNull Block block);
+	boolean breakRestrictions(Block block);
-
-    int hashCode();
-
-    boolean equals(@Nullable Object obj);
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/block/SkullBlockType.java

@@ -1,14 +1,13 @@
 package net.Indyuce.mmocore.api.block;
 
-import io.lumine.mythic.lib.MythicLib;
-import io.lumine.mythic.lib.api.MMOLineConfig;
-import net.Indyuce.mmocore.api.block.BlockInfo.RegeneratingBlock;
 import net.Indyuce.mmocore.api.util.MMOCoreUtils;
 import org.bukkit.Location;
-import org.bukkit.Material;
 import org.bukkit.block.Block;
 
-import java.util.Objects;
+import net.Indyuce.mmocore.api.block.BlockInfo.RegeneratingBlock;
+import io.lumine.mythic.lib.MythicLib;
+import io.lumine.mythic.lib.api.MMOLineConfig;
+import io.lumine.mythic.lib.version.VersionMaterial;
 
 public class SkullBlockType implements BlockType {
 	private final String value;
@@ -30,7 +29,7 @@ public class SkullBlockType implements BlockType {
 	@Override
 	public void place(RegeneratingBlock block) {
 		Location loc = block.getLocation();
-        loc.getBlock().setType(Material.PLAYER_HEAD);
+		loc.getBlock().setType(VersionMaterial.PLAYER_HEAD.toMaterial());
 
 		// save skull orientation if replaced block is a player head
 		if (MMOCoreUtils.isPlayerHead(block.getBlockData().getMaterial()))
@@ -49,25 +48,12 @@ public class SkullBlockType implements BlockType {
 	}
 
 	@Override
-    public String display() {
+	public String generateKey() {
-        return "Skull{" + value + "}";
+		return "vanilla-skull-" + value;
 	}
 
 	@Override
 	public boolean breakRestrictions(Block block) {
 		return true;
 	}
-
-    @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        SkullBlockType that = (SkullBlockType) o;
-        return Objects.equals(value, that.value);
-    }
-
-    @Override
-    public int hashCode() {
-        return Objects.hash(value);
-    }
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/block/VanillaBlockType.java

@@ -1,7 +1,5 @@
 package net.Indyuce.mmocore.api.block;
 
-import io.lumine.mythic.lib.api.MMOLineConfig;
-import net.Indyuce.mmocore.api.block.BlockInfo.RegeneratingBlock;
 import org.apache.commons.lang.Validate;
 import org.bukkit.Location;
 import org.bukkit.Material;
@@ -9,7 +7,8 @@ import org.bukkit.block.Block;
 import org.bukkit.block.data.Ageable;
 import org.bukkit.block.data.BlockData;
 
-import java.util.Objects;
+import net.Indyuce.mmocore.api.block.BlockInfo.RegeneratingBlock;
+import io.lumine.mythic.lib.api.MMOLineConfig;
 
 public class VanillaBlockType implements BlockType {
 	private final Material type;
@@ -59,25 +58,12 @@ public class VanillaBlockType implements BlockType {
 	}
 
 	@Override
-    public String display() {
+	public String generateKey() {
-        return "Vanilla{" + type.name() + "}";
+		return "vanilla-block-" + type.name();
 	}
 
 	@Override
 	public boolean breakRestrictions(Block block) {
 		return age == 0 || (block.getBlockData() instanceof Ageable && ((Ageable) block.getBlockData()).getAge() >= age);
 	}
-
-    @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        VanillaBlockType that = (VanillaBlockType) o;
-        return type == that.type;
-    }
-
-    @Override
-    public int hashCode() {
-        return Objects.hash(type);
-    }
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/eco/Withdraw.java

@@ -5,7 +5,6 @@ import java.util.Set;
 import java.util.UUID;
 
 import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.ConfigMessage;
 import net.Indyuce.mmocore.util.item.CurrencyItemBuilder;
 import org.bukkit.Bukkit;
 import org.bukkit.Sound;
@@ -39,7 +38,7 @@ public class Withdraw implements Listener {
 			return;
 
 		withdrawing.add(player.getUniqueId());
-		ConfigMessage.fromKey("withdrawing").send(player);
+		MMOCore.plugin.configManager.getSimpleMessage("withdrawing").send(player);
 		Bukkit.getPluginManager().registerEvents(this, MMOCore.plugin);
 		Bukkit.getScheduler().runTaskLater(MMOCore.plugin, this::close, 20 * 20);
 	}
@@ -61,7 +60,7 @@ public class Withdraw implements Listener {
 		if (!event.getPlayer().equals(player))
 			return;
 
-		ConfigMessage.fromKey("withdraw-cancel").send(player);
+		MMOCore.plugin.configManager.getSimpleMessage("withdraw-cancel").send(player);
 		close();
 	}
 
@@ -76,13 +75,13 @@ public class Withdraw implements Listener {
 		try {
 			worth = Integer.parseInt(event.getMessage());
 		} catch (Exception e) {
-			ConfigMessage.fromKey("wrong-number").addPlaceholders("arg", event.getMessage()).send(player);
+			MMOCore.plugin.configManager.getSimpleMessage("wrong-number", "arg", event.getMessage()).send(player);
 			return;
 		}
 
 		int left = (int) (MMOCore.plugin.economy.getEconomy().getBalance(player) - worth);
 		if (left < 0) {
-			ConfigMessage.fromKey("not-enough-money").addPlaceholders("left", -left).send(player);
+			MMOCore.plugin.configManager.getSimpleMessage("not-enough-money", "left", "" + -left).send(player);
 			return;
 		}
 
@@ -92,7 +91,7 @@ public class Withdraw implements Listener {
 			MMOCore.plugin.economy.getEconomy().withdrawPlayer(player, worth);
 			withdrawAlgorythm(worth);
 			player.playSound(player.getLocation(), Sound.ENTITY_PLAYER_LEVELUP, 1, 1);
-			ConfigMessage.fromKey("withdrew").addPlaceholders("worth", worth).send(player);
+			MMOCore.plugin.configManager.getSimpleMessage("withdrew", "worth", "" + worth).send(player);
 		});
 	}
 

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/event/AsyncPlayerDataEvent.java

@@ -0,0 +1,29 @@
+package net.Indyuce.mmocore.api.event;
+
+import net.Indyuce.mmocore.api.player.PlayerData;
+import org.bukkit.entity.Player;
+import org.bukkit.event.Event;
+
+/**
+ * In order to create bukkit async events we must call
+ * the right constructor:
+ * <p>
+ * {@link Event#Event(boolean)} and have the boolean set to true
+ */
+public abstract class AsyncPlayerDataEvent extends Event {
+	private final PlayerData playerData;
+
+	public AsyncPlayerDataEvent(PlayerData playerData) {
+		super(true);
+
+		this.playerData = playerData;
+	}
+
+	public PlayerData getData() {
+		return playerData;
+	}
+
+	public Player getPlayer() {
+		return playerData.getPlayer();
+	}
+}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/event/AsyncPlayerDataLoadEvent.java

@@ -0,0 +1,27 @@
+package net.Indyuce.mmocore.api.event;
+
+import net.Indyuce.mmocore.api.player.PlayerData;
+import org.bukkit.event.HandlerList;
+
+public class AsyncPlayerDataLoadEvent extends AsyncPlayerDataEvent {
+    private static final HandlerList handlers = new HandlerList();
+
+    /**
+     * Called when a player data is being loaded into the game.
+     * This event is called async.
+     *
+     * @param playerData Player data being loaded
+     */
+    public AsyncPlayerDataLoadEvent(PlayerData playerData) {
+        super(playerData);
+    }
+
+    @Override
+    public HandlerList getHandlers() {
+        return handlers;
+    }
+
+    public static HandlerList getHandlerList() {
+        return handlers;
+    }
+}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/event/PlayerDataLoadEvent.java

@@ -4,7 +4,7 @@ import net.Indyuce.mmocore.api.player.PlayerData;
 import org.bukkit.event.HandlerList;
 
 /**
- * @deprecated Use {@link io.lumine.mythic.lib.api.event.SynchronizedDataLoadEvent} instead
+ * @deprecated Use {@link AsyncPlayerDataLoadEvent} instead
  */
 @Deprecated
 public class PlayerDataLoadEvent extends PlayerDataEvent {

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/event/PlayerEnterCastingModeEvent.java

@@ -1,42 +0,0 @@
-package net.Indyuce.mmocore.api.event;
-
-import net.Indyuce.mmocore.api.player.PlayerData;
-import org.bukkit.entity.Player;
-import org.bukkit.event.Cancellable;
-import org.bukkit.event.HandlerList;
-import org.jetbrains.annotations.NotNull;
-
-public class PlayerEnterCastingModeEvent extends PlayerDataEvent implements Cancellable {
-    private boolean cancelled = false;
-
-    private static final HandlerList HANDLERS = new HandlerList();
-
-    @Deprecated
-    public PlayerEnterCastingModeEvent(@NotNull Player who) {
-        super(PlayerData.get(who));
-    }
-
-    public PlayerEnterCastingModeEvent(@NotNull PlayerData playerData) {
-        super(playerData);
-    }
-
-    @Override
-    public boolean isCancelled() {
-        return cancelled;
-    }
-
-    @Override
-    public void setCancelled(boolean cancelled) {
-        this.cancelled = cancelled;
-    }
-
-    @NotNull
-    @Override
-    public HandlerList getHandlers() {
-        return getHandlerList();
-    }
-
-    public static HandlerList getHandlerList() {
-        return HANDLERS;
-    }
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/event/PlayerExitCastingModeEvent.java

@@ -1,43 +0,0 @@
-package net.Indyuce.mmocore.api.event;
-
-
-import net.Indyuce.mmocore.api.player.PlayerData;
-import org.bukkit.entity.Player;
-import org.bukkit.event.Cancellable;
-import org.bukkit.event.HandlerList;
-import org.jetbrains.annotations.NotNull;
-
-public class PlayerExitCastingModeEvent extends PlayerDataEvent implements Cancellable {
-    private boolean cancelled = false;
-
-    private static final HandlerList HANDLERS = new HandlerList();
-
-    @Deprecated
-    public PlayerExitCastingModeEvent(@NotNull Player who) {
-        super(PlayerData.get(who));
-    }
-
-    public PlayerExitCastingModeEvent(@NotNull PlayerData who) {
-        super(who);
-    }
-
-    @Override
-    public boolean isCancelled() {
-        return cancelled;
-    }
-
-    @Override
-    public void setCancelled(boolean cancelled) {
-        this.cancelled = cancelled;
-    }
-
-    @NotNull
-    @Override
-    public HandlerList getHandlers() {
-        return HANDLERS;
-    }
-
-    public static HandlerList getHandlerList() {
-        return HANDLERS;
-    }
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/event/PlayerResourceUpdateEvent.java

@@ -56,10 +56,6 @@ public class PlayerResourceUpdateEvent extends PlayerDataEvent implements Cancel
 		return amount;
 	}
 
-	public UpdateReason getReason() {
-		return reason;
-	}
-
 	/**
 	 * Changes the amount of resource given/taken away
 	 *

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/event/unlocking/ItemChangeEvent.java

@@ -1,36 +0,0 @@
-package net.Indyuce.mmocore.api.event.unlocking;
-
-import net.Indyuce.mmocore.api.event.PlayerDataEvent;
-import net.Indyuce.mmocore.api.player.PlayerData;
-
-public abstract class ItemChangeEvent extends PlayerDataEvent {
-    private final String itemKey;
-
-    public ItemChangeEvent(PlayerData playerData, String itemKey) {
-        super(playerData);
-        this.itemKey = itemKey;
-    }
-
-    /**
-     * @return The full item key in the format <plugin-id>:<item-type-id>:<item-id>.
-     */
-    public String getItemKey() {
-        return itemKey;
-    }
-
-
-    /**
-     * @return The item-type-id which is the first parameter in the key format <item-type-id>:<item-id>.
-     */
-    public String getItemTypeId() {
-        return itemKey.split(":")[0];
-    }
-
-    /**
-     * @return The item--id which is the last parameter in the key format <item-type-id>:<item-id>.
-     */
-    public String getItemId() {
-        return itemKey.split(":")[1];
-    }
-}
-

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/event/unlocking/ItemLockedEvent.java

@@ -1,24 +0,0 @@
-package net.Indyuce.mmocore.api.event.unlocking;
-
-import net.Indyuce.mmocore.api.player.PlayerData;
-import org.bukkit.event.HandlerList;
-import org.jetbrains.annotations.NotNull;
-
-public class ItemLockedEvent extends ItemChangeEvent {
-    private static final HandlerList handlers = new HandlerList();
-
-    public ItemLockedEvent(PlayerData playerData, String itemKey) {
-        super(playerData, itemKey);
-    }
-
-    @NotNull
-    @Override
-    public HandlerList getHandlers() {
-        return handlers;
-    }
-
-    public static HandlerList getHandlerList() {
-        return handlers;
-    }
-}
-

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/event/unlocking/ItemUnlockedEvent.java

@@ -1,22 +0,0 @@
-package net.Indyuce.mmocore.api.event.unlocking;
-import net.Indyuce.mmocore.api.player.PlayerData;
-import org.bukkit.event.HandlerList;
-import org.jetbrains.annotations.NotNull;
-
-public class ItemUnlockedEvent extends ItemChangeEvent {
-    private static final HandlerList handlers = new HandlerList();
-
-    public ItemUnlockedEvent(PlayerData playerData, String itemKey) {
-        super(playerData, itemKey);
-    }
-
-    @NotNull
-    @Override
-    public HandlerList getHandlers() {
-        return handlers;
-    }
-
-    public static HandlerList getHandlerList() {
-        return handlers;
-    }
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/load/DefaultMMOLoader.java

@@ -19,28 +19,10 @@ public class DefaultMMOLoader extends MMOLoader {
     public Trigger loadTrigger(MMOLineConfig config) {
         if (config.getKey().equals("from"))
             return new FromTrigger(config);
-
         if (config.getKey().equals("stat"))
             return new StatTrigger(config);
-
-        if(config.getKey().equals("unlock_slot"))
-            return new UnlockSlotTrigger(config);
-
-        if (config.getKey().equals("unlock_skill"))
-            return new UnlockSkillTrigger(config);
-
-        if (config.getKey().equals("bind_skill"))
-            return new BindSkillTrigger(config);
-
-        if (config.getKey().equals("levelup_skill"))
-            return new LevelUpSkillTrigger(config);
-
-        if (config.getKey().equals("skill_buff")||config.getKey().equals("skill_modifier"))
-            return new SkillModifierTrigger(config);
-
         if (config.getKey().equals("message"))
             return new MessageTrigger(config);
-
         if (config.getKey().equals("sound") || config.getKey().equals("playsound"))
             return new SoundTrigger(config);
 
@@ -119,12 +101,6 @@ public class DefaultMMOLoader extends MMOLoader {
         if (config.getKey().equals("permission"))
             return new PermissionCondition(config);
 
-        if (config.getKey().equals("weather"))
-            return new WeatherCondition(config);
-
-        if (config.getKey().equals("time"))
-            return new TimeCondition(config);
-
         return null;
     }
 

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/player/CombatRunnable.java

@@ -0,0 +1,49 @@
+package net.Indyuce.mmocore.api.player;
+
+import net.Indyuce.mmocore.MMOCore;
+import net.Indyuce.mmocore.api.event.PlayerCombatEvent;
+import org.apache.commons.lang.Validate;
+import org.bukkit.Bukkit;
+import org.bukkit.scheduler.BukkitRunnable;
+
+public class CombatRunnable extends BukkitRunnable {
+    private final PlayerData player;
+
+    private long lastHit = System.currentTimeMillis();
+
+    private boolean open = true;
+
+    public CombatRunnable(PlayerData player) {
+        this.player = player;
+
+        if (player.isOnline()) {
+            MMOCore.plugin.configManager.getSimpleMessage("now-in-combat").send(player.getPlayer());
+            Bukkit.getPluginManager().callEvent(new PlayerCombatEvent(player, true));
+            runTaskTimer(MMOCore.plugin, 20, 20);
+        }
+    }
+
+    public void update() {
+        lastHit = System.currentTimeMillis();
+    }
+
+    @Override
+    public void run() {
+        if (!player.isOnline()) {
+            close();
+            return;
+        }
+        if (lastHit + MMOCore.plugin.configManager.combatLogTimer < System.currentTimeMillis()) {
+            Bukkit.getPluginManager().callEvent(new PlayerCombatEvent(player, false));
+            MMOCore.plugin.configManager.getSimpleMessage("leave-combat").send(player.getPlayer());
+            close();
+        }
+    }
+
+    private void close() {
+        Validate.isTrue(open, "Combat runnable has already been closed");
+        player.combat = null;
+        cancel();
+        open = false;
+    }
+}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/player/OfflinePlayerData.java

@@ -0,0 +1,32 @@
+package net.Indyuce.mmocore.api.player;
+
+import java.util.UUID;
+
+import net.Indyuce.mmocore.MMOCore;
+import net.Indyuce.mmocore.api.player.profess.PlayerClass;
+
+public abstract class OfflinePlayerData {
+	private final UUID uuid;
+
+	public OfflinePlayerData(UUID uuid) {
+		this.uuid = uuid;
+	}
+
+	public UUID getUniqueId() {
+		return uuid;
+	}
+
+	public abstract void removeFriend(UUID uuid);
+
+	public abstract boolean hasFriend(UUID uuid);
+
+	public abstract PlayerClass getProfess();
+
+	public abstract int getLevel();
+
+	public abstract long getLastLogin();
+
+	public static OfflinePlayerData get(UUID uuid) {
+		return MMOCore.plugin.dataProvider.getDataManager().getOffline(uuid);
+	}
+}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/player/PlayerActivity.java

@@ -6,11 +6,11 @@ import javax.inject.Provider;
 import java.util.Objects;
 
 /**
- * Used by MMOCore when it has to store the last time a player
+ * Used by MMOCore when it has to store the last time
- * did some action. This also features a time out function which
+ * a player did some action.
- * can be used for cooldowns.
+ * <p>
- *
+ * This also features a time out function which can
- * @deprecated Merge with {@link io.lumine.mythic.lib.player.cooldown.CooldownMap}
+ * be used for cooldowns
  */
 public enum PlayerActivity {
     USE_WAYPOINT(() -> 5 * 1000L),
@@ -21,9 +21,7 @@ public enum PlayerActivity {
 
     LOOT_CHEST_SPAWN(() -> MMOCore.plugin.configManager.lootChestPlayerCooldown),
 
-    CAST_SKILL(() -> MMOCore.plugin.configManager.globalSkillCooldown),
+    CAST_SKILL(() -> MMOCore.plugin.configManager.globalSkillCooldown);
-
-    ;
 
     private final Provider<Long> timeout;
 

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/player/attribute/AttributeModifier.java

@@ -1,30 +1,36 @@
 package net.Indyuce.mmocore.api.player.attribute;
 
+import io.lumine.mythic.lib.MythicLib;
 import io.lumine.mythic.lib.api.player.EquipmentSlot;
 import io.lumine.mythic.lib.api.player.MMOPlayerData;
-import io.lumine.mythic.lib.api.stat.api.InstanceModifier;
+import io.lumine.mythic.lib.api.stat.modifier.StatModifier;
 import io.lumine.mythic.lib.player.modifier.ModifierSource;
 import io.lumine.mythic.lib.player.modifier.ModifierType;
+import io.lumine.mythic.lib.player.modifier.PlayerModifier;
 import io.lumine.mythic.lib.util.configobject.ConfigObject;
 import net.Indyuce.mmocore.api.player.PlayerData;
+import org.apache.commons.lang.Validate;
 
-import java.util.UUID;
+import java.text.DecimalFormat;
+import java.util.Objects;
 
-public class AttributeModifier extends InstanceModifier {
+public class AttributeModifier extends PlayerModifier {
     private final String attribute;
+    private final double value;
+    private final ModifierType type;
+
+    private static final DecimalFormat oneDigit = MythicLib.plugin.getMMOConfig().newDecimalFormat("0.#");
 
     /**
      * Flat attribute modifier (simplest modifier you can think about)
      */
     public AttributeModifier(String key, String attribute, double value) {
-        super(key, value);
+        this(key, attribute, value, ModifierType.FLAT, EquipmentSlot.OTHER, ModifierSource.OTHER);
-
-        this.attribute = attribute;
     }
 
     /**
-     * Attribute modifier given by an external mechanic, like a party buff, item
+     * Attribute modifier given by an external mecanic, like a party buff, item set bonuses,
-     * set bonuses, skills or abilities... Anything apart from items and armor.
+     * skills or abilities... Anything apart from items and armor.
      */
     public AttributeModifier(String key, String attribute, double value, ModifierType type) {
         this(key, attribute, value, type, EquipmentSlot.OTHER, ModifierSource.OTHER);
@@ -41,23 +47,11 @@ public class AttributeModifier extends InstanceModifier {
      * @param source    Type of the item granting the stat modifier
      */
     public AttributeModifier(String key, String attribute, double value, ModifierType type, EquipmentSlot slot, ModifierSource source) {
-        this(UUID.randomUUID(), key, attribute, value, type, slot, source);
+        super(key, slot, source);
-    }
-
-    /**
-     * Attribute modifier given by an item, either a weapon or an armor piece.
-     *
-     * @param key       Player modifier key
-     * @param attribute Attribute being modified
-     * @param value     Value of stat modifier
-     * @param type      Is the modifier flat or multiplicative
-     * @param slot      Slot of the item granting the stat modifier
-     * @param source    Type of the item granting the stat modifier
-     */
-    public AttributeModifier(UUID uniqueId, String key, String attribute, double value, ModifierType type, EquipmentSlot slot, ModifierSource source) {
-        super(uniqueId, key, slot, source, value, type);
 
         this.attribute = attribute;
+        this.value = value;
+        this.type = type;
     }
 
     /**
@@ -68,14 +62,22 @@ public class AttributeModifier extends InstanceModifier {
      * @param str The string to be parsed
      */
     public AttributeModifier(String key, String attribute, String str) {
-        super(key, EquipmentSlot.OTHER, ModifierSource.OTHER, str);
+        super(key, EquipmentSlot.OTHER, ModifierSource.OTHER);
 
+        Validate.notNull(str, "String cannot be null");
+        Validate.notEmpty(str, "String cannot be empty");
+
+        type = str.toCharArray()[str.length() - 1] == '%' ? ModifierType.RELATIVE : ModifierType.FLAT;
+        value = Double.parseDouble(type == ModifierType.RELATIVE ? str.substring(0, str.length() - 1) : str);
         this.attribute = attribute;
     }
 
     public AttributeModifier(ConfigObject object) {
-        super(object);
+        super(object.getString("key"), EquipmentSlot.OTHER, ModifierSource.OTHER);
 
+        String str = Objects.requireNonNull(object.getString("value"));
+        type = str.toCharArray()[str.length() - 1] == '%' ? ModifierType.RELATIVE : ModifierType.FLAT;
+        value = Double.parseDouble(type == ModifierType.RELATIVE ? str.substring(0, str.length() - 1) : str);
         this.attribute = object.getString("attribute");
     }
 
@@ -83,6 +85,14 @@ public class AttributeModifier extends InstanceModifier {
         return attribute;
     }
 
+    public ModifierType getType() {
+        return type;
+    }
+
+    public double getValue() {
+        return value;
+    }
+
     /**
      * Used to multiply some existing stat modifier by a constant, usually an
      * integer, for instance when MMOCore party modifiers scale with the
@@ -91,19 +101,24 @@ public class AttributeModifier extends InstanceModifier {
      * @param coef The multiplicative constant
      * @return A new instance of StatModifier with modified value
      */
-    public AttributeModifier multiply(double coef) {
+    public StatModifier multiply(double coef) {
-        return new AttributeModifier(getUniqueId(), getKey(), attribute, value * coef, type, getSlot(), getSource());
+        return new StatModifier(getKey(), attribute, value * coef, type, getSlot(), getSource());
     }
 
     @Override
     public void register(MMOPlayerData mmoPlayerData) {
-        PlayerData playerData = PlayerData.get(mmoPlayerData);
+        PlayerData playerData = PlayerData.get(mmoPlayerData.getUniqueId());
         playerData.getAttributes().getInstance(attribute).addModifier(this);
     }
 
     @Override
     public void unregister(MMOPlayerData mmoPlayerData) {
-        PlayerData playerData = PlayerData.get(mmoPlayerData);
+        PlayerData playerData = PlayerData.get(mmoPlayerData.getUniqueId());
         playerData.getAttributes().getInstance(attribute).removeModifier(getKey());
     }
+
+    @Override
+    public String toString() {
+        return oneDigit.format(value) + (type == io.lumine.mythic.lib.player.modifier.ModifierType.RELATIVE ? "%" : "");
+    }
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/player/attribute/MMOCoreAttributeStatHandler.java

@@ -1,10 +1,9 @@
 package net.Indyuce.mmocore.api.player.attribute;
 
-import io.lumine.mythic.lib.api.stat.StatInstance;
+import io.lumine.mythic.lib.api.stat.StatMap;
 import io.lumine.mythic.lib.api.stat.handler.StatHandler;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
-import org.bukkit.configuration.ConfigurationSection;
 
 /**
  * This fixes an issue where registering new stat modifiers in ML
@@ -14,13 +13,17 @@ import org.bukkit.configuration.ConfigurationSection;
  * This stat handler MAY call subsequent stat handlers. There might
  * be infinite recursion problems if another attr. grants extra attribute pts.
  */
-public class MMOCoreAttributeStatHandler extends StatHandler {
+public class MMOCoreAttributeStatHandler implements StatHandler {
     private final PlayerAttribute attr;
+    private final String statName;
 
-    public MMOCoreAttributeStatHandler(ConfigurationSection config, PlayerAttribute attr) {
+    public MMOCoreAttributeStatHandler(PlayerAttribute attr) {
-        super(config, "ADDITIONAL_" + attr.getId().toUpperCase().replace("-", "_"));
-
         this.attr = attr;
+        this.statName = "ADDITIONAL_" + attr.getId().toUpperCase().replace("-", "_");
+    }
+
+    public String getStat() {
+        return statName;
     }
 
     /**
@@ -28,12 +31,22 @@ public class MMOCoreAttributeStatHandler extends StatHandler {
      * is not loaded yet, hence the try/catch clause
      */
     @Override
-    public void runUpdate(StatInstance instance) {
+    public void runUpdate(StatMap statMap) {
         try {
-            final PlayerData playerData = PlayerData.get(instance.getMap().getPlayerData());
+            final PlayerData playerData = MMOCore.plugin.dataProvider.getDataManager().get(statMap.getPlayerData().getUniqueId());
             playerData.getAttributes().getInstance(attr).updateStats();
         } catch (NullPointerException exception) {
             // Player data is not loaded yet so there's nothing to update.
         }
     }
+
+    @Override
+    public double getBaseValue(StatMap statMap) {
+        return 0;
+    }
+
+    @Override
+    public double getTotalValue(StatMap statMap) {
+        return statMap.getStat(statName);
+    }
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/player/attribute/PlayerAttribute.java

@@ -85,7 +85,7 @@ public class PlayerAttribute implements ExperienceObject {
     @NotNull
     @Override
     public ExperienceTable getExperienceTable() {
-        return Objects.requireNonNull(expTable, "Attribute has no exp table");
+        return Objects.requireNonNull(expTable);
     }
 
     @Override

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/player/attribute/PlayerAttributes.java

@@ -1,14 +1,13 @@
 package net.Indyuce.mmocore.api.player.attribute;
 
-import io.lumine.mythic.lib.MythicLib;
+import com.google.gson.Gson;
+import com.google.gson.JsonElement;
+import com.google.gson.JsonObject;
 import io.lumine.mythic.lib.UtilityMethods;
 import io.lumine.mythic.lib.api.player.EquipmentSlot;
-import io.lumine.mythic.lib.api.stat.StatInstance;
+import io.lumine.mythic.lib.player.modifier.Closeable;
-import io.lumine.mythic.lib.gson.JsonElement;
-import io.lumine.mythic.lib.gson.JsonObject;
 import io.lumine.mythic.lib.player.modifier.ModifierSource;
 import io.lumine.mythic.lib.player.modifier.ModifierType;
-import io.lumine.mythic.lib.util.Closeable;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
 import org.apache.commons.lang.Validate;
@@ -33,9 +32,11 @@ public class PlayerAttributes {
     public void load(ConfigurationSection config) {
         for (String key : config.getKeys(false))
             try {
-                final String id = key.toLowerCase().replace("_", "-").replace(" ", "-");
+                String id = key.toLowerCase().replace("_", "-").replace(" ", "-");
-                Validate.isTrue(MMOCore.plugin.attributeManager.has(id), "Could not find attribute called '" + id + "'");
+                Validate.isTrue(MMOCore.plugin.attributeManager.has(id), "Could not find attribute '" + id + "'");
-                final AttributeInstance ins = new AttributeInstance(id);
+
+                PlayerAttribute attribute = MMOCore.plugin.attributeManager.get(id);
+                AttributeInstance ins = new AttributeInstance(attribute.getId());
                 ins.setBase(config.getInt(key));
                 instances.put(id, ins);
             } catch (IllegalArgumentException exception) {
@@ -55,12 +56,15 @@ public class PlayerAttributes {
     }
 
     public void load(String json) {
-        JsonObject jo = MythicLib.plugin.getGson().fromJson(json, JsonObject.class);
+        Gson parser = new Gson();
+        JsonObject jo = parser.fromJson(json, JsonObject.class);
         for (Entry<String, JsonElement> entry : jo.entrySet()) {
             try {
-                final String id = entry.getKey().toLowerCase().replace("_", "-").replace(" ", "-");
+                String id = entry.getKey().toLowerCase().replace("_", "-").replace(" ", "-");
-                Validate.isTrue(MMOCore.plugin.attributeManager.has(id), "Could not find attribute called '" + id + "'");
+                Validate.isTrue(MMOCore.plugin.attributeManager.has(id), "Could not find attribute '" + id + "'");
-                final AttributeInstance ins = new AttributeInstance(id);
+
+                PlayerAttribute attribute = MMOCore.plugin.attributeManager.get(id);
+                AttributeInstance ins = new AttributeInstance(attribute.getId());
                 ins.setBase(entry.getValue().getAsInt());
                 instances.put(id, ins);
             } catch (IllegalArgumentException exception) {
@@ -109,14 +113,13 @@ public class PlayerAttributes {
         return n;
     }
 
-    // TODO have it extend ModifiedInstance
     public class AttributeInstance {
         private int spent;
 
         private final String id, enumName;
         private final Map<String, AttributeModifier> map = new HashMap<>();
 
-        public AttributeInstance(@NotNull String id) {
+        public AttributeInstance(String id) {
             this.id = id;
             this.enumName = UtilityMethods.enumName(this.id);
         }
@@ -125,11 +128,6 @@ public class PlayerAttributes {
             return spent;
         }
 
-        @Deprecated
-        public int getSpent() {
-            return getBase();
-        }
-
         public void setBase(int value) {
             spent = Math.max(0, value);
 
@@ -138,7 +136,7 @@ public class PlayerAttributes {
         }
 
         public void addBase(int value) {
-            setBase(getBase() + value);
+            setBase(spent + value);
         }
 
         /*
@@ -178,7 +176,8 @@ public class PlayerAttributes {
         public AttributeModifier addModifier(AttributeModifier modifier) {
             final AttributeModifier current = map.put(modifier.getKey(), modifier);
 
-            if (current instanceof Closeable) ((Closeable) current).close();
+            if (current != null && current instanceof Closeable)
+                ((Closeable) current).close();
 
             updateStats();
             return current;
@@ -211,12 +210,6 @@ public class PlayerAttributes {
         public void updateStats() {
             final PlayerAttribute attr = MMOCore.plugin.attributeManager.get(id);
             final int total = getTotal();
-
-            // Remove ALL stat modifiers
-            for (StatInstance ins : data.getMMOPlayerData().getStatMap().getInstances())
-                ins.removeIf(str -> str.equals("attribute." + id));
-
-            // Register new stat modifiers
             attr.getBuffs().forEach(buff -> buff.multiply(total).register(data.getMMOPlayerData()));
         }
 
@@ -225,9 +218,10 @@ public class PlayerAttributes {
         }
     }
 
-    @Deprecated
     public void setBaseAttribute(String id, int value) {
-        AttributeInstance ins = instances.get(id);
+        getInstances().forEach(ins -> {
-        if (ins != null) ins.setBase(value);
+            if (ins.getId().equals(id))
+                ins.setBase(value);
+        });
     }
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/player/profess/PlayerClass.java

@@ -1,9 +1,12 @@
 package net.Indyuce.mmocore.api.player.profess;
 
+import com.mojang.authlib.GameProfile;
+import com.mojang.authlib.properties.Property;
 import io.lumine.mythic.lib.MythicLib;
 import io.lumine.mythic.lib.UtilityMethods;
 import io.lumine.mythic.lib.api.MMOLineConfig;
 import io.lumine.mythic.lib.api.player.EquipmentSlot;
+import io.lumine.mythic.lib.api.util.PostLoadObject;
 import io.lumine.mythic.lib.player.modifier.ModifierSource;
 import io.lumine.mythic.lib.player.skill.PassiveSkill;
 import io.lumine.mythic.lib.script.Script;
@@ -11,10 +14,7 @@ import io.lumine.mythic.lib.skill.SimpleSkill;
 import io.lumine.mythic.lib.skill.Skill;
 import io.lumine.mythic.lib.skill.handler.MythicLibSkillHandler;
 import io.lumine.mythic.lib.skill.trigger.TriggerType;
-import io.lumine.mythic.lib.util.FileUtils;
+import io.lumine.mythic.lib.version.VersionMaterial;
-import io.lumine.mythic.lib.util.PostLoadAction;
-import io.lumine.mythic.lib.util.PreloadedObject;
-import io.lumine.mythic.lib.version.VParticle;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
 import net.Indyuce.mmocore.api.player.profess.event.EventTrigger;
@@ -25,29 +25,30 @@ import net.Indyuce.mmocore.api.util.MMOCoreUtils;
 import net.Indyuce.mmocore.api.util.math.formula.LinearValue;
 import net.Indyuce.mmocore.experience.EXPSource;
 import net.Indyuce.mmocore.experience.ExpCurve;
-import net.Indyuce.mmocore.experience.ExperienceObject;
 import net.Indyuce.mmocore.experience.droptable.ExperienceTable;
 import net.Indyuce.mmocore.loot.chest.particle.CastingParticle;
 import net.Indyuce.mmocore.player.stats.StatInfo;
 import net.Indyuce.mmocore.skill.ClassSkill;
 import net.Indyuce.mmocore.skill.RegisteredSkill;
-import net.Indyuce.mmocore.skill.binding.SkillSlot;
 import net.Indyuce.mmocore.skill.cast.ComboMap;
+import net.Indyuce.mmocore.experience.ExperienceObject;
 import net.Indyuce.mmocore.skilltree.tree.SkillTree;
 import net.md_5.bungee.api.ChatColor;
 import org.bukkit.Location;
 import org.bukkit.Material;
+import org.bukkit.Particle;
 import org.bukkit.configuration.ConfigurationSection;
+import org.bukkit.configuration.file.FileConfiguration;
 import org.bukkit.inventory.ItemStack;
 import org.bukkit.inventory.meta.ItemMeta;
-import org.bukkit.inventory.meta.SkullMeta;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
+import java.lang.reflect.Field;
 import java.util.*;
 import java.util.logging.Level;
 
-public class PlayerClass implements ExperienceObject, PreloadedObject {
+public class PlayerClass extends PostLoadObject implements ExperienceObject {
     private final String name, id, actionBarFormat;
     private final List<String> description = new ArrayList<>(), attrDescription = new ArrayList<>();
     private final ItemStack icon;
@@ -66,7 +67,8 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
     @Nullable
     private final CastingParticle castParticle;
 
-    private final List<SkillSlot> skillSlots = new ArrayList<>();
+    private final int maxBoundActiveSkills, maxBoundPassiveSkills;
+
     private final List<SkillTree> skillTrees = new ArrayList<>();
     private final List<PassiveSkill> classScripts = new LinkedList();
     private final Map<String, LinearValue> stats = new HashMap<>();
@@ -80,32 +82,26 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
     @Deprecated
     private final Map<String, EventTrigger> eventTriggers = new HashMap<>();
 
-    private final PostLoadAction postLoadAction = new PostLoadAction(config -> {
+    public PlayerClass(String id, FileConfiguration config) {
-        if (config.contains("subclasses"))
+        super(config);
-            for (String key : config.getConfigurationSection("subclasses").getKeys(false))
-                try {
-                    subclasses.add(new Subclass(
-                            MMOCore.plugin.classManager
-                                    .getOrThrow(key.toUpperCase().replace("-", "_").replace(" ", "_")),
-                            config.getInt("subclasses." + key)));
-                } catch (IllegalArgumentException exception) {
-                    MMOCore.plugin.getLogger().log(Level.WARNING, "Could not load subclass '" + key + "' from class '"
-                            + getId() + "': " + exception.getMessage());
-                }
-    });
-
-    public PlayerClass(String id, ConfigurationSection config) {
-        postLoadAction.cacheConfig(config);
 
         this.id = id.toUpperCase().replace("-", "_").replace(" ", "_");
 
         name = MythicLib.plugin.parseColors(config.getString("display.name", "INVALID DISPLAY NAME"));
         icon = MMOCoreUtils.readIcon(config.getString("display.item", "BARRIER"));
 
-        if (config.contains("display.texture") && icon.getType() == Material.PLAYER_HEAD) {
+        if (config.contains("display.texture") && icon.getType() == VersionMaterial.PLAYER_HEAD.toMaterial())
+            try {
                 ItemMeta meta = icon.getItemMeta();
-            UtilityMethods.setTextureValue((SkullMeta) meta, config.getString("display.texture"));
+                Field profileField = meta.getClass().getDeclaredField("profile");
+                profileField.setAccessible(true);
+                GameProfile gp = new GameProfile(UUID.randomUUID(), null);
+                gp.getProperties().put("textures", new Property("textures", config.getString("display.texture")));
+                profileField.set(meta, gp);
                 icon.setItemMeta(meta);
+            } catch (IllegalArgumentException | IllegalAccessException | NoSuchFieldException
+                     | SecurityException exception) {
+                throw new IllegalArgumentException("Could not apply playerhead texture: " + exception.getMessage());
             }
 
         for (String string : config.getStringList("display.lore"))
@@ -118,13 +114,12 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
         displayOrder = config.getInt("display.order");
         actionBarFormat = config.contains("action-bar", true) ? config.getString("action-bar") : null;
 
-        // Exp curve
         expCurve = config.contains("exp-curve")
                 ? MMOCore.plugin.experience.getCurveOrThrow(
                 config.get("exp-curve").toString().toLowerCase().replace("_", "-").replace(" ", "-"))
                 : ExpCurve.DEFAULT;
-
+        maxBoundActiveSkills = config.getInt("max-bound-active-skills", MMOCore.plugin.configManager.maxBoundActiveSkills);
-        // Main exp table
+        maxBoundPassiveSkills = config.getInt("max-bound-passive-skills", MMOCore.plugin.configManager.maxBoundPassiveSkills);
         ExperienceTable expTable = null;
         if (config.contains("exp-table"))
             try {
@@ -133,8 +128,6 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
                 MMOCore.plugin.getLogger().log(Level.WARNING, "Could not load exp table from class '" + id + "': " + exception.getMessage());
             }
         this.expTable = expTable;
-
-        // Skill trees
         if (config.contains("skill-trees"))
             for (String str : config.getStringList("skill-trees"))
                 try {
@@ -143,7 +136,6 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
                     MMOCore.log(Level.WARNING, "Could not find skill tree with ID: " + str);
                 }
 
-        // Class-specific scripts
         if (config.contains("scripts"))
             for (String key : config.getConfigurationSection("scripts").getKeys(false))
                 try {
@@ -166,7 +158,6 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
             }
         this.comboMap = comboMap;
 
-        // Triggers (DEPRECATED)
         if (config.contains("triggers"))
             for (String key : config.getConfigurationSection("triggers").getKeys(false))
                 try {
@@ -176,7 +167,6 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
                     MMOCore.log(Level.WARNING, "Could not load trigger '" + key + "' from class '" + id + "':" + exception.getMessage());
                 }
 
-        // Class STATS, not attributes (historic reasons)
         if (config.contains("attributes"))
             for (String key : config.getConfigurationSection("attributes").getKeys(false))
                 try {
@@ -187,28 +177,18 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
                             + id + "': " + exception.getMessage());
                 }
 
-        // Skill slots
+        if (config.contains("skills"))
-        if (config.isConfigurationSection("skill-slots"))
+            for (String key : config.getConfigurationSection("skills").getKeys(false))
-            FileUtils.iterateConfigSectionList(
+                try {
-                    config.getConfigurationSection("skill-slots"),
+                    RegisteredSkill registered = MMOCore.plugin.skillManager.getSkillOrThrow(UtilityMethods.enumName(key));
-                    skillSlots,
+                    skills.put(registered.getHandler().getId(), new ClassSkill(registered, config.getConfigurationSection("skills." + key)));
-                    SkillSlot::new,
+                } catch (RuntimeException exception) {
-                    index -> new SkillSlot(index, 0, "true", "&eUnconfigured Skill Slot " + MMOCoreUtils.intToRoman(index), new ArrayList<>(), false, true, new ArrayList<>()),
+                    MMOCore.plugin.getLogger().log(Level.WARNING, "Could not load skill info '" + key + "' from class '"
-                    (key, exception) -> MMOCore.plugin.getLogger().log(Level.WARNING, "Could not load skill slot '" + key + "' from class '" + getId() + "': " + exception.getMessage()));
+                            + id + "': " + exception.getMessage());
-
-        // Class skills
-        for (RegisteredSkill registered : MMOCore.plugin.skillManager.getAll()) {
-            final String key = registered.getHandler().getId();
-            if (config.contains("skills." + key))
-                skills.put(key, new ClassSkill(registered, config.getConfigurationSection("skills." + key)));
-            else
-                skills.put(key, new ClassSkill(registered, 1, 1, false));
                 }
 
-        // Casting particle
         castParticle = config.contains("cast-particle") ? new CastingParticle(config.getConfigurationSection("cast-particle")) : null;
 
-        // Other class options
         if (config.contains("options"))
             for (String key : config.getConfigurationSection("options").getKeys(false))
                 try {
@@ -219,12 +199,11 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
                             "Could not load option '" + key + "' from class '" + key + "': " + exception.getMessage());
                 }
 
-        // Experience sources
         if (config.contains("main-exp-sources")) {
             for (String key : config.getStringList("main-exp-sources"))
                 try {
                     MMOCore.plugin.experience.registerSource(MMOCore.plugin.loadManager.loadExperienceSource(new MMOLineConfig(key), this));
-                } catch (RuntimeException exception) {
+                } catch (IllegalArgumentException exception) {
                     MMOCore.plugin.getLogger().log(Level.WARNING, "Could not load exp source '" + key + "' from class '"
                             + id + "': " + exception.getMessage());
                 }
@@ -258,6 +237,8 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
      * option was not provided.
      */
     public PlayerClass(String id, String name, Material material) {
+        super(null);
+
         this.id = id;
         this.name = name;
         manaDisplay = ManaDisplayOptions.DEFAULT;
@@ -266,15 +247,32 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
         expCurve = ExpCurve.DEFAULT;
         expTable = null;
         comboMap = null;
-        castParticle = new CastingParticle(VParticle.INSTANT_EFFECT.get());
+        castParticle = new CastingParticle(Particle.SPELL_INSTANT);
         actionBarFormat = "";
         this.icon = new ItemStack(material);
         setOption(ClassOption.DISPLAY, false);
         setOption(ClassOption.DEFAULT, false);
+        maxBoundActiveSkills = 6;
+        maxBoundPassiveSkills = 3;
         for (PlayerResource resource : PlayerResource.values())
             resourceHandlers.put(resource, new ResourceRegeneration(resource));
     }
 
+    @Override
+    protected void whenPostLoaded(ConfigurationSection config) {
+        if (config.contains("subclasses"))
+            for (String key : config.getConfigurationSection("subclasses").getKeys(false))
+                try {
+                    subclasses.add(new Subclass(
+                            MMOCore.plugin.classManager
+                                    .getOrThrow(key.toUpperCase().replace("-", "_").replace(" ", "_")),
+                            config.getInt("subclasses." + key)));
+                } catch (IllegalArgumentException exception) {
+                    MMOCore.plugin.getLogger().log(Level.WARNING, "Could not load subclass '" + key + "' from class '"
+                            + getId() + "': " + exception.getMessage());
+                }
+    }
+
     public String getId() {
         return id;
     }
@@ -285,7 +283,7 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
 
     @Override
     public String getKey() {
-        return "class_" + getId();
+        return "class." + getId();
     }
 
     @NotNull
@@ -306,17 +304,18 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
         return displayOrder;
     }
 
-    @NotNull
-    @Override
-    public PostLoadAction getPostLoadAction() {
-        return postLoadAction;
-    }
-
     @Override
     public ExpCurve getExpCurve() {
         return expCurve;
     }
 
+    public int getMaxBoundActiveSkills() {
+        return maxBoundActiveSkills;
+    }
+
+    public int getMaxBoundPassiveSkills() {
+        return maxBoundPassiveSkills;
+    }
 
     @NotNull
     public ExperienceTable getExperienceTable() {
@@ -355,7 +354,8 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
 
     @Override
     public void giveExperience(PlayerData playerData, double experience, @Nullable Location hologramLocation, EXPSource source) {
-        hologramLocation = !MMOCore.plugin.getConfig().getBoolean("display-main-class-exp-holograms") ? null : hologramLocation;
+        hologramLocation = !MMOCore.plugin.getConfig().getBoolean("display-main-class-exp-holograms") ? null
+                : hologramLocation;
         playerData.giveExperience(experience, source, hologramLocation, true);
     }
 
@@ -416,35 +416,18 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
         return false;
     }
 
-    @Deprecated
     public boolean hasSkill(RegisteredSkill skill) {
         return hasSkill(skill.getHandler().getId());
     }
 
-    @Deprecated
     public boolean hasSkill(String id) {
         return skills.containsKey(id);
     }
 
-    public boolean hasSlot(int slot) {
-        return 1 <= slot && slot <= skillSlots.size();
-    }
-
     public List<SkillTree> getSkillTrees() {
         return skillTrees;
     }
 
-    @Nullable
-    public SkillSlot getSkillSlot(int slot) {
-        return hasSlot(slot) ? skillSlots.get(slot - 1) : null;
-    }
-
-    @NotNull
-    public List<SkillSlot> getSlots() {
-        return skillSlots;
-    }
-
-    @NotNull
     public ClassSkill getSkill(RegisteredSkill skill) {
         return getSkill(skill.getHandler().getId());
     }
@@ -454,7 +437,6 @@ public class PlayerClass implements ExperienceObject, PreloadedObject {
         return skills.get(id);
     }
 
-    @Deprecated
     public Collection<ClassSkill> getSkills() {
         return skills.values();
     }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/player/profess/SavedClassInformation.java

@@ -1,35 +1,29 @@
 package net.Indyuce.mmocore.api.player.profess;
 
-import io.lumine.mythic.lib.gson.JsonElement;
+import com.google.gson.JsonElement;
-import io.lumine.mythic.lib.gson.JsonObject;
+import com.google.gson.JsonObject;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
 import net.Indyuce.mmocore.api.player.attribute.PlayerAttribute;
-import net.Indyuce.mmocore.api.player.attribute.PlayerAttributes;
-import net.Indyuce.mmocore.api.util.MMOCoreUtils;
 import net.Indyuce.mmocore.player.ClassDataContainer;
+import net.Indyuce.mmocore.skill.ClassSkill;
 import net.Indyuce.mmocore.skill.RegisteredSkill;
 import net.Indyuce.mmocore.skilltree.SkillTreeNode;
 import net.Indyuce.mmocore.skilltree.tree.SkillTree;
-import org.bukkit.attribute.Attribute;
 import org.bukkit.configuration.ConfigurationSection;
 
-import java.util.HashMap;
+import java.util.*;
-import java.util.HashSet;
-import java.util.Map;
 import java.util.Map.Entry;
-import java.util.Set;
 
-public class SavedClassInformation implements ClassDataContainer {
+public class SavedClassInformation {
     private final int level, skillPoints, attributePoints, attributeReallocationPoints, skillTreeReallocationPoints, skillReallocationPoints;
-    private final double experience, health, mana, stellium, stamina;
+    private final double experience;
     private final Map<String, Integer> attributeLevels = new HashMap<>();
     private final Map<String, Integer> skillLevels = new HashMap<>();
     private final Map<String, Integer> skillTreePoints = new HashMap<>();
     private final Map<String, Integer> nodeLevels = new HashMap<>();
     private final Map<String, Integer> nodeTimesClaimed = new HashMap<>();
-    private final Map<Integer, String> boundSkills = new HashMap<>();
+    private final List<String> boundSkills = new ArrayList<>();
-    private final Set<String> unlockedItems = new HashSet<>();
 
     /**
      * Used by YAML storage
@@ -42,34 +36,18 @@ public class SavedClassInformation implements ClassDataContainer {
         attributeReallocationPoints = config.getInt("attribute-realloc-points");
         skillReallocationPoints = config.getInt("skill-reallocation-points");
         skillTreeReallocationPoints = config.getInt("skill-tree-reallocation-points");
-        health = config.getDouble("health", 20);
-        mana = config.getDouble("mana", 0);
-        stamina = config.getDouble("stamina", 0);
-        stellium = config.getDouble("stellium", 0);
         if (config.contains("attribute"))
-            config.getConfigurationSection("attribute").getKeys(false)
+            config.getConfigurationSection("attribute").getKeys(false).forEach(key -> attributeLevels.put(key, config.getInt("attribute." + key)));
-                    .forEach(key -> attributeLevels.put(key, config.getInt("attribute." + key)));
         if (config.contains("skill"))
-            config.getConfigurationSection("skill").getKeys(false)
+            config.getConfigurationSection("skill").getKeys(false).forEach(key -> skillLevels.put(key, config.getInt("skill." + key)));
-                    .forEach(key -> skillLevels.put(key, config.getInt("skill." + key)));
         if (config.contains("skill-tree-points"))
-            config.getConfigurationSection("skill-tree-points").getKeys(false)
+            config.getConfigurationSection("skill-tree-points").getKeys(false).forEach(key -> skillTreePoints.put(key, config.getInt("skill-tree-points." + key)));
-                    .forEach(key -> skillTreePoints.put(key, config.getInt("skill-tree-points." + key)));
         if (config.contains("node-levels"))
-            config.getConfigurationSection("node-levels").getKeys(false)
+            config.getConfigurationSection("node-levels").getKeys(false).forEach(key -> nodeLevels.put(key, config.getInt("node-levels." + key)));
-                    .forEach(key -> nodeLevels.put(key, config.getInt("node-levels." + key)));
         if (config.contains("node-times-claimed"))
-            config.getConfigurationSection("node-times-claimed").getKeys(false)
+            config.getConfigurationSection("node-times-claimed").getKeys(false).forEach(key -> nodeTimesClaimed.put(key, config.getInt("node-times-claimed." + key)));
-                    .forEach(key -> nodeTimesClaimed.put(key, config.getInt("node-times-claimed." + key)));
+        if (config.contains("bound-skills"))
-
+            config.getStringList("bound-skills").forEach(id -> boundSkills.add(id));
-        /**
-         * 'bound-skills' used to be a list. This condition makes
-         * sure that the config is using the newest format.
-         */
-        if (config.isConfigurationSection("bound-skills"))
-            config.getConfigurationSection("bound-skills").getKeys(false)
-                    .forEach(key -> boundSkills.put(Integer.parseInt(key), config.getString("bound-skills." + key)));
-        unlockedItems.addAll(config.getStringList("unlocked-items"));
     }
 
     /**
@@ -83,11 +61,6 @@ public class SavedClassInformation implements ClassDataContainer {
         attributeReallocationPoints = json.get("attribute-realloc-points").getAsInt();
         skillReallocationPoints = json.get("skill-reallocation-points").getAsInt();
         skillTreeReallocationPoints = json.get("skill-tree-reallocation-points").getAsInt();
-        health = json.has("health") ? json.get("health").getAsDouble() : 20;
-        mana = json.has("mana") ? json.get("mana").getAsDouble() : 0;
-        stamina = json.has("stamina") ? json.get("stamina").getAsDouble() : 0;
-        stellium = json.has("stellium") ? json.get("stellium").getAsDouble() : 0;
-
         if (json.has("attribute"))
             for (Entry<String, JsonElement> entry : json.getAsJsonObject("attribute").entrySet())
                 attributeLevels.put(entry.getKey(), entry.getValue().getAsInt());
@@ -103,13 +76,8 @@ public class SavedClassInformation implements ClassDataContainer {
         if (json.has("node-times-claimed"))
             for (Entry<String, JsonElement> entry : json.getAsJsonObject("node-times-claimed").entrySet())
                 nodeTimesClaimed.put(entry.getKey(), entry.getValue().getAsInt());
-        //Old system was using a JsonArray. If it saved with the old system the if condition won't be respected.
+        if (json.has("bound-skills"))
-        if (json.has("bound-skills") && json.get("bound-skills").isJsonObject())
+            json.getAsJsonArray("bound-skills").forEach(id -> boundSkills.add(id.getAsString()));
-            for (Entry<String, JsonElement> entry : json.getAsJsonObject("bound-skills").entrySet())
-                boundSkills.put(Integer.parseInt(entry.getKey()), entry.getValue().getAsString());
-        if (json.has("unlocked-items"))
-            for (JsonElement unlockedItem : json.get("unlocked-items").getAsJsonArray())
-                unlockedItems.add(unlockedItem.getAsString());
     }
 
     public SavedClassInformation(ClassDataContainer data) {
@@ -120,65 +88,37 @@ public class SavedClassInformation implements ClassDataContainer {
         this.skillTreeReallocationPoints = data.getSkillTreeReallocationPoints();
         this.skillReallocationPoints = data.getSkillReallocationPoints();
         this.experience = data.getExperience();
-        this.health = data.getHealth();
-        this.mana = data.getMana();
-        this.stellium = data.getStellium();
-        this.stamina = data.getStamina();
 
-        attributeLevels.putAll(data.mapAttributeLevels());
+        data.mapAttributeLevels().forEach((key, val) -> this.attributeLevels.put(key, val));
-        skillLevels.putAll(data.mapSkillLevels());
+        data.mapSkillLevels().forEach((key, val) -> skillLevels.put(key, val));
-        skillTreePoints.putAll(data.mapSkillTreePoints());
+        data.mapSkillTreePoints().forEach((key, val) -> skillTreePoints.put(key, val));
-        nodeLevels.putAll(data.getNodeLevels());
+        data.getNodeLevels().forEach((node, level) -> nodeLevels.put(node.getFullId(), level));
-        nodeTimesClaimed.putAll(data.getNodeTimesClaimed());
+        data.getNodeTimesClaimed().forEach((key, val) -> nodeTimesClaimed.put(key, val));
-        boundSkills.putAll(data.mapBoundSkills());
+
-        unlockedItems.addAll(data.getUnlockedItems());
+        data.getBoundPassiveSkills().forEach(skill -> boundSkills.add(skill.getTriggeredSkill().getHandler().getId()));
+        data.getBoundSkills().forEach(skill -> boundSkills.add(skill.getSkill().getHandler().getId()));
     }
 
-    @Override
     public int getLevel() {
         return level;
     }
 
-    @Override
     public double getExperience() {
         return experience;
     }
 
-    @Override
     public int getSkillPoints() {
         return skillPoints;
     }
 
-    @Override
     public int getAttributePoints() {
         return attributePoints;
     }
 
-    @Override
     public int getAttributeReallocationPoints() {
         return attributeReallocationPoints;
     }
 
-    @Override
-    public double getHealth() {
-        return health;
-    }
-
-    @Override
-    public double getMana() {
-        return mana;
-    }
-
-    @Override
-    public double getStellium() {
-        return stellium;
-    }
-
-    @Override
-    public double getStamina() {
-        return stamina;
-    }
-
     public Set<String> getSkillKeys() {
         return skillLevels.keySet();
     }
@@ -195,27 +135,10 @@ public class SavedClassInformation implements ClassDataContainer {
         registerSkillLevel(skill.getHandler().getId(), level);
     }
 
-    @Override
-    public Map<String, Integer> mapSkillLevels() {
-        return skillLevels;
-    }
-
-    @Override
-    public Map<String, Integer> mapSkillTreePoints() {
-        return skillTreePoints;
-    }
-
-    @Deprecated
-    public Map<Integer, String> getBoundSkills() {
-        return mapBoundSkills();
-    }
-
-    @Override
     public int getSkillTreeReallocationPoints() {
         return skillTreeReallocationPoints;
     }
 
-    @Override
     public int getSkillReallocationPoints() {
         return skillReallocationPoints;
     }
@@ -232,26 +155,6 @@ public class SavedClassInformation implements ClassDataContainer {
         return nodeLevels.get(node);
     }
 
-    @Override
-    public Map<String, Integer> mapAttributeLevels() {
-        return attributeLevels;
-    }
-
-    @Override
-    public Map<Integer, String> mapBoundSkills() {
-        return boundSkills;
-    }
-
-    @Override
-    public Map<String, Integer> getNodeLevels() {
-        return nodeLevels;
-    }
-
-    @Override
-    public Map<String, Integer> getNodeTimesClaimed() {
-        return nodeTimesClaimed;
-    }
-
     public Set<String> getSkillTreePointsKeys() {
         return skillTreePoints.keySet();
     }
@@ -276,11 +179,6 @@ public class SavedClassInformation implements ClassDataContainer {
         attributeLevels.put(attribute, level);
     }
 
-    @Override
-    public Set<String> getUnlockedItems() {
-        return unlockedItems;
-    }
-
     /**
      * @param profess Target player class
      * @param player  Player changing class
@@ -294,22 +192,32 @@ public class SavedClassInformation implements ClassDataContainer {
         if (!player.getProfess().hasOption(ClassOption.DEFAULT) || MMOCore.plugin.configManager.saveDefaultClassInfo)
             player.applyClassInfo(player.getProfess(), new SavedClassInformation(player));
 
-        // Remove class permanent buffs
-        player.getProfess().resetAdvancement(player, false);
-
         /*
          * Resets information which much be reset after everything is saved.
          */
         player.mapSkillLevels().forEach((skill, level) -> player.resetSkillLevel(skill));
-        for (PlayerAttribute attribute : MMOCore.plugin.attributeManager.getAll()) {
+        player.getAttributes().getInstances().forEach(ins -> ins.setBase(0));
-            attribute.resetAdvancement(player, false);
+        player.clearSkillTreePoints();
-            player.getAttributes().getInstance(attribute).setBase(0);
+        player.clearNodeLevels();
-        }
+        player.clearNodeStates();
-        player.resetSkillTrees();
+
+        // We remove perm stats for nodes and class.
+        for (SkillTree skillTree : player.getProfess().getSkillTrees())
+            for (SkillTreeNode node : skillTree.getNodes())
+                node.getExperienceTable().removePermStats(player, node);
+        if (player.getProfess().hasExperienceTable())
+            player.getProfess().getExperienceTable().removePermStats(player, player.getProfess());
+
+        while (player.hasPassiveSkillBound(0))
+            player.unbindPassiveSkill(0);
+
+        while (player.hasSkillBound(0))
+            player.unbindSkill(0);
 
         /*
          * Reads this class info, applies it to the player. set class after
-         * changing level so the player stats can be calculated based on new level
+         * changing level so the player stats can be calculated based on new
+         * level.
          */
         player.setLevel(level);
         player.setExperience(experience);
@@ -318,13 +226,16 @@ public class SavedClassInformation implements ClassDataContainer {
         player.setAttributeReallocationPoints(attributeReallocationPoints);
         player.setSkillTreeReallocationPoints(skillTreeReallocationPoints);
         player.setSkillReallocationPoints(skillReallocationPoints);
-        player.setUnlockedItems(unlockedItems);
+        for (String id : boundSkills) {
-        player.setClass(profess);
+            ClassSkill skill = profess.getSkill(id);
-        for (int slot : boundSkills.keySet())
+            if (skill.getSkill().getTrigger().isPassive())
-            player.bindSkill(slot, profess.getSkill(boundSkills.get(slot)));
+                player.bindPassiveSkill(-1, skill.toPassive(player));
+            else
+                player.getBoundSkills().add(skill);
+        }
 
         skillLevels.forEach(player::setSkillLevel);
-        attributeLevels.forEach((id, pts) -> player.getAttributes().getInstance(id).setBase(pts));
+        attributeLevels.forEach((id, pts) -> player.getAttributes().setBaseAttribute(id, pts));
 
         // Careful, the global points must not be forgotten.
         player.setSkillTreePoints("global", skillTreePoints.getOrDefault("global", 0));
@@ -339,17 +250,21 @@ public class SavedClassInformation implements ClassDataContainer {
         // Add the values to the times claimed table and claims the corresponding stat triggers.
         nodeTimesClaimed.forEach((str, val) -> player.setClaims(str, val));
 
-        // Unload current class information
+        // We claim back the stats triggers for all the skill tree nodes of the new class.
+        for (SkillTree skillTree : profess.getSkillTrees())
+            for (SkillTreeNode node : skillTree.getNodes())
+                node.getExperienceTable().claimStatTriggers(player, node);
+        profess.getExperienceTable().claimStatTriggers(player, profess);
+
+        /*
+         * Unload current class information and set
+         * the new profess once everything is changed
+         */
+        player.setClass(profess);
         player.unloadClassInfo(profess);
 
-        // This needs to be done at the end to make sure the MAX_HEALTH/MAX_MANA/... stats are loaded.
+        // Updates level on exp bar
-        player.getPlayer().setHealth(MMOCoreUtils.fixResource(health, player.getPlayer().getAttribute(Attribute.GENERIC_MAX_HEALTH).getValue()));
+        player.refreshVanillaExp();
-        player.setHealth(health);
-        player.setMana(mana);
-        player.setStellium(stellium);
-        player.setStamina(stamina);
-        player.applyTemporaryTriggers();
-        player.getStats().updateStats();
     }
 }
 

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/player/profess/event/trigger/AttackEventTrigger.java

@@ -21,7 +21,7 @@ public class AttackEventTrigger implements EventTriggerHandler {
         // We don't want players dying by themselves when using an enderpearl.
         if (event.getPlayer().equals(event.getEntity())) return;
 
-        PlayerData player = PlayerData.get(event.getData());
+        PlayerData player = PlayerData.get(event.getData().getUniqueId());
         PlayerClass profess = player.getProfess();
 
         for (DamageType type : event.getAttack().getDamage().collectTypes()) {

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/player/profess/event/trigger/BlockBrokenTrigger.java

@@ -1,6 +1,5 @@
 package net.Indyuce.mmocore.api.player.profess.event.trigger;
 
-import io.lumine.mythic.lib.UtilityMethods;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.EventPriority;
 import org.bukkit.event.block.BlockBreakEvent;
@@ -18,8 +17,6 @@ public class BlockBrokenTrigger implements EventTriggerHandler {
 
 	@EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true)
 	public void a(BlockBreakEvent event) {
-		if (UtilityMethods.isFake(event)) return;
-
 		PlayerData player = PlayerData.get(event.getPlayer());
 		if (player.getProfess().hasEventTriggers("break-block"))
 			player.getProfess().getEventTriggers("break-block").getTriggers().forEach(trigger -> trigger.apply(player));

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/player/social/FriendRequest.java

@@ -1,6 +1,6 @@
 package net.Indyuce.mmocore.api.player.social;
 
-import net.Indyuce.mmocore.api.ConfigMessage;
+import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerActivity;
 import net.Indyuce.mmocore.api.player.PlayerData;
 import org.bukkit.Sound;
@@ -21,8 +21,8 @@ public class FriendRequest extends Request {
         getCreator().addFriend(getTarget().getUniqueId());
         getTarget().addFriend(getCreator().getUniqueId());
         if (getCreator().isOnline()) {
-            ConfigMessage.fromKey("now-friends", "player", getTarget().getPlayer().getName()).send(getCreator().getPlayer());
+            MMOCore.plugin.configManager.getSimpleMessage("now-friends", "player", getTarget().getPlayer().getName()).send(getCreator().getPlayer());
-            ConfigMessage.fromKey("now-friends", "player", getCreator().getPlayer().getName()).send(getTarget().getPlayer());
+            MMOCore.plugin.configManager.getSimpleMessage("now-friends", "player", getCreator().getPlayer().getName()).send(getTarget().getPlayer());
         }
     }
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/player/stats/PlayerStats.java

@@ -8,7 +8,6 @@ import io.lumine.mythic.lib.player.modifier.ModifierSource;
 import io.lumine.mythic.lib.player.modifier.ModifierType;
 import io.lumine.mythic.lib.player.skill.PassiveSkill;
 import io.lumine.mythic.lib.player.skill.PassiveSkillMap;
-import io.lumine.mythic.lib.skill.trigger.TriggerType;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
 import net.Indyuce.mmocore.experience.Profession;
@@ -46,7 +45,7 @@ public class PlayerStats {
     }
 
     public double getStat(String stat) {
-        return getMap().getStat(stat);
+        return getMap().getInstance(stat).getTotal();
     }
 
     /**
@@ -67,24 +66,14 @@ public class PlayerStats {
         return data.getProfess().calculateStat(stat, profession == null ? data.getLevel() : data.getCollectionSkills().getLevel(profession));
     }
 
-    public void updateStats() {
-        updateStats(false);
-    }
-
     /**
      * Used to update MMOCore stat modifiers due to class and send them over to
-     * MythicLib. Must be ran everytime the player levels up, changes class or
+     * MythicLib. Must be ran everytime the player levels up or changes class.
-     * when the plugin reloads.
      * <p>
-     * Login scripts are a pretty special case of scripts/skills since they are
+     * This is also called when reloading the plugin to make class setup easier,
-     * not loaded yet when MythicLib triggers them naturally. Therefore, they
+     * see {@link PlayerData#update()} for more info
-     * need to be cast as soon as they are loaded into the MMOCore player data.
-     *
-     * @param castLoginScripts Should login scripts be cast
      */
-    public synchronized void updateStats(boolean castLoginScripts) {
+    public synchronized void updateStats() {
-
-        // Update player stats
         for (String stat : MMOCore.plugin.statManager.getRegistered()) {
             final StatInstance instance = getMap().getInstance(stat);
             final StatInstance.ModifierPacket packet = instance.newPacket();
@@ -101,32 +90,25 @@ public class PlayerStats {
             packet.runUpdate();
         }
 
-        // Updates the player's unbindable CLASS passive skills
+        /*
+         * This is here because it requires updates for the same reasons
+         * as statistics (when the player level changes, when his class
+         * changes, when he logs on..)
+         *
+         * This updates the player's PASSIVE skills
+         */
         final PassiveSkillMap skillMap = data.getMMOPlayerData().getPassiveSkillMap();
-        skillMap.removeModifiers("MMOCorePermanentSkill");
-        for (ClassSkill skill : data.getProfess().getSkills())
-            if (skill.isPermanent()
-                    && skill.getSkill().getTrigger() != TriggerType.LOGIN
-                    && data.hasUnlocked(skill)
-                    && data.hasUnlockedLevel(skill))
-                skillMap.addModifier(skill.toPassive(data));
 
-        // Updates the player's CLASS scripts
+        if (!MMOCore.plugin.configManager.passiveSkillNeedBound) {
+            skillMap.removeModifiers("MMOCorePassiveSkill");
+            for (ClassSkill skill : data.getProfess().getSkills())
+                if (skill.getSkill().getTrigger().isPassive())
+                    skillMap.addModifier(skill.toPassive(data));
+        }
+
+        // This updates the player's class SCRIPTS
         skillMap.removeModifiers("MMOCoreClassScript");
         for (PassiveSkill script : data.getProfess().getScripts())
-            if (script.getType() != TriggerType.LOGIN) skillMap.addModifier(script);
+            skillMap.addModifier(script);
-
-        // If data hasn't been synchronized yet, cast LOGIN scripts
-        if (castLoginScripts) {
-
-            // Call class login skills
-            for (ClassSkill skill : data.getProfess().getSkills())
-                if (skill.getSkill().getTrigger() == TriggerType.LOGIN)
-                    skill.toCastable(data).cast(data.getMMOPlayerData());
-
-            // Call class login scripts
-            for (PassiveSkill skill : data.getProfess().getScripts())
-                if (skill.getType() == TriggerType.LOGIN) skill.getTriggeredSkill().cast(data.getMMOPlayerData());
-        }
     }
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/ObjectiveProgress.java

@@ -1,14 +1,15 @@
 package net.Indyuce.mmocore.api.quest;
 
-import io.lumine.mythic.lib.util.Closeable;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.api.quest.objective.Objective;
+import net.Indyuce.mmocore.api.util.Closable;
 import org.bukkit.Bukkit;
 import org.bukkit.event.HandlerList;
 import org.bukkit.event.Listener;
 
-public abstract class ObjectiveProgress implements Closeable {
+import net.Indyuce.mmocore.api.quest.objective.Objective;
+
+public abstract class ObjectiveProgress implements Closable {
 	private final Objective objective;
 	private final QuestProgress questProgress;
 

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/PlayerQuests.java

@@ -1,11 +1,11 @@
 package net.Indyuce.mmocore.api.quest;
 
-import io.lumine.mythic.lib.MythicLib;
+import com.google.gson.Gson;
-import io.lumine.mythic.lib.gson.JsonElement;
+import com.google.gson.JsonElement;
-import io.lumine.mythic.lib.gson.JsonObject;
+import com.google.gson.JsonObject;
-import io.lumine.mythic.lib.util.Closeable;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
+import net.Indyuce.mmocore.api.util.Closable;
 import org.bukkit.Bukkit;
 import org.bukkit.NamespacedKey;
 import org.bukkit.boss.BarColor;
@@ -21,7 +21,7 @@ import java.util.Map.Entry;
 import java.util.Set;
 import java.util.logging.Level;
 
-public class PlayerQuests implements Closeable {
+public class PlayerQuests implements Closable {
     private final PlayerData playerData;
     private final Map<String, Long> finished = new HashMap<>();
 
@@ -38,7 +38,6 @@ public class PlayerQuests implements Closeable {
             bossbarNamespacedKey = new NamespacedKey(MMOCore.plugin, "mmocore_quest_progress_" + playerData.getUniqueId().toString());
             bossbar = Bukkit.createBossBar(bossbarNamespacedKey, "", BarColor.PURPLE, BarStyle.SEGMENTED_20);
             bossbar.addPlayer(playerData.getPlayer());
-            bossbar.setVisible(false); // Safety
 
             // Bossbar is disabled
         } else {
@@ -99,7 +98,8 @@ public class PlayerQuests implements Closeable {
     }
 
     public void load(String json) {
-        JsonObject jo = MythicLib.plugin.getGson().fromJson(json, JsonObject.class);
+        Gson parser = new Gson();
+        JsonObject jo = parser.fromJson(json, JsonObject.class);
         if (jo.has("current")) {
             JsonObject cur = jo.getAsJsonObject("current");
             try {
@@ -173,9 +173,6 @@ public class PlayerQuests implements Closeable {
             bossbar.removeAll();
             Bukkit.removeBossBar(bossbarNamespacedKey);
         }
-
-        // Close current objective progress
-        closeCurrentQuest();
     }
 
     public boolean checkCooldownAvailability(Quest quest) {

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/Quest.java

@@ -1,8 +1,12 @@
 package net.Indyuce.mmocore.api.quest;
 
-import io.lumine.mythic.lib.api.MMOLineConfig;
+import java.util.ArrayList;
-import io.lumine.mythic.lib.util.PostLoadAction;
+import java.util.HashMap;
-import io.lumine.mythic.lib.util.PreloadedObject;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+import java.util.logging.Level;
+
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
 import net.Indyuce.mmocore.api.quest.objective.Objective;
@@ -10,32 +14,26 @@ import net.Indyuce.mmocore.experience.Profession;
 import org.apache.commons.lang.Validate;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.configuration.file.FileConfiguration;
-import org.jetbrains.annotations.NotNull;
 
-import java.util.*;
+import io.lumine.mythic.lib.api.MMOLineConfig;
-import java.util.logging.Level;
+import io.lumine.mythic.lib.api.util.PostLoadObject;
 
-public class Quest implements PreloadedObject {
+public class Quest extends PostLoadObject {
-	private final String id, name;
+	private final String id;
+
+	private final String name;
 	private final List<Quest> parents = new ArrayList<>();
 	private final List<Objective> objectives = new ArrayList<>();
 	private final List<String> lore;
+
 	private final int mainLevelRestriction;
 	private final Map<Profession, Integer> levelRestrictions = new HashMap<>();
 
-	// Cooldown in millis
+	// cooldown in millis
 	private final long cooldown;
 
-	private final PostLoadAction postLoadAction = new PostLoadAction(config -> {
+	public Quest(String id, FileConfiguration config) {
-
+		super(config);
-		// Load parent quests
-		if (config.contains("parent"))
-			for (String parent : config.getStringList("parent"))
-				parents.add(MMOCore.plugin.questManager.getOrThrow(parent.toLowerCase().replace(" ", "-").replace("_", "-")));
-	});
-
-	public Quest(String id, ConfigurationSection config) {
-		postLoadAction.cacheConfig(config);
 
 		this.id = id.toLowerCase().replace("_", "-").replace(" ", "-");
 		cooldown = (long) (config.contains("delay") ? config.getDouble("delay") * 60 * 60 * 1000 : -1);
@@ -71,10 +69,11 @@ public class Quest implements PreloadedObject {
 			}
 	}
 
-	@NotNull
 	@Override
-	public PostLoadAction getPostLoadAction() {
+	protected void whenPostLoaded(ConfigurationSection config) {
-		return postLoadAction;
+		if (config.contains("parent"))
+			for (String parent : config.getStringList("parent"))
+				parents.add(MMOCore.plugin.questManager.getOrThrow(parent.toLowerCase().replace(" ", "-").replace("_", "-")));
 	}
 
 	public String getId() {

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/QuestProgress.java

@@ -46,19 +46,17 @@ public class QuestProgress {
 	public void completeObjective() {
 		objective++;
 		objectiveProgress.close();
-        final ObjectiveProgress finishedObjectiveProgress = objectiveProgress;
 
-        // Start next objective, or end quest.
+		// apply triggers
-        if (objective >= quest.getObjectives().size()) player.getQuestData().finishCurrent();
+		objectiveProgress.getObjective().getTriggers().forEach(trigger -> trigger.schedule(getPlayer()));
-        else objectiveProgress = nextObjective().newProgress(this);
+
+		// end quest
+		if (objective >= quest.getObjectives().size())
+			player.getQuestData().finishCurrent();
+		else
+			objectiveProgress = nextObjective().newProgress(this);
 
 		player.getQuestData().updateBossBar();
-
-        /*
-         * Apply triggers only at the end! It comes handy when starting another
-         * quest in some storyline using triggers from the previous quest.
-         */
-        finishedObjectiveProgress.getObjective().getTriggers().forEach(trigger -> trigger.schedule(getPlayer()));
 	}
 
 	public String getFormattedLore() {

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/objective/MineBlockObjective.java

@@ -1,6 +1,5 @@
 package net.Indyuce.mmocore.api.quest.objective;
 
-import io.lumine.mythic.lib.UtilityMethods;
 import net.Indyuce.mmocore.api.quest.ObjectiveProgress;
 import net.Indyuce.mmocore.api.event.CustomBlockMineEvent;
 import net.Indyuce.mmocore.api.quest.QuestProgress;
@@ -43,8 +42,6 @@ public class MineBlockObjective extends Objective {
 		@EventHandler(priority = EventPriority.HIGH, ignoreCancelled = true)
 		public void a(BlockBreakEvent event) {
 			if(!getQuestProgress().getPlayer().isOnline()) return;
-			if (UtilityMethods.isFake(event)) return;
-
 			if ((!playerPlaced) && event.getBlock().hasMetadata("player_placed"))
 				return;
 			if (event.getPlayer().equals(getQuestProgress().getPlayer().getPlayer()) && event.getBlock().getType() == block) {

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/trigger/BindSkillTrigger.java

@@ -1,34 +0,0 @@
-package net.Indyuce.mmocore.api.quest.trigger;
-
-import io.lumine.mythic.lib.api.MMOLineConfig;
-import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.api.quest.trigger.api.Removable;
-import net.Indyuce.mmocore.skill.ClassSkill;
-import net.Indyuce.mmocore.skill.RegisteredSkill;
-
-import javax.annotation.Nullable;
-
-public class BindSkillTrigger extends Trigger implements Removable {
-    private final RegisteredSkill skill;
-    private final int slot;
-
-    public BindSkillTrigger(MMOLineConfig config) {
-        super(config);
-
-        config.validateKeys("skill", "slot");
-        slot = config.getInt("slot");
-        skill = MMOCore.plugin.skillManager.getSkillOrThrow(config.getString("skill"));
-    }
-
-    @Override
-    public void apply(PlayerData playerData) {
-        final @Nullable ClassSkill found = playerData.getProfess().getSkill(skill);
-        if (found != null) playerData.bindSkill(slot, found);
-    }
-
-    @Override
-    public void remove(PlayerData playerData) {
-        playerData.unbindSkill(slot);
-    }
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/trigger/CommandTrigger.java

@@ -1,6 +1,5 @@
 package net.Indyuce.mmocore.api.quest.trigger;
 
-import io.lumine.mythic.lib.util.annotation.BackwardsCompatibility;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
 import org.bukkit.Bukkit;
@@ -24,9 +23,7 @@ public class CommandTrigger extends Trigger {
 		Bukkit.dispatchCommand(Bukkit.getConsoleSender(), format(player.getPlayer()));
 	}
 
-	@BackwardsCompatibility(version = "1.12-SNAPSHOT")
 	private String format(Player player) {
-		// TODO remove use of confusing non-PAPI %player% placeholder
 		return MMOCore.plugin.placeholderParser.parse(player, command.replace("%player%", player.getName()));
 	}
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/trigger/ExperienceTrigger.java

@@ -36,6 +36,6 @@ public class ExperienceTrigger extends Trigger {
 
 	@Override
 	public void apply(PlayerData player) {
-		dispenser.giveExperience(player, amount.calculate(), null, source);
+		dispenser.giveExperience(player, amount.calculateInt(), null, source);
 	}
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/trigger/LevelUpSkillTrigger.java

@@ -1,30 +0,0 @@
-package net.Indyuce.mmocore.api.quest.trigger;
-
-import io.lumine.mythic.lib.api.MMOLineConfig;
-import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.api.quest.trigger.api.Removable;
-import net.Indyuce.mmocore.skill.RegisteredSkill;
-
-public class LevelUpSkillTrigger extends Trigger implements Removable {
-    private final RegisteredSkill skill;
-    private final int amount;
-
-    public LevelUpSkillTrigger(MMOLineConfig config) {
-        super(config);
-
-        config.validateKeys("skill", "amount");
-        amount = config.getInt("amount");
-        skill = MMOCore.plugin.skillManager.getSkillOrThrow(config.getString("skill"));
-    }
-
-    @Override
-    public void apply(PlayerData playerData) {
-        playerData.setSkillLevel(skill, playerData.getSkillLevel(skill) + amount);
-    }
-
-    @Override
-    public void remove(PlayerData playerData) {
-        playerData.setSkillLevel(skill, Math.max(0, playerData.getSkillLevel(skill) - amount));
-    }
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/trigger/MessageTrigger.java

@@ -1,6 +1,5 @@
 package net.Indyuce.mmocore.api.quest.trigger;
 
-import io.lumine.mythic.lib.util.annotation.BackwardsCompatibility;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
 import org.bukkit.entity.Player;
@@ -23,9 +22,7 @@ public class MessageTrigger extends Trigger {
 		player.getPlayer().sendMessage(format(player.getPlayer()));
 	}
 
-	@BackwardsCompatibility(version = "1.12-SNAPSHOT")
 	private String format(Player player) {
-		// TODO remove use of confusing non-PAPI %player% placeholder
 		return MMOCore.plugin.placeholderParser.parse(player, message.replace("%player%", player.getName()));
 	}
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/trigger/SkillModifierTrigger.java

@@ -1,73 +0,0 @@
-package net.Indyuce.mmocore.api.quest.trigger;
-
-import io.lumine.mythic.lib.UtilityMethods;
-import io.lumine.mythic.lib.api.MMOLineConfig;
-import io.lumine.mythic.lib.player.modifier.ModifierType;
-import io.lumine.mythic.lib.player.skillmod.SkillModifier;
-import io.lumine.mythic.lib.skill.handler.SkillHandler;
-import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.api.quest.trigger.api.Removable;
-import net.Indyuce.mmocore.api.quest.trigger.api.Temporary;
-import net.Indyuce.mmocore.skill.RegisteredSkill;
-import org.apache.commons.lang.Validate;
-import org.jetbrains.annotations.NotNull;
-
-import java.util.List;
-import java.util.stream.Collectors;
-
-public class SkillModifierTrigger extends Trigger implements Removable, Temporary {
-    private SkillModifier mod;
-    private boolean mutable = true;
-
-    public SkillModifierTrigger(MMOLineConfig config) {
-        super(config);
-
-        config.validateKeys("modifier");
-        config.validateKeys("amount");
-
-        final double amount = config.getDouble("amount");
-        final String parameter = config.getString("modifier");
-        final String formula = config.getString("formula", "true");
-        final ModifierType type = config.contains("type") ? ModifierType.valueOf(UtilityMethods.enumName(config.getString("type"))) : ModifierType.FLAT;
-        final List<SkillHandler<?>> targetSkills = MMOCore.plugin.skillManager.getAll().stream().filter(skill -> skill.matchesFormula(formula)).map(RegisteredSkill::getHandler).collect(Collectors.toList());
-
-        mod = new SkillModifier(Trigger.STAT_MODIFIER_KEY, parameter, targetSkills, amount, type);
-    }
-
-    public void updateKey(@NotNull String key) {
-        Validate.isTrue(mutable, "No longer mutable");
-        this.mod = new SkillModifier(key, mod.getParameter(), mod.getSkills(), mod.getValue(), mod.getType());
-    }
-
-    public List<SkillHandler<?>> getTargetSkills() {
-        return mod.getSkills();
-    }
-
-    @Override
-    public void apply(PlayerData player) {
-        mod.register(player.getMMOPlayerData());
-    }
-
-    @Override
-    public void remove(PlayerData playerData) {
-        mod.unregister(playerData.getMMOPlayerData());
-    }
-
-    /**
-     * Used by skill slots to apply a skill modifier.
-     * to a dynamically chosen skill handler.
-     */
-    public void apply(PlayerData playerData, SkillHandler<?> skill) {
-        mutable = false;
-        mod.register(playerData.getMMOPlayerData(), skill);
-    }
-
-    /**
-     * Used by skill slots to remove a skillBuff
-     * from a dynamically chosen skill handler.
-     */
-    public void remove(PlayerData playerData, SkillHandler<?> skill) {
-        mod.unregister(playerData.getMMOPlayerData(), skill);
-    }
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/trigger/StatTrigger.java

@@ -4,14 +4,16 @@ import io.lumine.mythic.lib.api.MMOLineConfig;
 import io.lumine.mythic.lib.api.stat.modifier.StatModifier;
 import io.lumine.mythic.lib.player.modifier.ModifierType;
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.api.quest.trigger.api.Removable;
-import net.Indyuce.mmocore.api.quest.trigger.api.Temporary;
 import org.apache.commons.lang.Validate;
 
-public class StatTrigger extends Trigger implements Removable, Temporary {
+import java.util.UUID;
-    private final StatModifier modifier;
+
+public class StatTrigger extends Trigger {
     private final String stat;
     private final double amount;
+    private final ModifierType type;
+    private double totalAmount;
+    private final UUID uuid =UUID.randomUUID();
 
     public StatTrigger(MMOLineConfig config) {
         super(config);
@@ -23,18 +25,23 @@ public class StatTrigger extends Trigger implements Removable, Temporary {
         Validate.isTrue(type.equals("FLAT") || type.equals("RELATIVE"));
         stat = config.getString("stat");
         amount = config.getDouble("amount");
-        modifier = new StatModifier(Trigger.STAT_MODIFIER_KEY, stat, amount, ModifierType.valueOf(type));
+        this.type = ModifierType.valueOf(type);
+        this.totalAmount = 0;
     }
 
     @Override
     public void apply(PlayerData player) {
-        StatModifier prevModifier = player.getMMOPlayerData().getStatMap().getInstance(stat).getModifier(modifier.getUniqueId());
+        totalAmount+=amount;
-        if (prevModifier == null) modifier.register(player.getMMOPlayerData());
+        new StatModifier("trigger."+uuid.toString(),stat,totalAmount,type).register(player.getMMOPlayerData());
-        else prevModifier.add(amount).register(player.getMMOPlayerData());
     }
 
-    @Override
+    /**
+     * Removes the effect of the trigger to the player by registering the
+     * opposite amount. (Little corrective term for the relative to have the inverse.
+     * Not a problem to store twice the stat modifiers are there only remain in the RAM.
+     */
     public void remove(PlayerData playerData) {
-        modifier.unregister(playerData.getMMOPlayerData());
+        totalAmount-=amount;
+        new StatModifier("trigger."+uuid.toString(), stat, totalAmount, type).register(playerData.getMMOPlayerData());
     }
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/trigger/Trigger.java

@@ -6,8 +6,6 @@ import net.Indyuce.mmocore.api.player.PlayerData;
 import org.bukkit.Bukkit;
 
 public abstract class Trigger {
-
-	public static String STAT_MODIFIER_KEY = "mmocore_trigger";
 	private final long delay;
 
 	public Trigger(MMOLineConfig config) {

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/trigger/UnlockSkillTrigger.java

@@ -1,33 +0,0 @@
-package net.Indyuce.mmocore.api.quest.trigger;
-
-import io.lumine.mythic.lib.api.MMOLineConfig;
-import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.api.quest.trigger.api.Removable;
-import net.Indyuce.mmocore.skill.ClassSkill;
-import net.Indyuce.mmocore.skill.RegisteredSkill;
-
-import javax.annotation.Nullable;
-
-public class UnlockSkillTrigger extends Trigger implements Removable {
-    private final RegisteredSkill skill;
-
-    public UnlockSkillTrigger(MMOLineConfig config) {
-        super(config);
-
-        config.validateKeys("skill");
-        skill = MMOCore.plugin.skillManager.getSkillOrThrow(config.getString("skill"));
-    }
-
-    @Override
-    public void apply(PlayerData playerData) {
-        final @Nullable ClassSkill found = playerData.getProfess().getSkill(skill);
-        if (found != null) playerData.unlock(found);
-    }
-
-    @Override
-    public void remove(PlayerData playerData) {
-        final @Nullable ClassSkill found = playerData.getProfess().getSkill(skill);
-        if (found != null) playerData.lock(found);
-    }
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/trigger/UnlockSlotTrigger.java

@@ -1,35 +0,0 @@
-package net.Indyuce.mmocore.api.quest.trigger;
-
-import io.lumine.mythic.lib.api.MMOLineConfig;
-import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.skill.binding.SkillSlot;
-import net.Indyuce.mmocore.api.quest.trigger.api.Removable;
-import org.apache.commons.lang.Validate;
-
-
-public class UnlockSlotTrigger extends Trigger implements Removable {
-    private final int slot;
-
-    public UnlockSlotTrigger(MMOLineConfig config) {
-        super(config);
-
-        config.validateKeys("slot");
-        try {
-            slot = Integer.parseInt(config.getString("slot"));
-        } catch (NumberFormatException exception) {
-            throw new IllegalArgumentException("Slot should be a number");
-        }
-        Validate.isTrue(slot > 0, "Slot number must be positive");
-    }
-
-    @Override
-    public void apply(PlayerData player) {
-        player.unlock(player.getProfess().getSkillSlot(slot));
-    }
-
-    @Override
-    public void remove(PlayerData player) {
-        player.lock(player.getProfess().getSkillSlot(slot));
-    }
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/trigger/api/Removable.java

@@ -1,13 +0,0 @@
-package net.Indyuce.mmocore.api.quest.trigger.api;
-
-import net.Indyuce.mmocore.api.player.PlayerData;
-
-/**
- * Cancelable triggers cause problems when letting the player reset
- * their advancement on things they can spend points in/level up.
- * If you give access to some resource to the player via a trigger,
- * you must take it away when resetting their progression.
- */
-public interface Removable {
-    public void remove(PlayerData playerData);
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/quest/trigger/api/Temporary.java

@@ -1,12 +0,0 @@
-package net.Indyuce.mmocore.api.quest.trigger.api;
-
-/**
- * Non-permanent triggers are triggers which are not saved
- * by the player and taken off when the player logs off,
- * for instance temporary player modifiers. They need to
- * be re-applied everytime the player logs back.
- *
- * @author jules
- */
-public interface Temporary extends Removable {
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/util/Closable.java

@@ -0,0 +1,15 @@
+package net.Indyuce.mmocore.api.util;
+
+/**
+ * Indicates that a class temporarily registers something
+ * such as a Bukkit event, which needs to be unregistered
+ * when the class is finally garbage collected.
+ */
+public interface Closable {
+
+    /**
+     * Method that must be called before the class
+     * is garbage collected
+     */
+    void close();
+}

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/util/MMOCoreUtils.java

@@ -1,14 +1,15 @@
 package net.Indyuce.mmocore.api.util;
 
-import com.google.common.collect.MultimapBuilder;
+import com.google.gson.JsonArray;
+import com.google.gson.JsonObject;
 import io.lumine.mythic.lib.MythicLib;
-import io.lumine.mythic.lib.UtilityMethods;
-import io.lumine.mythic.lib.gson.JsonArray;
-import io.lumine.mythic.lib.gson.JsonObject;
 import io.lumine.mythic.lib.hologram.Hologram;
-import io.lumine.mythic.lib.version.VEnchantment;
+import io.lumine.mythic.lib.version.VersionMaterial;
 import net.Indyuce.mmocore.MMOCore;
-import org.bukkit.*;
+import org.bukkit.Bukkit;
+import org.bukkit.Location;
+import org.bukkit.Material;
+import org.bukkit.Sound;
 import org.bukkit.attribute.Attribute;
 import org.bukkit.entity.Entity;
 import org.bukkit.entity.LivingEntity;
@@ -17,13 +18,11 @@ import org.bukkit.event.entity.EntityRegainHealthEvent;
 import org.bukkit.event.entity.EntityRegainHealthEvent.RegainReason;
 import org.bukkit.event.player.PlayerItemDamageEvent;
 import org.bukkit.inventory.EquipmentSlot;
-import org.bukkit.inventory.ItemFlag;
 import org.bukkit.inventory.ItemStack;
 import org.bukkit.inventory.meta.Damageable;
 import org.bukkit.inventory.meta.ItemMeta;
 import org.bukkit.util.io.BukkitObjectInputStream;
 import org.bukkit.util.io.BukkitObjectOutputStream;
-import org.jetbrains.annotations.NotNull;
 import org.yaml.snakeyaml.external.biz.base64Coder.Base64Coder;
 
 import java.io.ByteArrayInputStream;
@@ -35,37 +34,11 @@ public class MMOCoreUtils {
         return item != null && item.hasItemMeta() && item.getItemMeta().hasDisplayName();
     }
 
-    /**
-     * If a given player is not in the server cache, no information
-     * cannot be retrieved from that player (without using requests
-     * to MC servers obviously). In that case, the instance of
-     * OfflinePlayer is pretty much useless and it only wraps its
-     * UUID which was already known beforehand.
-     *
-     * @param player Offline player instance to test
-     * @return Is the instance valid
-     */
-    public static boolean isInvalid(OfflinePlayer player) {
-        return player.getName() == null;
-    }
-
-    @Deprecated
     public static String displayName(ItemStack item) {
         return item.hasItemMeta() && item.getItemMeta().hasDisplayName() ? item.getItemMeta().getDisplayName()
-                : UtilityMethods.caseOnWords(item.getType().name().replace("_", " "));
+                : caseOnWords(item.getType().name().replace("_", " "));
     }
 
-    /**
-     * @param current Current value of resource
-     * @param maxStat Maximum value of resource
-     * @return Clamped resource value. If the provided current value is 0,
-     * this function will return the maximum resource value.
-     */
-    public static double fixResource(double current, double maxStat) {
-        return current == 0 ? maxStat : Math.max(0, Math.min(current, maxStat));
-    }
-
-    @Deprecated
     public static String caseOnWords(String s) {
         StringBuilder builder = new StringBuilder(s);
         boolean isLastSpace = true;
@@ -80,26 +53,44 @@ public class MMOCoreUtils {
         return builder.toString();
     }
 
-    public static String ymlName(String str) {
+    /**
-        return str.toLowerCase().replace("_", "-").replace(" ", "-");
+     *
-    }
+     * @param value an integer you want to convert
-
+     * @return the string representing the integer but with roman letters
-    @Deprecated
+     */
     public static String toRomanNumerals(int value) {
-        return intToRoman(value);
+        LinkedHashMap<String, Integer> roman_numerals = new LinkedHashMap<String, Integer>();
+        roman_numerals.put("M", 1000);
+        roman_numerals.put("CM", 900);
+        roman_numerals.put("D", 500);
+        roman_numerals.put("CD", 400);
+        roman_numerals.put("C", 100);
+        roman_numerals.put("XC", 90);
+        roman_numerals.put("L", 50);
+        roman_numerals.put("XL", 40);
+        roman_numerals.put("X", 10);
+        roman_numerals.put("IX", 9);
+        roman_numerals.put("V", 5);
+        roman_numerals.put("IV", 4);
+        roman_numerals.put("I", 1);
+        String res = "";
+        for(Map.Entry<String, Integer> entry : roman_numerals.entrySet()){
+            int matches = value/entry.getValue();
+            res += repeat(entry.getKey(), matches);
+            value = value % entry.getValue();
+        }
+        return res;
     }
-
     private static String repeat(String s, int n) {
-        if (s == null) {
+        if(s == null) {
             return null;
         }
         final StringBuilder sb = new StringBuilder();
-        for (int i = 0; i < n; i++) {
+        for(int i = 0; i < n; i++) {
             sb.append(s);
         }
         return sb.toString();
     }
-
     /**
      * Displays an in game indicator using a hologram. This uses
      * LumineUtils hologramFactory to summon holograms
@@ -110,38 +101,18 @@ public class MMOCoreUtils {
      * @param message Message to display
      */
     public static void displayIndicator(Location loc, String message) {
-        Hologram holo = Hologram.create(loc, MythicLib.plugin.parseColors(Collections.singletonList(message)));
+        Hologram holo = Hologram.create(loc, Arrays.asList(message));
-        Bukkit.getScheduler().runTaskLater(MMOCore.plugin, holo::despawn, 20);
+        Bukkit.getScheduler().runTaskLater(MMOCore.plugin, () -> holo.despawn(), 20);
     }
 
     public static boolean isPlayerHead(Material material) {
-        return material == Material.PLAYER_HEAD || material == Material.PLAYER_WALL_HEAD;
+        return material == VersionMaterial.PLAYER_HEAD.toMaterial() || material == VersionMaterial.PLAYER_WALL_HEAD.toMaterial();
     }
 
-    public static void addAllItemFlags(@NotNull ItemMeta meta) {
+    public static ItemStack readIcon(String string) throws IllegalArgumentException {
-        meta.addItemFlags(ItemFlag.values());
+        String[] split = string.split(":");
-
+        Material material = Material.valueOf(split[0].toUpperCase().replace("-", "_").replace(" ", "_"));
-        // Fix 1.20.6+ Paper bug that sucks. HIDE_ATTRIBUTES no longer works when item attribute list is empty
+        return split.length > 1 ? MythicLib.plugin.getVersion().getWrapper().textureItem(material, Integer.parseInt(split[1])) : new ItemStack(material);
-        // TODO refactor with GUI update.
-        try {
-            meta.setAttributeModifiers(MultimapBuilder.hashKeys(0).hashSetValues(0).build());
-        } catch (Exception exception) {
-            // Not needed
-        }
-    }
-
-    @NotNull
-    public static ItemStack readIcon(String string) {
-        final String[] split = string.split(":");
-
-        final ItemStack item = new ItemStack(Material.valueOf(split[0].toUpperCase().replace("-", "_").replace(" ", "_")));
-        if (split.length > 1) {
-            final ItemMeta meta = item.getItemMeta();
-            meta.setCustomModelData(Integer.parseInt(split[1]));
-            item.setItemMeta(meta);
-        }
-
-        return item;
     }
 
     public static int getWorth(ItemStack[] items) {
@@ -257,7 +228,6 @@ public class MMOCoreUtils {
         return entities;
     }
 
-    @Deprecated
     public static void heal(LivingEntity target, double value) {
         double max = target.getAttribute(Attribute.GENERIC_MAX_HEALTH).getValue();
         double gain = Math.min(max, target.getHealth() + value) - target.getHealth();
@@ -268,8 +238,6 @@ public class MMOCoreUtils {
             target.setHealth(target.getHealth() + gain);
     }
 
-    private static final Random RANDOM = new Random();
-
     /**
      * Method used when mining a custom block or fishing, as the corresponding
      * interaction event is cancelled durability is not handled. This method is
@@ -284,27 +252,21 @@ public class MMOCoreUtils {
      * @param damage Damage that needs to be applied
      */
     public static void decreaseDurability(Player player, EquipmentSlot slot, int damage) {
-
+        ItemStack item = player.getInventory().getItem(slot);
-        ItemStack item = UtilityMethods.getHandItem(player, slot);
+        if (item == null || item.getType().getMaxDurability() == 0 || !item.hasItemMeta() || !(item.getItemMeta() instanceof Damageable) || item.getItemMeta().isUnbreakable())
-        if (item == null || item.getType().getMaxDurability() == 0 || item.getItemMeta().isUnbreakable())
             return;
 
-        // Check unbreakable, ignore if necessary
-        final ItemMeta meta = item.getItemMeta();
-        final int unbreakingLevel = meta.getEnchantLevel(VEnchantment.UNBREAKING.get());
-        if (unbreakingLevel > 0 && RANDOM.nextInt(unbreakingLevel + 1) != 0) return;
-
         PlayerItemDamageEvent event = new PlayerItemDamageEvent(player, item, damage);
         Bukkit.getPluginManager().callEvent(event);
         if (event.isCancelled())
             return;
 
-        final int newDamage = event.getDamage() + ((Damageable) meta).getDamage();
+        ItemMeta meta = item.getItemMeta();
-        if (newDamage >= item.getType().getMaxDurability()) {
+        if (event.getDamage() + ((Damageable) meta).getDamage() >= item.getType().getMaxDurability()) {
             player.playSound(player.getLocation(), Sound.ENTITY_ITEM_BREAK, 1F, 1F);
             player.getInventory().setItem(slot, null);
         } else {
-            ((Damageable) meta).setDamage(newDamage);
+            ((Damageable) meta).setDamage(((Damageable) meta).getDamage() + event.getDamage());
             item.setItemMeta(meta);
         }
     }
@@ -315,11 +277,4 @@ public class MMOCoreUtils {
     public static Location getCenterLocation(Entity entity) {
         return entity.getBoundingBox().getCenter().toLocation(entity.getWorld());
     }
-
-    public static void debug(String message) {
-        message = ChatColor.YELLOW + "Debug> " + ChatColor.WHITE + message;
-        for (Player player : Bukkit.getOnlinePlayers())
-            player.sendMessage(message);
-        Bukkit.getConsoleSender().sendMessage(message);
-    }
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/util/input/AnvilGUI.java

@@ -1,7 +1,6 @@
 package net.Indyuce.mmocore.api.util.input;
 
-import io.lumine.mythic.lib.MythicLib;
+import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.ConfigMessage;
 import org.bukkit.Material;
 import org.bukkit.entity.Player;
 import org.bukkit.event.EventHandler;
@@ -13,6 +12,8 @@ import org.bukkit.inventory.ItemStack;
 import org.bukkit.inventory.meta.ItemMeta;
 import org.bukkit.util.Consumer;
 
+import io.lumine.mythic.lib.MythicLib;
+
 @Deprecated
 public class AnvilGUI extends PlayerInput {
 	private final int containerId;
@@ -23,7 +24,7 @@ public class AnvilGUI extends PlayerInput {
 
 		ItemStack paper = new ItemStack(Material.PAPER);
 		ItemMeta paperMeta = paper.getItemMeta();
-		paperMeta.setDisplayName(ConfigMessage.fromKey("player-input.anvil." + type.getLowerCaseName()).asLine());
+		paperMeta.setDisplayName(MMOCore.plugin.configManager.getSimpleMessage("player-input.anvil." + type.getLowerCaseName()).message());
 		paper.setItemMeta(paperMeta);
 
 		MythicLib.plugin.getVersion().getWrapper().handleInventoryCloseEvent(player);

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/util/input/ChatInput.java

@@ -1,7 +1,6 @@
 package net.Indyuce.mmocore.api.util.input;
 
 import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.ConfigMessage;
 import net.Indyuce.mmocore.gui.api.PluginInventory;
 import org.bukkit.Bukkit;
 import org.bukkit.entity.Player;
@@ -38,7 +37,8 @@ public class ChatInput extends PlayerInput {
         this.lastOpened = lastOpened;
 
         player.closeInventory();
-        ConfigMessage.fromKey("player-input.chat." + inputType.getLowerCaseName()).send(player);
+        MMOCore.plugin.configManager.getSimpleMessage("player-input.chat." + inputType.getLowerCaseName()).send(player);
+        MMOCore.plugin.configManager.getSimpleMessage("player-input.chat.cancel").send(player);
     }
 
     @Override
@@ -56,7 +56,7 @@ public class ChatInput extends PlayerInput {
             if (event.getMessage().equals("cancel")) {
                 if (lastOpened != null)
                     Bukkit.getScheduler().runTask(MMOCore.plugin, () -> lastOpened.open());
-                ConfigMessage.fromKey("player-input.chat." + inputType.getLowerCaseName() + "-cancel").send(getPlayer());
+                MMOCore.plugin.configManager.getSimpleMessage("player-input.chat." + inputType.getLowerCaseName() + "-cancel").send(getPlayer());
             } else
                 // Run sync
                 Bukkit.getScheduler().runTask(MMOCore.plugin, () -> output(event.getMessage()));

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/util/input/PlayerInput.java

@@ -29,6 +29,7 @@ public abstract class PlayerInput implements Listener {
 
 	public enum InputType {
 		FRIEND_REQUEST,
+
 		PARTY_INVITE,
 		GUILD_INVITE,
 		GUILD_CREATION_TAG,

MMOCore-API/src/main/java/net/Indyuce/mmocore/api/util/math/formula/LinearValue.java

@@ -2,13 +2,7 @@ package net.Indyuce.mmocore.api.util.math.formula;
 
 import io.lumine.mythic.lib.MythicLib;
 import org.bukkit.configuration.ConfigurationSection;
-import org.jetbrains.annotations.NotNull;
 
-/**
- * Bounded linear formula.
- *
- * @author Jules
- */
 public class LinearValue {
     private final double base, perLevel, min, max;
     private final boolean hasMin, hasMax;
@@ -103,7 +97,6 @@ public class LinearValue {
         return hasMin;
     }
 
-    @NotNull
     public String getDisplay(int level) {
         return MythicLib.plugin.getMMOConfig().decimals.format(calculate(level));
     }

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/AttributesCommand.java

@@ -1,26 +1,25 @@
 package net.Indyuce.mmocore.command;
 
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.command.api.RegisteredCommand;
-import net.Indyuce.mmocore.command.api.ToggleableCommand;
 import net.Indyuce.mmocore.manager.InventoryManager;
 import net.Indyuce.mmocore.api.event.MMOCommandEvent;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
+import org.bukkit.command.defaults.BukkitCommand;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.entity.Player;
 
-public class AttributesCommand extends RegisteredCommand {
+public class AttributesCommand extends BukkitCommand {
 	public AttributesCommand(ConfigurationSection config) {
-		super(config, ToggleableCommand.ATTRIBUTES);
+		super(config.getString("main"));
+		
+		setAliases(config.getStringList("aliases"));
+		setDescription("Opens the attribute menu.");
 	}
 
 	@Override
 	public boolean execute(CommandSender sender, String label, String[] args) {
-		if (!sender.hasPermission("mmocore.attributes"))
-			return false;
-
 		if (!(sender instanceof Player)) {
 			sender.sendMessage(ChatColor.RED + "This command is for players only.");
 			return true;

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/ClassCommand.java

@@ -1,19 +1,21 @@
 package net.Indyuce.mmocore.command;
 
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.command.api.RegisteredCommand;
-import net.Indyuce.mmocore.command.api.ToggleableCommand;
 import net.Indyuce.mmocore.manager.InventoryManager;
 import net.Indyuce.mmocore.api.event.MMOCommandEvent;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
+import org.bukkit.command.defaults.BukkitCommand;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.entity.Player;
 
-public class ClassCommand extends RegisteredCommand {
+public class ClassCommand extends BukkitCommand {
 	public ClassCommand(ConfigurationSection config) {
-		super(config, ToggleableCommand.CLASS);
+		super(config.getString("main"));
+		
+		setAliases(config.getStringList("aliases"));
+		setDescription("Select a new class.");
 	}
 
 	@Override

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/CommandVerbose.java

@@ -1,4 +1,4 @@
-package net.Indyuce.mmocore.command.api;
+package net.Indyuce.mmocore.command;
 
 import io.lumine.mythic.lib.UtilityMethods;
 import net.Indyuce.mmocore.MMOCore;

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/DepositCommand.java

@@ -1,17 +1,19 @@
 package net.Indyuce.mmocore.command;
 
-import net.Indyuce.mmocore.command.api.RegisteredCommand;
-import net.Indyuce.mmocore.command.api.ToggleableCommand;
 import net.Indyuce.mmocore.gui.eco.DepositMenu;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
+import org.bukkit.command.defaults.BukkitCommand;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.entity.Player;
 
-public class DepositCommand extends RegisteredCommand {
+public class DepositCommand extends BukkitCommand {
 	public DepositCommand(ConfigurationSection config) {
-		super(config, ToggleableCommand.DEPOSIT);
+		super(config.getString("main"));
+		
+		setAliases(config.getStringList("aliases"));
+		setDescription("Opens the currency deposit menu.");
 	}
 
 	@Override
@@ -27,7 +29,7 @@ public class DepositCommand extends RegisteredCommand {
 
 		// if (sender instanceof Player)
 		// if (!isNearEnderchest(((Player) sender).getLocation())) {
-		// sender.sendMessage(ConfigMessage.fromKey("stand-near-enderchest"));
+		// sender.sendMessage(MMOCore.plugin.configManager.getSimpleMessage("stand-near-enderchest"));
 		// return true;
 		// }
 

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/FriendsCommand.java

@@ -3,8 +3,6 @@ package net.Indyuce.mmocore.command;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
 import net.Indyuce.mmocore.api.player.social.FriendRequest;
-import net.Indyuce.mmocore.command.api.RegisteredCommand;
-import net.Indyuce.mmocore.command.api.ToggleableCommand;
 import net.Indyuce.mmocore.manager.InventoryManager;
 import net.Indyuce.mmocore.api.event.MMOCommandEvent;
 import net.Indyuce.mmocore.api.player.social.Request;
@@ -12,21 +10,23 @@ import org.apache.commons.lang.Validate;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
+import org.bukkit.command.defaults.BukkitCommand;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.entity.Player;
 import org.jetbrains.annotations.Nullable;
 
 import java.util.UUID;
 
-public class FriendsCommand extends RegisteredCommand {
+public class FriendsCommand extends BukkitCommand {
     public FriendsCommand(ConfigurationSection config) {
-        super(config, ToggleableCommand.FRIENDS);
+        super(config.getString("main"));
+
+        setAliases(config.getStringList("aliases"));
+        setDescription("Opens the friends menu.");
     }
 
     @Override
     public boolean execute(CommandSender sender, String label, String[] args) {
-        if (!sender.hasPermission("mmocore.friends"))
-            return false;
         if (!(sender instanceof Player)) {
             sender.sendMessage(ChatColor.RED + "This command is for players only.");
             return true;

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/GuildCommand.java

@@ -2,8 +2,6 @@ package net.Indyuce.mmocore.command;
 
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.command.api.RegisteredCommand;
-import net.Indyuce.mmocore.command.api.ToggleableCommand;
 import net.Indyuce.mmocore.manager.InventoryManager;
 import net.Indyuce.mmocore.api.event.MMOCommandEvent;
 import net.Indyuce.mmocore.api.player.social.Request;
@@ -13,21 +11,24 @@ import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.OfflinePlayer;
 import org.bukkit.command.CommandSender;
+import org.bukkit.command.defaults.BukkitCommand;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.entity.Player;
 import org.jetbrains.annotations.Nullable;
 
 import java.util.UUID;
 
-public class GuildCommand extends RegisteredCommand {
+public class GuildCommand extends BukkitCommand {
+
     public GuildCommand(ConfigurationSection config) {
-        super(config, ToggleableCommand.GUILD);
+        super(config.getString("main"));
+
+        setAliases(config.getStringList("aliases"));
+        setDescription("Opens the guilds menu.");
     }
 
     @Override
     public boolean execute(CommandSender sender, String label, String[] args) {
-        if (!sender.hasPermission("mmocore.guild"))
-            return false;
         if (!(sender instanceof Player)) {
             sender.sendMessage(ChatColor.RED + "This command is for players only.");
             return true;
@@ -49,7 +50,7 @@ public class GuildCommand extends RegisteredCommand {
                     final Request req = MMOCore.plugin.requestManager.getRequest(uuid);
                     Validate.isTrue(!req.isTimedOut() && req instanceof GuildInvite);
                     invite = (GuildInvite) req;
-                    Validate.isTrue(MMOCore.plugin.nativeGuildManager.isRegistered(invite.getGuild()));
+                    Validate.isTrue(MMOCore.plugin.dataProvider.getGuildManager().isRegistered(invite.getGuild()));
                 } catch (Exception exception) {
                     return true;
                 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/PartyCommand.java

@@ -2,8 +2,6 @@ package net.Indyuce.mmocore.command;
 
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.command.api.RegisteredCommand;
-import net.Indyuce.mmocore.command.api.ToggleableCommand;
 import net.Indyuce.mmocore.manager.InventoryManager;
 import net.Indyuce.mmocore.api.event.MMOCommandEvent;
 import net.Indyuce.mmocore.api.player.social.Request;
@@ -14,29 +12,28 @@ import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.OfflinePlayer;
 import org.bukkit.command.CommandSender;
+import org.bukkit.command.defaults.BukkitCommand;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.entity.Player;
 import org.jetbrains.annotations.Nullable;
 
 import java.util.UUID;
 
-public class PartyCommand extends RegisteredCommand {
+public class PartyCommand extends BukkitCommand {
+
     public PartyCommand(ConfigurationSection config) {
-        super(config, ToggleableCommand.PARTY);
+        super(config.getString("main"));
+
+        setAliases(config.getStringList("aliases"));
+        setDescription("Opens the party menu.");
     }
 
     @Override
     public boolean execute(CommandSender sender, String label, String[] args) {
-        if (!sender.hasPermission("mmocore.party"))
-            return false;
         if (!(sender instanceof Player)) {
             sender.sendMessage(ChatColor.RED + "This command is for players only.");
             return true;
         }
-        if(!(MMOCore.plugin.partyModule instanceof MMOCorePartyModule)){
-            sender.sendMessage(ChatColor.RED+"You can't use MMOCore party system as you delegated the party system to another plugin.");
-            return true;
-        }
 
         PlayerData data = PlayerData.get((OfflinePlayer) sender);
         MMOCommandEvent event = new MMOCommandEvent(data, "party");

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/PlayerStatsCommand.java

@@ -1,25 +1,25 @@
 package net.Indyuce.mmocore.command;
 
-import net.Indyuce.mmocore.api.event.MMOCommandEvent;
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.command.api.RegisteredCommand;
-import net.Indyuce.mmocore.command.api.ToggleableCommand;
 import net.Indyuce.mmocore.manager.InventoryManager;
+import net.Indyuce.mmocore.api.event.MMOCommandEvent;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
+import org.bukkit.command.defaults.BukkitCommand;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.entity.Player;
 
-public class PlayerStatsCommand extends RegisteredCommand {
+public class PlayerStatsCommand extends BukkitCommand {
 	public PlayerStatsCommand(ConfigurationSection config) {
-        super(config, ToggleableCommand.PLAYER);
+		super(config.getString("main"));
+		
+		setAliases(config.getStringList("aliases"));
+		setDescription("Show player stats.");
 	}
 
 	@Override
 	public boolean execute(CommandSender sender, String label, String[] args) {
-        if (!sender.hasPermission("mmocore.profile"))
-            return false;
 		if (!(sender instanceof Player)) {
 			sender.sendMessage(ChatColor.RED + "This command is for players only.");
 			return true;
@@ -28,7 +28,7 @@ public class PlayerStatsCommand extends RegisteredCommand {
 		PlayerData data = PlayerData.get((Player) sender);
 		MMOCommandEvent event = new MMOCommandEvent(data, "profile");
 		Bukkit.getServer().getPluginManager().callEvent(event);
-        if (!event.isCancelled()) InventoryManager.PLAYER_STATS.newInventory(data).open();
+		if(!event.isCancelled()) InventoryManager.PLAYER_STATS.newInventory(data).open();
 		return true;
 	}
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/PvpModeCommand.java

@@ -1,57 +0,0 @@
-package net.Indyuce.mmocore.command;
-
-import io.lumine.mythic.lib.MythicLib;
-import io.lumine.mythic.lib.comp.flags.CustomFlag;
-import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.ConfigMessage;
-import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.command.api.RegisteredCommand;
-import net.Indyuce.mmocore.command.api.ToggleableCommand;
-import org.bukkit.ChatColor;
-import org.bukkit.command.CommandSender;
-import org.bukkit.configuration.ConfigurationSection;
-import org.bukkit.entity.Player;
-
-public class PvpModeCommand extends RegisteredCommand {
-    public PvpModeCommand(ConfigurationSection config) {
-        super(config, ToggleableCommand.PVP_MODE);
-    }
-
-    public static final String COOLDOWN_KEY = "PvpMode";
-
-    @Override
-    public boolean execute(CommandSender sender, String label, String[] args) {
-        if (!(sender instanceof Player)) {
-            sender.sendMessage(ChatColor.RED + "This command is for players only.");
-            return false;
-        }
-
-        if (!sender.hasPermission("mmocore.pvpmode")) {
-            ConfigMessage.fromKey("not-enough-perms").send((Player) sender);
-            return false;
-        }
-
-        final PlayerData playerData = PlayerData.get((Player) sender);
-
-        // Command cooldown
-        if (playerData.getCooldownMap().isOnCooldown(COOLDOWN_KEY)) {
-            ConfigMessage.fromKey("pvp-mode.cooldown", "remaining", MythicLib.plugin.getMMOConfig().decimal.format(playerData.getCooldownMap().getCooldown(COOLDOWN_KEY))).send((Player) sender);
-            return true;
-        }
-
-        playerData.getCombat().setPvpMode(!playerData.getCombat().isInPvpMode());
-        playerData.getCooldownMap().applyCooldown(COOLDOWN_KEY, playerData.getCombat().isInPvpMode() ? MMOCore.plugin.configManager.pvpModeToggleOnCooldown : MMOCore.plugin.configManager.pvpModeToggleOffCooldown);
-
-        // Toggling on when in PVP region
-        if (playerData.getCombat().isInPvpMode() &&
-                MythicLib.plugin.getFlags().isFlagAllowed(playerData.getPlayer(), CustomFlag.PVP_MODE)) {
-            playerData.getCombat().setInvulnerable(MMOCore.plugin.configManager.pvpModeInvulnerabilityTimeCommand);
-            ConfigMessage.fromKey("pvp-mode.toggle.on-invulnerable", "time",
-                    MythicLib.plugin.getMMOConfig().decimal.format(MMOCore.plugin.configManager.pvpModeInvulnerabilityTimeCommand)).send(playerData.getPlayer());
-
-            // Just send message otherwise
-        } else
-            ConfigMessage.fromKey("pvp-mode.toggle." + (playerData.getCombat().isInPvpMode() ? "on" : "off") + "-safe").send((Player) sender);
-        return true;
-    }
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/QuestsCommand.java

@@ -1,24 +1,24 @@
 package net.Indyuce.mmocore.command;
 
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.command.api.RegisteredCommand;
-import net.Indyuce.mmocore.command.api.ToggleableCommand;
 import net.Indyuce.mmocore.manager.InventoryManager;
 import net.Indyuce.mmocore.api.event.MMOCommandEvent;
 import org.bukkit.Bukkit;
 import org.bukkit.command.CommandSender;
+import org.bukkit.command.defaults.BukkitCommand;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.entity.Player;
 
-public class QuestsCommand extends RegisteredCommand {
+public class QuestsCommand extends BukkitCommand {
 	public QuestsCommand(ConfigurationSection config) {
-		super(config, ToggleableCommand.QUESTS);
+		super(config.getString("main"));
+		
+		setAliases(config.getStringList("aliases"));
+		setDescription("Opens the quests menu.");
 	}
 
 	@Override
 	public boolean execute(CommandSender sender, String label, String[] args) {
-		if (!sender.hasPermission("mmocore.quests"))
-			return false;
 		if (sender instanceof Player) {
 			PlayerData data = PlayerData.get((Player) sender);
 			MMOCommandEvent event = new MMOCommandEvent(data, "quests");

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/SkillTreeCommand.java

@@ -0,0 +1,44 @@
+package net.Indyuce.mmocore.command;
+
+import net.Indyuce.mmocore.MMOCore;
+import net.Indyuce.mmocore.api.ConfigMessage;
+import net.Indyuce.mmocore.api.player.PlayerData;
+import net.Indyuce.mmocore.api.event.MMOCommandEvent;
+import net.Indyuce.mmocore.manager.InventoryManager;
+import org.bukkit.Bukkit;
+import org.bukkit.command.CommandSender;
+import org.bukkit.command.defaults.BukkitCommand;
+import org.bukkit.configuration.ConfigurationSection;
+import org.bukkit.entity.Player;
+import org.jetbrains.annotations.NotNull;
+
+public class SkillTreeCommand extends BukkitCommand {
+    public SkillTreeCommand(ConfigurationSection config) {
+        super(config.getString("main"));
+
+        setAliases(config.getStringList("aliases"));
+        setDescription("Opens the skills menu.");
+    }
+
+    @Override
+    public boolean execute(@NotNull CommandSender sender, String s, String[] args) {
+        if (!(sender instanceof Player player))
+            return false;
+        PlayerData data = PlayerData.get(player);
+        MMOCommandEvent event = new MMOCommandEvent(data, "skilltrees");
+        Bukkit.getServer().getPluginManager().callEvent(event);
+        if (event.isCancelled())
+            return true;
+        if (data.getProfess().getSkillTrees().size() != 0) {
+            InventoryManager.TREE_VIEW.newInventory(data).open();
+            return false;
+        }
+        else {
+            MMOCore.plugin.configManager.getSimpleMessage("no-skill-tree").send(player);
+            return true;
+        }
+
+    }
+
+
+}

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/SkillTreesCommand.java

@@ -1,77 +0,0 @@
-package net.Indyuce.mmocore.command;
-
-import io.lumine.mythic.lib.UtilityMethods;
-import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.ConfigMessage;
-import net.Indyuce.mmocore.api.event.MMOCommandEvent;
-import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.command.api.RegisteredCommand;
-import net.Indyuce.mmocore.command.api.ToggleableCommand;
-import net.Indyuce.mmocore.manager.InventoryManager;
-import org.bukkit.Bukkit;
-import org.bukkit.ChatColor;
-import org.bukkit.command.CommandSender;
-import org.bukkit.configuration.ConfigurationSection;
-import org.bukkit.entity.Player;
-import org.jetbrains.annotations.NotNull;
-
-import java.util.stream.Collectors;
-
-public class SkillTreesCommand extends RegisteredCommand {
-    public SkillTreesCommand(ConfigurationSection config) {
-        super(config, ToggleableCommand.SKILL_TREES);
-    }
-
-    @Override
-    public boolean execute(@NotNull CommandSender sender, String s, String[] args) {
-        if (!sender.hasPermission("mmocore.skilltrees"))
-            return false;
-        if (!(sender instanceof Player))
-            return false;
-        final Player player = (Player) sender;
-        PlayerData data = PlayerData.get(player);
-        MMOCommandEvent event = new MMOCommandEvent(data, "skilltrees");
-        Bukkit.getServer().getPluginManager().callEvent(event);
-        if (event.isCancelled())
-            return false;
-
-        // Default skilltree command
-        if (args.length == 0) {
-            if (!MMOCore.plugin.configManager.enableGlobalSkillTreeGUI) {
-                sender.sendMessage(ChatColor.RED + "Usage: /skilltrees <skilltree_id>");
-                return false;
-            }
-            if (data.getProfess().getSkillTrees().size() != 0) {
-                InventoryManager.TREE_VIEW.newInventory(data).open();
-                return true;
-            } else {
-                ConfigMessage.fromKey("no-skill-tree").send(player);
-                return false;
-            }
-        }
-        if (args.length == 1) {
-            if (!MMOCore.plugin.configManager.enableSpecificSkillTreeGUI) {
-                sender.sendMessage(ChatColor.RED + "Usage: /skilltrees <skilltree-id>");
-                return false;
-            }
-
-            if (data.getProfess().getSkillTrees()
-                    .stream()
-                    .filter(skillTree -> UtilityMethods.ymlName(skillTree.getId()).equals(UtilityMethods.ymlName(args[0])))
-                    .collect(Collectors.toList())
-                    .size() != 0) {
-                InventoryManager.SPECIFIC_TREE_VIEW.get(UtilityMethods.ymlName(args[0])).newInventory(data).open();
-                return true;
-            } else {
-                sender.sendMessage(ChatColor.RED + "Your class does not have a skill tree with id: " + args[0]);
-                return false;
-            }
-        } else {
-            if (MMOCore.plugin.configManager.enableSpecificSkillTreeGUI)
-                sender.sendMessage(ChatColor.RED + "Usage: /skilltrees <skilltree-id>");
-            else
-                sender.sendMessage(ChatColor.RED + "Usage: /skilltrees");
-            return false;
-        }
-    }
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/SkillsCommand.java

@@ -1,33 +1,33 @@
 package net.Indyuce.mmocore.command;
 
-import net.Indyuce.mmocore.api.ConfigMessage;
+import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.event.MMOCommandEvent;
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.command.api.RegisteredCommand;
-import net.Indyuce.mmocore.command.api.ToggleableCommand;
 import net.Indyuce.mmocore.manager.InventoryManager;
+import net.Indyuce.mmocore.api.event.MMOCommandEvent;
 import org.bukkit.Bukkit;
 import org.bukkit.command.CommandSender;
+import org.bukkit.command.defaults.BukkitCommand;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.entity.Player;
 
-public class SkillsCommand extends RegisteredCommand {
+public class SkillsCommand extends BukkitCommand {
 	public SkillsCommand(ConfigurationSection config) {
-		super(config, ToggleableCommand.SKILLS);
+		super(config.getString("main"));
+		
+		setAliases(config.getStringList("aliases"));
+		setDescription("Opens the skills menu.");
 	}
 
 	@Override
 	public boolean execute(CommandSender sender, String label, String[] args) {
-		if (!sender.hasPermission("mmocore.skills"))
-			return false;
 		if (sender instanceof Player) {
 			PlayerData data = PlayerData.get((Player) sender);
 			MMOCommandEvent event = new MMOCommandEvent(data, "skills");
 			Bukkit.getServer().getPluginManager().callEvent(event);
 			if(event.isCancelled()) return true;
 			
-			if (data.getUnlockedSkills().isEmpty()) {
+			if (data.getProfess().getSkills().size() < 1) {
-				ConfigMessage.fromKey("no-class-skill").send((Player) sender);
+				MMOCore.plugin.configManager.getSimpleMessage("no-class-skill").send((Player) sender);
 				return true;
 			}
 

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/WaypointsCommand.java

@@ -1,18 +1,20 @@
 package net.Indyuce.mmocore.command;
 
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.command.api.RegisteredCommand;
-import net.Indyuce.mmocore.command.api.ToggleableCommand;
 import net.Indyuce.mmocore.manager.InventoryManager;
 import net.Indyuce.mmocore.api.event.MMOCommandEvent;
 import org.bukkit.Bukkit;
 import org.bukkit.command.CommandSender;
+import org.bukkit.command.defaults.BukkitCommand;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.entity.Player;
 
-public class WaypointsCommand extends RegisteredCommand {
+public class WaypointsCommand extends BukkitCommand {
 	public WaypointsCommand(ConfigurationSection config) {
-		super(config, ToggleableCommand.WAYPOINTS);
+		super(config.getString("main"));
+		
+		setAliases(config.getStringList("aliases"));
+		setDescription("Open the waypoints menu.");
 	}
 
 	@Override

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/WithdrawCommand.java

@@ -1,21 +1,23 @@
 package net.Indyuce.mmocore.command;
 
 import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.ConfigMessage;
-import net.Indyuce.mmocore.api.eco.Withdraw;
-import net.Indyuce.mmocore.command.api.RegisteredCommand;
-import net.Indyuce.mmocore.command.api.ToggleableCommand;
 import org.apache.commons.lang.Validate;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.Sound;
 import org.bukkit.command.CommandSender;
+import org.bukkit.command.defaults.BukkitCommand;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.entity.Player;
 
-public class WithdrawCommand extends RegisteredCommand {
+import net.Indyuce.mmocore.api.eco.Withdraw;
+
+public class WithdrawCommand extends BukkitCommand {
 	public WithdrawCommand(ConfigurationSection config) {
-		super(config, ToggleableCommand.WITHDRAW);
+		super(config.getString("main"));
+		
+		setAliases(config.getStringList("aliases"));
+		setDescription("Creates a withdraw request.");
 	}
 
 	@Override
@@ -37,7 +39,7 @@ public class WithdrawCommand extends RegisteredCommand {
 			amount = Integer.parseInt(amountArgument);
 			Validate.isTrue(amount >= 0);
 		} catch (IllegalArgumentException exception) {
-			sender.sendMessage(ConfigMessage.fromKey("wrong-number", "arg", "" + args[0]).asLine());
+			sender.sendMessage(MMOCore.plugin.configManager.getSimpleMessage("wrong-number", "arg", "" + args[0]).message());
 			return true;
 		}
 
@@ -50,14 +52,14 @@ public class WithdrawCommand extends RegisteredCommand {
 
 		int left = (int) MMOCore.plugin.economy.getEconomy().getBalance(player) - amount;
 		if (left < 0) {
-			ConfigMessage.fromKey("not-enough-money", "left", "" + -left).send(player);
+			MMOCore.plugin.configManager.getSimpleMessage("not-enough-money", "left", "" + -left).send(player);
 			return true;
 		}
 
 		MMOCore.plugin.economy.getEconomy().withdrawPlayer(player, amount);
 		request.withdrawAlgorythm(amount);
 		player.playSound(player.getLocation(), Sound.ENTITY_PLAYER_LEVELUP, 1, 1);
-		ConfigMessage.fromKey("withdrew", "worth", amount).send(player);
+		MMOCore.plugin.configManager.getSimpleMessage("withdrew", "worth", "" + amount).send(player);
 		return true;
 	}
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/api/RegisteredCommand.java

@@ -1,14 +0,0 @@
-package net.Indyuce.mmocore.command.api;
-
-import org.bukkit.command.defaults.BukkitCommand;
-import org.bukkit.configuration.ConfigurationSection;
-import org.jetbrains.annotations.NotNull;
-
-public abstract class RegisteredCommand extends BukkitCommand {
-    public RegisteredCommand(@NotNull ConfigurationSection config, ToggleableCommand command) {
-        super(config.getString("main"));
-
-        setAliases(config.getStringList("aliases"));
-        setDescription(config.getString("description", command.getDescription()));
-    }
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/api/ToggleableCommand.java

@@ -1,112 +0,0 @@
-package net.Indyuce.mmocore.command.api;
-
-import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.ConfigFile;
-import net.Indyuce.mmocore.command.*;
-import org.bukkit.Bukkit;
-import org.bukkit.command.CommandMap;
-import org.bukkit.configuration.ConfigurationSection;
-import org.bukkit.configuration.file.FileConfiguration;
-import org.jetbrains.annotations.NotNull;
-import org.jetbrains.annotations.Nullable;
-
-import java.io.File;
-import java.lang.reflect.Field;
-import java.util.Arrays;
-import java.util.List;
-import java.util.function.Function;
-import java.util.function.Predicate;
-import java.util.logging.Level;
-
-/**
- * Commands which can be disabled using commands.yml
- */
-public enum ToggleableCommand {
-    PLAYER("player", "Displays player stats", config -> new PlayerStatsCommand(config), "p", "profile"),
-    ATTRIBUTES("attributes", "Display and manage attributes", config -> new AttributesCommand(config), "att", "stats"),
-    CLASS("class", "Select a new class", config -> new ClassCommand(config), "c"),
-    WAYPOINTS("waypoints", "Display discovered waypoints", config -> new WaypointsCommand(config), "wp"),
-    QUESTS("quests", "Display available quests", config -> new QuestsCommand(config), "q", "journal"),
-    SKILLS("skills", "Spend skill points to unlock new skills", config -> new SkillsCommand(config), "s"),
-    FRIENDS("friends", "Show online/offline friends", config -> new FriendsCommand(config), "f"),
-    PARTY("party", "Invite players in a party to split exp", config -> new PartyCommand(config)),
-    GUILD("guild", "Show players in current guild", config -> new GuildCommand(config)),
-    WITHDRAW("withdraw", "Withdraw money into coins and notes", config -> new WithdrawCommand(config), v -> MMOCore.plugin.hasEconomy() && MMOCore.plugin.economy.isValid(), "w"),
-    SKILL_TREES("skilltrees", "Open up the skill tree menu", config -> new SkillTreesCommand(config), "st", "trees", "tree"),
-    DEPOSIT("deposit", "Open the currency deposit menu", config -> new DepositCommand(config), "d"),
-    PVP_MODE("pvpmode", "Toggle on/off PVP mode.", config -> new PvpModeCommand(config), "pvp");
-
-    private final String mainLabel;
-    private final String description;
-    private final Function<ConfigurationSection, RegisteredCommand> generator;
-    private final List<String> aliases;
-    private final Predicate<Void> enabled;
-
-    ToggleableCommand(@NotNull String mainLabel, @NotNull String description, @NotNull Function<ConfigurationSection, RegisteredCommand> generator, @NotNull String... aliases) {
-        this(mainLabel, description, generator, null, aliases);
-    }
-
-    ToggleableCommand(@NotNull String mainLabel, @NotNull String description, @NotNull Function<ConfigurationSection, RegisteredCommand> generator, @Nullable Predicate<Void> enabled, @NotNull String... aliases) {
-        this.mainLabel = mainLabel;
-        this.description = description;
-        this.generator = generator;
-        this.aliases = Arrays.asList(aliases);
-        this.enabled = enabled == null ? v -> true : enabled;
-    }
-
-    public String getMainLabel() {
-        return mainLabel;
-    }
-
-    public String getDescription() {
-        return description;
-    }
-
-    public List<String> getAliases() {
-        return aliases;
-    }
-
-    public String getConfigPath() {
-        return name().toLowerCase().replace("_", "-");
-    }
-
-    public boolean isEnabled() {
-        return enabled.test(null);
-    }
-
-    public static void register() {
-
-        // Load default config file
-        if (!new File(MMOCore.plugin.getDataFolder(), "commands.yml").exists()) {
-            final ConfigFile config = new ConfigFile("commands");
-
-            for (ToggleableCommand cmd : values()) {
-                final String path = cmd.getConfigPath();
-                config.getConfig().set(path + ".main", cmd.mainLabel);
-                config.getConfig().set(path + ".aliases", cmd.aliases);
-                config.getConfig().set(path + ".description", cmd.description);
-            }
-
-            config.save();
-        }
-
-
-        try {
-
-            // Find command map
-            final Field commandMapField = Bukkit.getServer().getClass().getDeclaredField("commandMap");
-            commandMapField.setAccessible(true);
-            final CommandMap commandMap = (CommandMap) commandMapField.get(Bukkit.getServer());
-
-            // Enable commands individually
-            final FileConfiguration config = new ConfigFile("commands").getConfig();
-            for (ToggleableCommand cmd : values())
-                if (cmd.isEnabled() && config.contains(cmd.getConfigPath()))
-                    commandMap.register("mmocore", cmd.generator.apply(config.getConfigurationSection(cmd.getConfigPath())));
-
-        } catch (NoSuchFieldException | IllegalArgumentException | IllegalAccessException exception) {
-            MMOCore.plugin.getLogger().log(Level.WARNING, "Unable to register custom commands:");
-            exception.printStackTrace();
-        }
-    }
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/AdminCommandTreeNode.java

@@ -21,15 +21,14 @@ public class AdminCommandTreeNode extends CommandTreeNode {
         addChild(new LevelCommandTreeNode(this));
         addChild(new AttributeCommandTreeNode(this));
         addChild(new SkillCommandTreeNode(this));
-        addChild(new SaveDataTreeNode(this));
+
-        addChild(new SlotCommandTreeNode(this));
         addChild(new PointsCommandTreeNode("skill", this, PlayerData::setSkillPoints, PlayerData::giveSkillPoints, PlayerData::getSkillPoints));
         addChild(new PointsCommandTreeNode("class", this, PlayerData::setClassPoints, PlayerData::giveClassPoints, PlayerData::getClassPoints));
         addChild(new PointsCommandTreeNode("attribute", this, PlayerData::setAttributePoints, PlayerData::giveAttributePoints, PlayerData::getAttributePoints));
         addChild(new PointsCommandTreeNode("attr-realloc", this, PlayerData::setAttributeReallocationPoints, PlayerData::giveAttributeReallocationPoints, PlayerData::getAttributeReallocationPoints));
         addChild(new PointsCommandTreeNode("skill-realloc", this, PlayerData::setSkillReallocationPoints, PlayerData::giveSkillReallocationPoints, PlayerData::getSkillReallocationPoints));
         addChild(new PointsCommandTreeNode("skill-tree-realloc", this, PlayerData::setSkillTreeReallocationPoints, PlayerData::giveSkillTreeReallocationPoints, PlayerData::getSkillTreeReallocationPoints));
-        addChild(new SkillTreePointsCommandTreeNode(this, (playerData, integer, s) -> playerData.setSkillTreePoints(s, integer), (playerData, integer, s) -> playerData.giveSkillTreePoints(s, integer), PlayerData::getSkillTreePoints));
+        addChild(new SkillTreePointsCommandTreeNode(this,(playerData, integer, s) -> playerData.setSkillTreePoints(s,integer),(playerData, integer, s) -> playerData.giveSkillTreePoints(s,integer),((playerData, s) -> playerData.getSkillTreePoint(s))));
         for (PlayerResource res : PlayerResource.values())
             addChild(new ResourceCommandTreeNode(res.name().toLowerCase(), this, res));
     }

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/AttributeCommandTreeNode.java

@@ -4,7 +4,7 @@ import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
 import net.Indyuce.mmocore.api.player.attribute.PlayerAttribute;
 import net.Indyuce.mmocore.api.player.attribute.PlayerAttributes;
-import net.Indyuce.mmocore.command.api.CommandVerbose;
+import net.Indyuce.mmocore.command.CommandVerbose;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/ClassCommandTreeNode.java

@@ -1,20 +1,20 @@
 package net.Indyuce.mmocore.command.rpg.admin;
 
-import io.lumine.mythic.lib.command.api.CommandTreeNode;
-import io.lumine.mythic.lib.command.api.Parameter;
 import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.ConfigMessage;
-import net.Indyuce.mmocore.api.event.PlayerChangeClassEvent;
 import net.Indyuce.mmocore.api.player.PlayerData;
+import net.Indyuce.mmocore.api.event.PlayerChangeClassEvent;
 import net.Indyuce.mmocore.api.player.profess.PlayerClass;
-import net.Indyuce.mmocore.api.player.profess.SavedClassInformation;
-import net.Indyuce.mmocore.command.api.CommandVerbose;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.Sound;
 import org.bukkit.command.CommandSender;
 import org.bukkit.entity.Player;
 
+import net.Indyuce.mmocore.api.player.profess.SavedClassInformation;
+import net.Indyuce.mmocore.command.CommandVerbose;
+import io.lumine.mythic.lib.command.api.CommandTreeNode;
+import io.lumine.mythic.lib.command.api.Parameter;
+
 public class ClassCommandTreeNode extends CommandTreeNode {
 	public ClassCommandTreeNode(CommandTreeNode parent) {
 		super(parent, "class");
@@ -52,9 +52,9 @@ public class ClassCommandTreeNode extends CommandTreeNode {
 			return CommandResult.SUCCESS;
 
 		(data.hasSavedClass(profess) ? data.getClassInfo(profess)
-				: new SavedClassInformation(MMOCore.plugin.playerDataManager.getDefaultData())).load(profess, data);
+				: new SavedClassInformation(MMOCore.plugin.dataProvider.getDataManager().getDefaultData())).load(profess, data);
 		if (data.isOnline()) {
-			ConfigMessage.fromKey("class-select", "class", profess.getName()).send(data.getPlayer());
+			MMOCore.plugin.configManager.getSimpleMessage("class-select", "class", profess.getName()).send(data.getPlayer());
 			data.getPlayer().playSound(data.getPlayer().getLocation(), Sound.UI_TOAST_CHALLENGE_COMPLETE, 1, 1);
 		}
 

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/ExperienceCommandTreeNode.java

@@ -8,7 +8,7 @@ import net.Indyuce.mmocore.api.player.PlayerData;
 import net.Indyuce.mmocore.experience.EXPSource;
 import net.Indyuce.mmocore.experience.PlayerProfessions;
 import net.Indyuce.mmocore.experience.Profession;
-import net.Indyuce.mmocore.command.api.CommandVerbose;
+import net.Indyuce.mmocore.command.CommandVerbose;
 import net.Indyuce.mmocore.command.MMOCoreCommandTreeRoot;
 import net.Indyuce.mmocore.util.TriConsumer;
 import org.apache.commons.lang.Validate;

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/ExportDataTreeNode.java

@@ -1,13 +1,21 @@
 package net.Indyuce.mmocore.command.rpg.admin;
 
+import io.lumine.mythic.lib.api.player.MMOPlayerData;
 import io.lumine.mythic.lib.command.api.CommandTreeNode;
-import io.lumine.mythic.lib.data.DataExport;
-import io.lumine.mythic.lib.data.sql.SQLDataSource;
 import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.manager.data.sql.SQLDataHandler;
+import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.manager.data.yaml.YAMLPlayerDataHandler;
+import net.Indyuce.mmocore.manager.data.mysql.MySQLDataProvider;
 import org.bukkit.command.CommandSender;
-import org.jetbrains.annotations.NotNull;
+import org.bukkit.scheduler.BukkitRunnable;
+
+import java.io.File;
+import java.text.DecimalFormat;
+import java.util.Arrays;
+import java.util.List;
+import java.util.UUID;
+import java.util.logging.Level;
+import java.util.stream.Collectors;
+
 
 /**
  * This command allows to transfer data from your actual storage type
@@ -18,15 +26,82 @@ public class ExportDataTreeNode extends CommandTreeNode {
         super(parent, "exportdata");
     }
 
+    /**
+     * Amount of requests generated every batch
+     */
+    private static final int BATCH_AMOUNT = 50;
+
+    /**
+     * Period in ticks
+     */
+    private static final int BATCH_PERIOD = 20;
+
+    private static final DecimalFormat decFormat = new DecimalFormat("0.#");
+
     @Override
-    @NotNull
     public CommandResult execute(CommandSender sender, String[] strings) {
 
-        // Export YAML to SQL
+        if (!MMOCore.plugin.dataProvider.getDataManager().getLoaded().isEmpty()) {
-        final boolean result = new DataExport<>(MMOCore.plugin.playerDataManager, sender).start(
+            sender.sendMessage("Please make sure no players are logged in when using this command. " +
-                () -> new YAMLPlayerDataHandler(MMOCore.plugin),
+                    "If you are still seeing this message, restart your server and execute this command before any player logs in.");
-                () -> new SQLDataHandler(new SQLDataSource(MMOCore.plugin)));
+            return CommandResult.FAILURE;
+        }
 
-        return result ? CommandResult.SUCCESS : CommandResult.FAILURE;
+        final List<UUID> playerIds = Arrays.stream(new File(MMOCore.plugin.getDataFolder() + "/userdata").listFiles())
+                .map(file -> UUID.fromString(file.getName().replace(".yml", ""))).collect(Collectors.toList());
+
+        // Initialize fake SQL data provider
+        final MySQLDataProvider sqlProvider;
+        try {
+            sqlProvider = new MySQLDataProvider(MMOCore.plugin.getConfig());
+        } catch (RuntimeException exception) {
+            sender.sendMessage("Could not initialize SQL provider (see console for stack trace): " + exception.getMessage());
+            return CommandResult.FAILURE;
+        }
+
+        final double timeEstimation = (double) playerIds.size() / BATCH_AMOUNT * BATCH_PERIOD / 20;
+        sender.sendMessage("Exporting " + playerIds.size() + " player data(s).. See console for details");
+        sender.sendMessage("Minimum expected time: " + decFormat.format(timeEstimation) + "s");
+
+        // Save player data
+        new BukkitRunnable() {
+            int errorCount = 0;
+            int batchCounter = 0;
+
+            @Override
+            public void run() {
+                for (int i = 0; i < BATCH_AMOUNT; i++) {
+                    final int index = BATCH_AMOUNT * batchCounter + i;
+
+                    /*
+                     * Saving is done. Close connection to avoid memory
+                     * leaks and ouput the results to the command executor
+                     */
+                    if (index >= playerIds.size()) {
+                        cancel();
+
+                        sqlProvider.close();
+                        MMOCore.plugin.getLogger().log(Level.WARNING, "Exported " + playerIds.size() + " player datas to SQL database. Total errors: " + errorCount);
+                        return;
+                    }
+
+                    final UUID playerId = playerIds.get(index);
+                    try {
+                        final PlayerData offlinePlayerData = new PlayerData(new MMOPlayerData(playerId));
+                        MMOCore.plugin.dataProvider.getDataManager().loadData(offlinePlayerData);
+
+                        // Player data is loaded, now it gets saved through SQL
+                        sqlProvider.getDataManager().saveData(offlinePlayerData);
+                    } catch (RuntimeException exception) {
+                        errorCount++;
+                        exception.printStackTrace();
+                    }
+                }
+
+                batchCounter++;
+            }
+        }.runTaskTimerAsynchronously(MMOCore.plugin, 0, BATCH_PERIOD);
+
+        return CommandResult.SUCCESS;
     }
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/ForceClassCommandTreeNode.java

@@ -3,7 +3,7 @@ package net.Indyuce.mmocore.command.rpg.admin;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
 import net.Indyuce.mmocore.api.player.profess.PlayerClass;
-import net.Indyuce.mmocore.command.api.CommandVerbose;
+import net.Indyuce.mmocore.command.CommandVerbose;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/HideActionBarCommandTreeNode.java

@@ -36,7 +36,7 @@ public class HideActionBarCommandTreeNode extends CommandTreeNode {
 			return CommandResult.FAILURE;
 		}
 
-		final long lastUsed = System.currentTimeMillis() - PlayerActivity.ACTION_BAR_MESSAGE.getTimeOut() + amount * 50;
+		long lastUsed = System.currentTimeMillis() - PlayerActivity.ACTION_BAR_MESSAGE.getTimeOut() + amount * 50;
 		PlayerData.get(player).setLastActivity(PlayerActivity.ACTION_BAR_MESSAGE, lastUsed);
 		return CommandResult.SUCCESS;
 	}

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/LevelCommandTreeNode.java

@@ -7,7 +7,7 @@ import net.Indyuce.mmocore.api.player.PlayerData;
 import net.Indyuce.mmocore.experience.EXPSource;
 import net.Indyuce.mmocore.experience.PlayerProfessions;
 import net.Indyuce.mmocore.experience.Profession;
-import net.Indyuce.mmocore.command.api.CommandVerbose;
+import net.Indyuce.mmocore.command.CommandVerbose;
 import net.Indyuce.mmocore.util.TriConsumer;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/NoCooldownCommandTreeNode.java

@@ -1,7 +1,7 @@
 package net.Indyuce.mmocore.command.rpg.admin;
 
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.command.api.CommandVerbose;
+import net.Indyuce.mmocore.command.CommandVerbose;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/PointsCommandTreeNode.java

@@ -4,7 +4,7 @@ import java.util.function.BiConsumer;
 import java.util.function.Function;
 
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.command.api.CommandVerbose;
+import net.Indyuce.mmocore.command.CommandVerbose;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/ResetCommandTreeNode.java

@@ -1,33 +1,29 @@
 package net.Indyuce.mmocore.command.rpg.admin;
 
-import io.lumine.mythic.lib.command.api.CommandTreeNode;
-import io.lumine.mythic.lib.command.api.Parameter;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.api.player.attribute.PlayerAttribute;
-import net.Indyuce.mmocore.api.player.attribute.PlayerAttributes;
-import net.Indyuce.mmocore.command.api.CommandVerbose;
 import net.Indyuce.mmocore.experience.Profession;
+import net.Indyuce.mmocore.api.player.attribute.PlayerAttributes;
+import net.Indyuce.mmocore.skilltree.tree.SkillTree;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;
 import org.bukkit.entity.Player;
-import org.jetbrains.annotations.NotNull;
 
-import java.util.HashSet;
+import net.Indyuce.mmocore.command.CommandVerbose;
+import io.lumine.mythic.lib.command.api.CommandTreeNode;
+import io.lumine.mythic.lib.command.api.Parameter;
 
 public class ResetCommandTreeNode extends CommandTreeNode {
     public ResetCommandTreeNode(CommandTreeNode parent) {
         super(parent, "reset");
 
-        addChild(new ResetClassesCommandTreeNode(this));
         addChild(new ResetLevelsCommandTreeNode(this));
         addChild(new ResetSkillsCommandTreeNode(this));
-        addChild(new ResetQuestsCommandTreeNode(this));
+        addChild(new ResetAllCommandTreeNode(this));
         addChild(new ResetAttributesCommandTreeNode(this));
         addChild(new ResetWaypointsCommandTreeNode(this));
         addChild(new ResetSkillTreesCommandTreeNode(this));
-        addChild(new ResetAllCommandTreeNode(this));
     }
 
     @Override
@@ -44,7 +40,8 @@ public class ResetCommandTreeNode extends CommandTreeNode {
 
         @Override
         public CommandResult execute(CommandSender sender, String[] args) {
-            if (args.length < 4) return CommandResult.THROW_USAGE;
+            if (args.length < 4)
+                return CommandResult.THROW_USAGE;
 
             Player player = Bukkit.getPlayer(args[3]);
             if (player == null) {
@@ -52,25 +49,36 @@ public class ResetCommandTreeNode extends CommandTreeNode {
                 return CommandResult.FAILURE;
             }
 
-            final boolean givePoints = args.length > 4 && args[4].equalsIgnoreCase("-reallocate");
-
             PlayerData data = PlayerData.get(player);
-            ResetClassesCommandTreeNode.resetClasses(data);
+            MMOCore.plugin.dataProvider.getDataManager().getDefaultData().apply(data);
-            ResetLevelsCommandTreeNode.resetLevels(data);
+            data.setExperience(0);
-            ResetSkillsCommandTreeNode.resetSkills(data);
+            for (Profession profession : MMOCore.plugin.professionManager.getAll()) {
-            ResetQuestsCommandTreeNode.resetQuests(data);
+                data.getCollectionSkills().setExperience(profession, 0);
-            ResetAttributesCommandTreeNode.resetAttributes(data, givePoints);
+                data.getCollectionSkills().setLevel(profession, 0);
-            ResetWaypointsCommandTreeNode.resetWaypoints(data);
+            }
-            ResetSkillTreesCommandTreeNode.resetSkillTrees(data);
+            MMOCore.plugin.classManager.getAll().forEach(data::unloadClassInfo);
-            // Reset times-claimed not being properly emptied otherwise
+            data.getAttributes().getInstances().forEach(ins -> ins.setBase(0));
-            data.getItemClaims().clear();
+            data.mapSkillLevels().forEach((skill, level) -> data.resetSkillLevel(skill));
-            CommandVerbose.verbose(sender, CommandVerbose.CommandType.RESET, ChatColor.GOLD + player.getName() + ChatColor.YELLOW + "'s data was successfully reset.");
+            data.setSkillTreePoints("global", 0);
+            for (SkillTree skillTree : data.getProfess().getSkillTrees()) {
+                data.resetSkillTree(skillTree);
+                data.setSkillTreePoints(skillTree.getId(), 0);
+            }
+
+            data.resetTimesClaimed();
+            while (data.hasSkillBound(0))
+                data.unbindSkill(0);
+            while (data.hasPassiveSkillBound(0))
+                data.unbindPassiveSkill(0);
+            data.getQuestData().resetFinishedQuests();
+            data.getQuestData().start(null);
+            CommandVerbose.verbose(sender, CommandVerbose.CommandType.RESET,
+                    ChatColor.GOLD + player.getName() + ChatColor.YELLOW + "'s data was succesfully reset.");
             return CommandResult.SUCCESS;
         }
     }
-}
 
-class ResetWaypointsCommandTreeNode extends CommandTreeNode {
+    public static class ResetWaypointsCommandTreeNode extends CommandTreeNode {
         public ResetWaypointsCommandTreeNode(CommandTreeNode parent) {
             super(parent, "waypoints");
 
@@ -79,7 +87,8 @@ class ResetWaypointsCommandTreeNode extends CommandTreeNode {
 
         @Override
         public CommandResult execute(CommandSender sender, String[] args) {
-        if (args.length < 4) return CommandResult.THROW_USAGE;
+            if (args.length < 4)
+                return CommandResult.THROW_USAGE;
 
             Player player = Bukkit.getPlayer(args[3]);
             if (player == null) {
@@ -87,17 +96,13 @@ class ResetWaypointsCommandTreeNode extends CommandTreeNode {
                 return CommandResult.FAILURE;
             }
 
-        resetWaypoints(PlayerData.get(player));
+            PlayerData data = PlayerData.get(player);
-        CommandVerbose.verbose(sender, CommandVerbose.CommandType.RESET, ChatColor.GOLD + player.getName() + ChatColor.YELLOW + "'s waypoints were successfully reset.");
+            data.getWaypoints().clear();
             return CommandResult.SUCCESS;
         }
-
-    static void resetWaypoints(@NotNull PlayerData playerData) {
-        playerData.getWaypoints().clear();
     }
-}
 
-class ResetQuestsCommandTreeNode extends CommandTreeNode {
+    public static class ResetQuestsCommandTreeNode extends CommandTreeNode {
         public ResetQuestsCommandTreeNode(CommandTreeNode parent) {
             super(parent, "quests");
 
@@ -106,7 +111,8 @@ class ResetQuestsCommandTreeNode extends CommandTreeNode {
 
         @Override
         public CommandResult execute(CommandSender sender, String[] args) {
-        if (args.length < 4) return CommandResult.THROW_USAGE;
+            if (args.length < 4)
+                return CommandResult.THROW_USAGE;
 
             Player player = Bukkit.getPlayer(args[3]);
             if (player == null) {
@@ -114,18 +120,14 @@ class ResetQuestsCommandTreeNode extends CommandTreeNode {
                 return CommandResult.FAILURE;
             }
 
-        resetQuests(PlayerData.get(player));
+            PlayerData data = PlayerData.get(player);
-        CommandVerbose.verbose(sender, CommandVerbose.CommandType.RESET, ChatColor.GOLD + player.getName() + ChatColor.YELLOW + "'s quests were successfully reset.");
-        return CommandResult.SUCCESS;
-    }
-
-    static void resetQuests(@NotNull PlayerData data) {
             data.getQuestData().resetFinishedQuests();
             data.getQuestData().start(null);
+            return CommandResult.SUCCESS;
+        }
     }
-}
 
-class ResetSkillsCommandTreeNode extends CommandTreeNode {
+    public static class ResetSkillsCommandTreeNode extends CommandTreeNode {
         public ResetSkillsCommandTreeNode(CommandTreeNode parent) {
             super(parent, "skills");
 
@@ -134,7 +136,8 @@ class ResetSkillsCommandTreeNode extends CommandTreeNode {
 
         @Override
         public CommandResult execute(CommandSender sender, String[] args) {
-        if (args.length < 4) return CommandResult.THROW_USAGE;
+            if (args.length < 4)
+                return CommandResult.THROW_USAGE;
 
             Player player = Bukkit.getPlayer(args[3]);
             if (player == null) {
@@ -142,19 +145,18 @@ class ResetSkillsCommandTreeNode extends CommandTreeNode {
                 return CommandResult.FAILURE;
             }
 
-        resetSkills(PlayerData.get(player));
+            PlayerData data = PlayerData.get(player);
-        CommandVerbose.verbose(sender, CommandVerbose.CommandType.RESET, ChatColor.GOLD + player.getName() + ChatColor.YELLOW + "'s skill data was successfully reset.");
+            data.mapSkillLevels().forEach((skill, level) -> data.resetSkillLevel(skill));
+            while (data.hasSkillBound(0))
+                data.unbindSkill(0);
+            CommandVerbose.verbose(sender, CommandVerbose.CommandType.RESET,
+                    ChatColor.GOLD + player.getName() + ChatColor.YELLOW + "'s skill data was succesfully reset.");
             return CommandResult.SUCCESS;
         }
-
-    static void resetSkills(@NotNull PlayerData data) {
-        data.mapSkillLevels().forEach((skill, ignored) -> data.resetSkillLevel(skill));
-        while (data.hasSkillBound(0)) data.unbindSkill(0);
-        data.setUnlockedItems(new HashSet<>()); // TODO class-specific unlockables etc.
     }
-}
 
-class ResetSkillTreesCommandTreeNode extends CommandTreeNode {
+
+    public static class ResetSkillTreesCommandTreeNode extends CommandTreeNode {
         public ResetSkillTreesCommandTreeNode(CommandTreeNode parent) {
             super(parent, "skill-trees");
 
@@ -163,7 +165,8 @@ class ResetSkillTreesCommandTreeNode extends CommandTreeNode {
 
         @Override
         public CommandResult execute(CommandSender sender, String[] args) {
-        if (args.length < 4) return CommandResult.THROW_USAGE;
+            if (args.length < 4)
+                return CommandResult.THROW_USAGE;
 
             Player player = Bukkit.getPlayer(args[3]);
             if (player == null) {
@@ -171,18 +174,16 @@ class ResetSkillTreesCommandTreeNode extends CommandTreeNode {
                 return CommandResult.FAILURE;
             }
 
-        resetSkillTrees(PlayerData.get(player));
+            PlayerData data = PlayerData.get(player);
-        CommandVerbose.verbose(sender, CommandVerbose.CommandType.RESET, ChatColor.GOLD + player.getName() + ChatColor.YELLOW + "'s skill-tree data was successfully reset.");
+            for (SkillTree skillTree : data.getProfess().getSkillTrees())
+                data.resetSkillTree(skillTree);
+            CommandVerbose.verbose(sender, CommandVerbose.CommandType.RESET,
+                    ChatColor.GOLD + player.getName() + ChatColor.YELLOW + "'s skill-tree data was succesfully reset.");
             return CommandResult.SUCCESS;
         }
-
-    // TODO option to reallocate skill tree points instead of not giving any back
-    static void resetSkillTrees(@NotNull PlayerData data) {
-        data.resetSkillTrees();
     }
-}
 
-class ResetAttributesCommandTreeNode extends CommandTreeNode {
+    public class ResetAttributesCommandTreeNode extends CommandTreeNode {
         public ResetAttributesCommandTreeNode(CommandTreeNode parent) {
             super(parent, "attributes");
 
@@ -192,7 +193,8 @@ class ResetAttributesCommandTreeNode extends CommandTreeNode {
 
         @Override
         public CommandResult execute(CommandSender sender, String[] args) {
-        if (args.length < 4) return CommandResult.THROW_USAGE;
+            if (args.length < 4)
+                return CommandResult.THROW_USAGE;
 
             Player player = Bukkit.getPlayer(args[3]);
             if (player == null) {
@@ -200,16 +202,12 @@ class ResetAttributesCommandTreeNode extends CommandTreeNode {
                 return CommandResult.FAILURE;
             }
 
-        final boolean givePoints = args.length > 4 && args[4].equalsIgnoreCase("-reallocate");
+            PlayerData data = PlayerData.get(player);
-        resetAttributes(PlayerData.get(player), givePoints);
-        CommandVerbose.verbose(sender, CommandVerbose.CommandType.RESET, ChatColor.GOLD + player.getName() + ChatColor.YELLOW + "'s attributes were successfully reset.");
-        return CommandResult.SUCCESS;
-    }
 
-    static void resetAttributes(@NotNull PlayerData data, boolean givePoints) {
+            /*
-
+             * force reallocating of player attribute points
-        // Give back attribute points
+             */
-        if (givePoints) {
+            if (args.length > 4 && args[4].equalsIgnoreCase("-reallocate")) {
 
                 int points = 0;
                 for (PlayerAttributes.AttributeInstance ins : data.getAttributes().getInstances()) {
@@ -218,17 +216,19 @@ class ResetAttributesCommandTreeNode extends CommandTreeNode {
                 }
 
                 data.giveAttributePoints(points);
-            return;
+                CommandVerbose.verbose(sender, CommandVerbose.CommandType.RESET,
+                        ChatColor.GOLD + player.getName() + ChatColor.YELLOW + "'s attribute points spendings were successfully reset.");
+                return CommandResult.SUCCESS;
             }
 
-        for (PlayerAttribute attribute : MMOCore.plugin.attributeManager.getAll()) {
+            data.getAttributes().getInstances().forEach(ins -> ins.setBase(0));
-            attribute.resetAdvancement(data, true);
+            CommandVerbose.verbose(sender, CommandVerbose.CommandType.RESET,
-            data.getAttributes().getInstance(attribute).setBase(0);
+                    ChatColor.GOLD + player.getName() + ChatColor.YELLOW + "'s attributes were succesfully reset.");
+            return CommandResult.SUCCESS;
         }
     }
-}
 
-class ResetLevelsCommandTreeNode extends CommandTreeNode {
+    public static class ResetLevelsCommandTreeNode extends CommandTreeNode {
         public ResetLevelsCommandTreeNode(CommandTreeNode parent) {
             super(parent, "levels");
 
@@ -237,7 +237,8 @@ class ResetLevelsCommandTreeNode extends CommandTreeNode {
 
         @Override
         public CommandResult execute(CommandSender sender, String[] args) {
-        if (args.length < 4) return CommandResult.THROW_USAGE;
+            if (args.length < 4)
+                return CommandResult.THROW_USAGE;
 
             Player player = Bukkit.getPlayer(args[3]);
             if (player == null) {
@@ -245,52 +246,19 @@ class ResetLevelsCommandTreeNode extends CommandTreeNode {
                 return CommandResult.FAILURE;
             }
 
-        resetLevels(PlayerData.get(player));
+            PlayerData data = PlayerData.get(player);
-        CommandVerbose.verbose(sender, CommandVerbose.CommandType.RESET, ChatColor.GOLD + player.getName() + ChatColor.YELLOW + "'s levels were successfully reset.");
+            data.setLevel(MMOCore.plugin.dataProvider.getDataManager().getDefaultData().getLevel());
-        return CommandResult.SUCCESS;
-    }
-
-    static void resetLevels(@NotNull PlayerData data) {
-
-        // Class
-        data.setLevel(MMOCore.plugin.playerDataManager.getDefaultData().getLevel());
             data.setExperience(0);
-        data.getProfess().resetAdvancement(data, true);
-
-        // Professions
             for (Profession profession : MMOCore.plugin.professionManager.getAll()) {
                 data.getCollectionSkills().setExperience(profession, 0);
                 data.getCollectionSkills().setLevel(profession, 0);
-            profession.resetAdvancement(data, true);
+                profession.getExperienceTable().reset(data, profession);
-        }
-    }
-}
-
-class ResetClassesCommandTreeNode extends CommandTreeNode {
-    public ResetClassesCommandTreeNode(CommandTreeNode parent) {
-        super(parent, "classes");
-
-        addParameter(Parameter.PLAYER);
             }
 
-    @Override
+            CommandVerbose.verbose(sender, CommandVerbose.CommandType.RESET,
-    public CommandResult execute(CommandSender sender, String[] args) {
+                    ChatColor.GOLD + player.getName() + ChatColor.YELLOW + "'s levels were succesfully reset.");
-        if (args.length < 4) return CommandResult.THROW_USAGE;
 
-        Player player = Bukkit.getPlayer(args[3]);
-        if (player == null) {
-            sender.sendMessage(ChatColor.RED + "Could not find the player called " + args[3] + ".");
-            return CommandResult.FAILURE;
-        }
-
-        resetClasses(PlayerData.get(player));
-        CommandVerbose.verbose(sender, CommandVerbose.CommandType.RESET, ChatColor.GOLD + player.getName() + ChatColor.YELLOW + "'s classes were successfully reset.");
             return CommandResult.SUCCESS;
         }
-
-    static void resetClasses(@NotNull PlayerData data) {
-        MMOCore.plugin.classManager.getAll().forEach(data::unloadClassInfo);
-        MMOCore.plugin.playerDataManager.getDefaultData().apply(data);
-        data.setClass(MMOCore.plugin.classManager.getDefaultClass());
     }
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/ResourceCommandTreeNode.java

@@ -5,7 +5,7 @@ import io.lumine.mythic.lib.command.api.Parameter;
 import net.Indyuce.mmocore.api.player.PlayerData;
 import net.Indyuce.mmocore.api.quest.trigger.ManaTrigger;
 import net.Indyuce.mmocore.api.player.profess.resource.PlayerResource;
-import net.Indyuce.mmocore.command.api.CommandVerbose;
+import net.Indyuce.mmocore.command.CommandVerbose;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.command.CommandSender;

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/SaveDataTreeNode.java

@@ -1,37 +0,0 @@
-package net.Indyuce.mmocore.command.rpg.admin;
-
-import io.lumine.mythic.lib.command.api.CommandTreeNode;
-import io.lumine.mythic.lib.command.api.Parameter;
-import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.player.PlayerData;
-import org.bukkit.Bukkit;
-import org.bukkit.ChatColor;
-import org.bukkit.command.CommandSender;
-import org.bukkit.entity.Player;
-
-/**
- * Saves player data
- */
-public class SaveDataTreeNode extends CommandTreeNode {
-    public SaveDataTreeNode(CommandTreeNode parent) {
-        super(parent, "savedata");
-
-        addParameter(Parameter.PLAYER);
-    }
-
-    @Override
-    public CommandResult execute(CommandSender sender, String[] args) {
-        if (args.length < 3)
-            return CommandResult.THROW_USAGE;
-
-        Player player = Bukkit.getPlayer(args[2]);
-        if (player == null) {
-            sender.sendMessage(ChatColor.RED + "Could not find the player called " + args[2] + ".");
-            return CommandResult.FAILURE;
-        }
-
-        MMOCore.plugin.playerDataManager.getDataHandler().saveData(PlayerData.get(player), false);
-
-        return CommandResult.SUCCESS;
-    }
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/SkillCommandTreeNode.java

@@ -5,7 +5,7 @@ import io.lumine.mythic.lib.command.api.Parameter;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
 import net.Indyuce.mmocore.skill.RegisteredSkill;
-import net.Indyuce.mmocore.command.api.CommandVerbose;
+import net.Indyuce.mmocore.command.CommandVerbose;
 import net.Indyuce.mmocore.skill.ClassSkill;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
@@ -19,21 +19,19 @@ public class SkillCommandTreeNode extends CommandTreeNode {
     public SkillCommandTreeNode(CommandTreeNode parent) {
         super(parent, "skill");
 
-        addChild(new LockSkillCommandTreeNode(this, "lock", true));
+        addChild(new ActionCommandTreeNode(this, "give", (old, amount) -> old + amount));
-        addChild(new LockSkillCommandTreeNode(this, "unlock", false));
+        addChild(new ActionCommandTreeNode(this, "set", (old, amount) -> amount));
-        addChild(new LevelCommandTreeNode(this, "give", (old, amount) -> old + amount));
-        addChild(new LevelCommandTreeNode(this, "set", (old, amount) -> amount));
     }
 
-    public class LevelCommandTreeNode extends CommandTreeNode {
+
+    public class ActionCommandTreeNode extends CommandTreeNode {
         private final BiFunction<Integer, Integer, Integer> change;
 
-        public LevelCommandTreeNode(CommandTreeNode parent, String type, BiFunction<Integer, Integer, Integer> change) {
+        public ActionCommandTreeNode(CommandTreeNode parent, String type, BiFunction<Integer, Integer, Integer> change) {
             super(parent, type);
-
             this.change = change;
             addParameter(Parameter.PLAYER);
-            addParameter(new Parameter("<skill>",
+            addParameter(new Parameter("<attribute>",
                     (explorer, list) -> MMOCore.plugin.skillManager.getAll().forEach(skill -> list.add(skill.getHandler().getId().toUpperCase()))));
             addParameter(Parameter.AMOUNT);
         }
@@ -57,17 +55,19 @@ public class SkillCommandTreeNode extends CommandTreeNode {
             }
 
 
-            ClassSkill classSkill = null;
+
-            for (ClassSkill var : playerData.getProfess().getSkills()) {
+            ClassSkill classSkill=null;
-                if (var.getSkill().equals(skill))
+            for(ClassSkill var:playerData.getProfess().getSkills()) {
-                    classSkill = var;
+                if(var.getSkill().equals(skill))
+                    classSkill=var;
             }
 
-            if (classSkill == null || classSkill.getUnlockLevel() > playerData.getLevel()) {
+            if(classSkill==null||classSkill.getUnlockLevel() > playerData.getLevel()) {
-                sender.sendMessage(ChatColor.RED + skill.getName() + " is not unlockable for " + player.getName() + ".");
+                sender.sendMessage(ChatColor.RED+ skill.getName()+" is not unlockable for "+player.getName()+".");
                 return CommandResult.FAILURE;
             }
 
+
             int amount;
             try {
                 amount = Integer.parseInt(args[5]);
@@ -77,59 +77,12 @@ public class SkillCommandTreeNode extends CommandTreeNode {
             }
             int value = change.apply(playerData.getSkillLevel(skill), amount);
             playerData.setSkillLevel(skill, value);
-            CommandVerbose.verbose(sender, CommandVerbose.CommandType.SKILL, ChatColor.YELLOW + player.getName() + ChatColor.YELLOW
+            CommandVerbose.verbose(sender, CommandVerbose.CommandType.SKILL, ChatColor.GOLD + player.getName() + ChatColor.YELLOW
                     + " is now level  " + ChatColor.GOLD + value + ChatColor.YELLOW + " for " + skill.getName() + ".");
             return CommandResult.SUCCESS;
         }
     }
 
-    public class LockSkillCommandTreeNode extends CommandTreeNode {
-        private final boolean lock;
-
-        public LockSkillCommandTreeNode(CommandTreeNode parent, String id, boolean lock) {
-            super(parent, id);
-            this.lock = lock;
-            addParameter(Parameter.PLAYER);
-            addParameter(new Parameter("<skill>",
-                    (explorer, list) -> MMOCore.plugin.skillManager.getAll().forEach(skill -> list.add(skill.getHandler().getId().toUpperCase()))));
-        }
-
-        @Override
-        public CommandResult execute(CommandSender sender, String[] args) {
-            if (args.length < 5)
-                return CommandResult.THROW_USAGE;
-            Player player = Bukkit.getPlayer(args[3]);
-            if (player == null) {
-                sender.sendMessage(ChatColor.RED + "Could not find player called " + args[3] + ".");
-                return CommandResult.FAILURE;
-            }
-            PlayerData playerData = PlayerData.get(player);
-
-            ClassSkill skill = playerData.getProfess().getSkill(args[4]);
-            if (skill == null) {
-                sender.sendMessage(ChatColor.RED + "Class doesn't have a skill called  " + args[4] + ".");
-                return CommandResult.FAILURE;
-            }
-            if (lock) {
-                if (!playerData.hasUnlocked(skill)) {
-                    CommandVerbose.verbose(sender, CommandVerbose.CommandType.SKILL, ChatColor.RED + "Skill " + skill.getSkill().getName() + " already locked for " + player.getName());
-                    return CommandResult.SUCCESS;
-                }
-                playerData.lock(skill);
-
-            } else {
-                if (playerData.hasUnlocked(skill)) {
-                    CommandVerbose.verbose(sender, CommandVerbose.CommandType.SKILL, ChatColor.RED + "Skill " + skill.getSkill().getName() + " already unlocked for " + player.getName());
-                    return CommandResult.SUCCESS;
-                }
-                playerData.unlock(skill);
-            }
-            CommandVerbose.verbose(sender, CommandVerbose.CommandType.SKILL, ChatColor.YELLOW + "Skill " + ChatColor.GOLD + skill.getSkill().getName() + ChatColor.YELLOW + " now " + (lock ? "locked" : "unlocked") + " for " + ChatColor.GOLD + player.getName());
-            return CommandResult.SUCCESS;
-        }
-    }
-
-
     @Override
     public CommandResult execute(CommandSender sender, String[] args) {
         return CommandResult.THROW_USAGE;

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/SkillTreePointsCommandTreeNode.java

@@ -4,7 +4,7 @@ import io.lumine.mythic.lib.command.api.CommandTreeNode;
 import io.lumine.mythic.lib.command.api.Parameter;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.command.api.CommandVerbose;
+import net.Indyuce.mmocore.command.CommandVerbose;
 import net.Indyuce.mmocore.util.TriConsumer;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/admin/SlotCommandTreeNode.java

@@ -1,159 +0,0 @@
-package net.Indyuce.mmocore.command.rpg.admin;
-
-import io.lumine.mythic.lib.command.api.CommandTreeNode;
-import io.lumine.mythic.lib.command.api.Parameter;
-import net.Indyuce.mmocore.MMOCore;
-import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.skill.binding.BoundSkillInfo;
-import net.Indyuce.mmocore.skill.binding.SkillSlot;
-import net.Indyuce.mmocore.command.api.CommandVerbose;
-import net.Indyuce.mmocore.skill.ClassSkill;
-import org.bukkit.Bukkit;
-import org.bukkit.ChatColor;
-import org.bukkit.command.CommandSender;
-import org.bukkit.entity.Player;
-
-
-public class SlotCommandTreeNode extends CommandTreeNode {
-    public SlotCommandTreeNode(CommandTreeNode parent) {
-        super(parent, "slot");
-        addChild(new LockSlotCommandTreeNode(this, "lock", true));
-        addChild(new LockSlotCommandTreeNode(this, "unlock", false));
-        addChild(new UnbindSlotCommandTreeNode(this, "unbind"));
-        addChild(new BindSlotCommandTreeNode(this, "bind"));
-    }
-
-    public class LockSlotCommandTreeNode extends CommandTreeNode {
-        private final boolean lock;
-
-        public LockSlotCommandTreeNode(CommandTreeNode parent, String id, boolean lock) {
-            super(parent, id);
-            this.lock = lock;
-            addParameter(Parameter.PLAYER);
-            addParameter(Parameter.AMOUNT);
-        }
-
-        @Override
-        public CommandResult execute(CommandSender sender, String[] args) {
-            if (args.length < 5)
-                return CommandResult.THROW_USAGE;
-            Player player = Bukkit.getPlayer(args[3]);
-            if (player == null) {
-                sender.sendMessage(ChatColor.RED + "Could not find the player called " + args[3] + ".");
-                return CommandResult.FAILURE;
-            }
-            PlayerData playerData = PlayerData.get(player);
-            int slot;
-            try {
-                slot = Integer.parseInt(args[4]);
-            } catch (NumberFormatException e) {
-                sender.sendMessage(ChatColor.RED + args[4] + " is not a valid number.");
-                return CommandResult.FAILURE;
-            }
-            if (slot <= 0) {
-                sender.sendMessage(ChatColor.RED + "The slot can't be negative.");
-                return CommandResult.FAILURE;
-            }
-            SkillSlot skillSlot = playerData.getProfess().getSkillSlot(slot);
-            if (skillSlot.isUnlockedByDefault()) {
-                sender.sendMessage(ChatColor.RED + "You can't lock a skill that is unlocked by default.");
-                return CommandResult.FAILURE;
-            }
-            if (lock) {
-                if (!playerData.hasUnlocked(skillSlot)) {
-                    CommandVerbose.verbose(sender, CommandVerbose.CommandType.SKILL, ChatColor.RED + "The skill slot " + skillSlot.getName() + " is already locked" + " for " + player.getName());
-                    return CommandResult.SUCCESS;
-                }
-                playerData.lock(skillSlot);
-
-            } else {
-                if (playerData.hasUnlocked(skillSlot)) {
-                    CommandVerbose.verbose(sender, CommandVerbose.CommandType.SKILL, ChatColor.RED + "The skill slot " + skillSlot.getName() + " is already unlocked" + " for " + player.getName());
-                    return CommandResult.SUCCESS;
-                }
-                playerData.unlock(skillSlot);
-            }
-            CommandVerbose.verbose(sender, CommandVerbose.CommandType.SKILL, ChatColor.YELLOW + "The skill slot " + skillSlot.getName() + " is now " + (lock ? "locked" : "unlocked" + " for " + player.getName()));
-            return CommandResult.SUCCESS;
-        }
-    }
-
-
-    public class BindSlotCommandTreeNode extends CommandTreeNode {
-
-        public BindSlotCommandTreeNode(CommandTreeNode parent, String id) {
-            super(parent, id);
-            addParameter(Parameter.PLAYER);
-            addParameter(Parameter.AMOUNT);
-            addParameter(new Parameter("<skill>",
-                    (explorer, list) -> MMOCore.plugin.skillManager.getAll().forEach(skill -> list.add(skill.getHandler().getId().toUpperCase()))));
-
-        }
-
-        @Override
-        public CommandResult execute(CommandSender sender, String[] args) {
-            if (args.length < 6)
-                return CommandResult.THROW_USAGE;
-            Player player = Bukkit.getPlayer(args[3]);
-            if (player == null) {
-                sender.sendMessage(ChatColor.RED + "Could not find the player called " + args[3] + ".");
-                return CommandResult.FAILURE;
-            }
-            PlayerData playerData = PlayerData.get(player);
-            int slot;
-            try {
-                slot = Integer.parseInt(args[4]);
-            } catch (NumberFormatException e) {
-                sender.sendMessage(ChatColor.RED + args[4] + " is not a valid number.");
-                return CommandResult.FAILURE;
-            }
-            ClassSkill skill = playerData.getProfess().getSkill(args[5]);
-            if (skill == null) {
-                sender.sendMessage(ChatColor.RED + "The player's class doesn't have a skill called  " + args[5] + ".");
-                return CommandResult.FAILURE;
-            }
-            playerData.bindSkill(slot, skill);
-
-            CommandVerbose.verbose(sender, CommandVerbose.CommandType.SKILL, ChatColor.YELLOW + "Skill " + ChatColor.GOLD + skill.getSkill().getHandler().getId() + ChatColor.YELLOW + " now bound to slot " + ChatColor.GOLD + slot);
-            return CommandResult.SUCCESS;
-        }
-    }
-
-    public class UnbindSlotCommandTreeNode extends CommandTreeNode {
-
-        public UnbindSlotCommandTreeNode(CommandTreeNode parent, String id) {
-            super(parent, id);
-            addParameter(Parameter.PLAYER);
-            addParameter(Parameter.AMOUNT);
-        }
-
-        @Override
-        public CommandResult execute(CommandSender sender, String[] args) {
-            if (args.length < 5)
-                return CommandResult.THROW_USAGE;
-            Player player = Bukkit.getPlayer(args[3]);
-            if (player == null) {
-                sender.sendMessage(ChatColor.RED + "Could not find the player called " + args[3] + ".");
-                return CommandResult.FAILURE;
-            }
-            PlayerData playerData = PlayerData.get(player);
-            int slot;
-            try {
-                slot = Integer.parseInt(args[4]);
-            } catch (NumberFormatException e) {
-                sender.sendMessage(ChatColor.RED + args[4] + " is not a valid number.");
-                return CommandResult.FAILURE;
-            }
-            final BoundSkillInfo found = playerData.unbindSkill(slot);
-            CommandVerbose.verbose(sender, CommandVerbose.CommandType.SKILL, ChatColor.YELLOW + (found != null ?
-                    "Skill " + ChatColor.GOLD + found.getClassSkill().getSkill().getName() + ChatColor.YELLOW + " was taken off the slot " + ChatColor.GOLD + slot :
-                    "Could not find skill at slot " + ChatColor.GOLD + slot));
-            return CommandResult.SUCCESS;
-        }
-    }
-
-    @Override
-    public CommandResult execute(CommandSender sender, String[] args) {
-        return CommandResult.THROW_USAGE;
-    }
-}

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/booster/CreateCommandTreeNode.java

@@ -49,7 +49,7 @@ public class CreateCommandTreeNode extends CommandTreeNode {
 
 		if (args[2].equalsIgnoreCase("main")) {
 			MMOCore.plugin.boosterManager.register(new Booster(args.length > 5 ? args[5] : null, extra, length));
-			ConfigMessage.fromKey("booster-main").addPlaceholders("multiplier", "" + (1 + extra)).send(Bukkit.getOnlinePlayers());
+			new ConfigMessage("booster-main").addPlaceholders("multiplier", "" + (1 + extra)).send(Bukkit.getOnlinePlayers());
 			Bukkit.getOnlinePlayers().forEach(player -> player.playSound(player.getLocation(), Sound.ENTITY_EXPERIENCE_ORB_PICKUP, 1, 1));
 			return CommandResult.SUCCESS;
 		}
@@ -62,7 +62,7 @@ public class CreateCommandTreeNode extends CommandTreeNode {
 
 		Profession profession = MMOCore.plugin.professionManager.get(format);
 		MMOCore.plugin.boosterManager.register(new Booster(args.length > 5 ? args[5] : null, profession, extra, length));
-		ConfigMessage.fromKey("booster-skill").addPlaceholders("multiplier", "" + (1 + extra), "profession", profession.getName())
+		new ConfigMessage("booster-skill").addPlaceholders("multiplier", "" + (1 + extra), "profession", profession.getName())
 				.send(Bukkit.getOnlinePlayers());
 		Bukkit.getOnlinePlayers().forEach(player -> player.playSound(player.getLocation(), Sound.ENTITY_EXPERIENCE_ORB_PICKUP, 1, 1));
 		return CommandResult.SUCCESS;

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/debug/StatModifiersCommandTreeNode.java

@@ -33,7 +33,7 @@ public class StatModifiersCommandTreeNode extends CommandTreeNode {
 		sender.sendMessage("Stat Modifiers (" + instance.getKeys().size() + "):");
 		for (String key : instance.getKeys()) {
 			StatModifier mod = instance.getModifier(key);
-            sender.sendMessage("-> '" + key + "' " + mod.getValue() + " " + mod.getType().name() + " " + mod.getSlot() + " " + mod.getSource());
+			sender.sendMessage("- " + key + ": " + mod.getValue() + " " + mod.getType().name());
 		}
 
 		return CommandResult.SUCCESS;

MMOCore-API/src/main/java/net/Indyuce/mmocore/command/rpg/waypoint/LockCommandTreeNode.java

@@ -4,7 +4,7 @@ import io.lumine.mythic.lib.command.api.CommandTreeNode;
 import io.lumine.mythic.lib.command.api.Parameter;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.command.api.CommandVerbose;
+import net.Indyuce.mmocore.command.CommandVerbose;
 import net.Indyuce.mmocore.waypoint.Waypoint;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;

MMOCore-API/src/main/java/net/Indyuce/mmocore/comp/mythicmobs/load/CurrencyItemDrop.java

@@ -4,7 +4,7 @@ import io.lumine.mythic.api.adapters.AbstractItemStack;
 import io.lumine.mythic.api.config.MythicLineConfig;
 import io.lumine.mythic.api.drops.DropMetadata;
 import io.lumine.mythic.api.drops.IItemDrop;
-import io.lumine.mythic.bukkit.BukkitAdapter;
+import io.lumine.mythic.bukkit.adapters.BukkitItemStack;
 import io.lumine.mythic.core.drops.Drop;
 import net.Indyuce.mmocore.util.item.CurrencyItemBuilder;
 
@@ -27,8 +27,7 @@ public class CurrencyItemDrop extends Drop implements IItemDrop {
 
     @Override
     public AbstractItemStack getDrop(DropMetadata dropMetadata, double v) {
-        // Not great wrt to performance. Should build the item like MM does
+        return new BukkitItemStack(new CurrencyItemBuilder(key, random(minw, maxw)).build());
-        return BukkitAdapter.adapt(new CurrencyItemBuilder(key, random(minw, maxw)).build());
     }
 
     private int random(int a, int b) {

MMOCore-API/src/main/java/net/Indyuce/mmocore/comp/mythicmobs/load/GoldPouchDrop.java

@@ -4,7 +4,7 @@ import io.lumine.mythic.api.adapters.AbstractItemStack;
 import io.lumine.mythic.api.config.MythicLineConfig;
 import io.lumine.mythic.api.drops.DropMetadata;
 import io.lumine.mythic.api.drops.IItemDrop;
-import io.lumine.mythic.bukkit.BukkitAdapter;
+import io.lumine.mythic.bukkit.adapters.BukkitItemStack;
 import io.lumine.mythic.core.drops.Drop;
 import io.lumine.mythic.lib.api.item.ItemTag;
 import io.lumine.mythic.lib.api.item.NBTItem;
@@ -49,8 +49,7 @@ public class GoldPouchDrop extends Drop implements IItemDrop {
         }
 
         nbt.addTag(new ItemTag("RpgPouchSize", 18), new ItemTag("RpgPouchMob", true), new ItemTag("RpgPouchInventory", MMOCoreUtils.toBase64(content)));
-        // Not great wrt to performance. Should build the item like MM does
+        return new BukkitItemStack(nbt.toItem());
-        return BukkitAdapter.adapt(nbt.toItem());
     }
 
     private ItemStack setAmount(ItemStack item, int amount) {

MMOCore-API/src/main/java/net/Indyuce/mmocore/comp/placeholder/DefaultParser.java

@@ -1,6 +1,5 @@
 package net.Indyuce.mmocore.comp.placeholder;
 
-import io.lumine.mythic.lib.util.annotation.BackwardsCompatibility;
 import org.bukkit.OfflinePlayer;
 
 import io.lumine.mythic.lib.MythicLib;
@@ -8,9 +7,7 @@ import io.lumine.mythic.lib.MythicLib;
 public class DefaultParser implements PlaceholderParser {
 
 	@Override
-	@BackwardsCompatibility(version = "1.12-SNAPSHOT")
 	public String parse(OfflinePlayer player, String string) {
-		// TODO remove use of confusing non-PAPI %player% placeholder
 		return MythicLib.plugin.parseColors(string.replace("%player%", player.getName()));
 	}
 }

MMOCore-API/src/main/java/net/Indyuce/mmocore/comp/placeholder/RPGPlaceholders.java

@@ -1,30 +1,22 @@
 package net.Indyuce.mmocore.comp.placeholder;
 
 import io.lumine.mythic.lib.MythicLib;
-import io.lumine.mythic.lib.UtilityMethods;
 import io.lumine.mythic.lib.api.util.AltChar;
 import io.lumine.mythic.lib.manager.StatManager;
 import me.clip.placeholderapi.expansion.PlaceholderExpansion;
 import net.Indyuce.mmocore.MMOCore;
 import net.Indyuce.mmocore.api.player.PlayerData;
-import net.Indyuce.mmocore.api.player.attribute.PlayerAttributes;
 import net.Indyuce.mmocore.api.quest.PlayerQuests;
 import net.Indyuce.mmocore.experience.PlayerProfessions;
 import net.Indyuce.mmocore.experience.Profession;
 import net.Indyuce.mmocore.party.AbstractParty;
-import net.Indyuce.mmocore.skill.CastableSkill;
-import net.Indyuce.mmocore.skill.ClassSkill;
 import net.Indyuce.mmocore.skill.RegisteredSkill;
-import org.apache.commons.lang.Validate;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.OfflinePlayer;
 import org.bukkit.attribute.Attribute;
-import org.bukkit.entity.Player;
 
-import javax.annotation.Nullable;
 import java.util.Objects;
-import java.util.UUID;
 
 
 public class RPGPlaceholders extends PlaceholderExpansion {
@@ -53,63 +45,27 @@ public class RPGPlaceholders extends PlaceholderExpansion {
 		return MMOCore.plugin.getDescription().getVersion();
 	}
 
-    private static final String ERROR_PLACEHOLDER = " ";
-
 	@SuppressWarnings("DuplicateExpressions")
 	@Override
 	public String onRequest(OfflinePlayer player, String identifier) {
 		if (!PlayerData.has(player.getUniqueId()))
 			return null;
-        final PlayerData playerData = PlayerData.get(player);
 
+		PlayerData playerData = PlayerData.get(player);
 		if (identifier.equals("mana_icon"))
 			return playerData.getProfess().getManaDisplay().getIcon();
-
 		if (identifier.equals("mana_name"))
 			return playerData.getProfess().getManaDisplay().getName();
 
 		if (identifier.equals("level"))
-            return String.valueOf(playerData.getLevel());
+			return "" + playerData.getLevel();
 
 		else if (identifier.startsWith("skill_level_")) {
 			String id = identifier.substring(12);
-            RegisteredSkill skill = MMOCore.plugin.skillManager.getSkillOrThrow(id);
+			RegisteredSkill skill = Objects.requireNonNull(MMOCore.plugin.skillManager.getSkill(id), "Could not find skill with ID '" + id + "'");
 			return String.valueOf(playerData.getSkillLevel(skill));
 		}
 
-        /*
-         * Given a skill slot number (integer) and a parameter name,
-         * return the player's value of that skill parameter from that
-         * specific skill slot.
-         */
-        else if (identifier.startsWith("bound_skill_parameter_")) {
-            final String[] ids = identifier.substring(22).split(":");
-            final String parameterId = ids[0];
-            final int skillSlot = Integer.parseInt(ids[1]);
-            final ClassSkill found = playerData.getBoundSkill(skillSlot);
-            if (found == null) return "";
-            final CastableSkill castable = found.toCastable(playerData);
-            final double value = playerData.getMMOPlayerData().getSkillModifierMap().calculateValue(castable, parameterId);
-            return MythicLib.plugin.getMMOConfig().decimal.format(value);
-        }
-
-        // Returns a player's value of a skill parameter.
-        else if (identifier.startsWith("skill_modifier_") || identifier.startsWith("skill_parameter_")) {
-            final String[] ids = identifier.substring(identifier.startsWith("skill_modifier_") ? 15 : 16).split(":");
-            final String parameterId = ids[0];
-            final String skillId = ids[1];
-            final RegisteredSkill skill = Objects.requireNonNull(MMOCore.plugin.skillManager.getSkill(skillId), "Could not find skill with ID '" + skillId + "'");
-            final CastableSkill castable = playerData.getProfess().getSkill(skill).toCastable(playerData);
-            final double value = playerData.getMMOPlayerData().getSkillModifierMap().calculateValue(castable, parameterId);
-            return MythicLib.plugin.getMMOConfig().decimal.format(value);
-        }
-
-        else if (identifier.startsWith("attribute_points_spent_")) {
-            final String attributeId = identifier.substring(23);
-            final PlayerAttributes.AttributeInstance attributeInstance = Objects.requireNonNull(playerData.getAttributes().getInstance(attributeId), "Could not find attribute with ID '" + attributeId + "'");
-            return String.valueOf(attributeInstance.getBase());
-        }
-
 		else if (identifier.equals("level_percent")) {
 			double current = playerData.getExperience(), next = playerData.getLevelUpExperience();
 			return MythicLib.plugin.getMMOConfig().decimal.format(current / next * 100);
@@ -127,7 +83,9 @@ public class RPGPlaceholders extends PlaceholderExpansion {
 			for (double j = 1; j < 20; j++)
 				format.append(ratio >= j ? ChatColor.RED : ratio >= j - .5 ? ChatColor.DARK_RED : ChatColor.DARK_GRAY).append(AltChar.listSquare);
 			return format.toString();
-        } else if (identifier.equals("class"))
+		}
+
+		else if (identifier.equals("class"))
 			return playerData.getProfess().getName();
 
 		else if (identifier.startsWith("profession_percent_")) {
@@ -136,92 +94,34 @@ public class RPGPlaceholders extends PlaceholderExpansion {
 			Profession profession = MMOCore.plugin.professionManager.get(name);
 			double current = professions.getExperience(profession), next = professions.getLevelUpExperience(profession);
 			return MythicLib.plugin.getMMOConfig().decimal.format(current / next * 100);
+		}
 
-        } else if (identifier.equals("is_casting"))
+		else if (identifier.startsWith("is_casting")) {
 			return String.valueOf(playerData.isCasting());
-
+		} else if (identifier.startsWith("in_combat")) {
-        else if (identifier.equals("in_combat"))
 			return String.valueOf(playerData.isInCombat());
-
-        else if (identifier.equals("pvp_mode"))
-            return String.valueOf(playerData.getCombat().isInPvpMode());
-
-        else if (identifier.startsWith("since_enter_combat"))
-            return playerData.isInCombat() ? MythicLib.plugin.getMMOConfig().decimal.format((System.currentTimeMillis() - playerData.getCombat().getLastEntry()) / 1000.) : "-1";
-
-        else if (identifier.startsWith("invulnerability_left"))
-            return MythicLib.plugin.getMMOConfig().decimal.format(Math.max(0, (playerData.getCombat().getInvulnerableTill() - System.currentTimeMillis()) / 1000.));
-
-        else if (identifier.startsWith("since_last_hit"))
-            return playerData.isInCombat() ? MythicLib.plugin.getMMOConfig().decimal.format((System.currentTimeMillis() - playerData.getCombat().getLastHit()) / 1000.) : "-1";
-
-        // Returns the bound skill ID
-        else if (identifier.startsWith("id_bound_")) {
-            final int slot = Math.max(1, Integer.parseInt(identifier.substring(9)));
-            final ClassSkill info = playerData.getBoundSkill(slot);
-            return info == null ? "" : info.getSkill().getHandler().getId();
 		}
 
-        // Returns the casting slot taking into account the skill slot offset
-        else if (identifier.startsWith("cast_slot_offset_")) {
-            final Player online = player.getPlayer();
-            Validate.notNull(online, "Player is offline");
-            final int slot = Integer.parseInt(identifier.substring(17));
-            return String.valueOf(slot + (online.getInventory().getHeldItemSlot() < slot ? 1 : 0));
-        }
-
-        // Is there a passive skill bound to given slot
-        else if (identifier.startsWith("passive_bound_")) {
-            final int slot = Integer.parseInt(identifier.substring(14));
-            final ClassSkill skill = playerData.getBoundSkill(slot);
-            return String.valueOf(skill != null && skill.getSkill().getTrigger().isPassive());
-        }
-
-        // Returns the bound skill name
 		else if (identifier.startsWith("bound_")) {
-            final int slot = Math.max(1, Integer.parseInt(identifier.substring(6)));
+			int slot = Math.max(0, Integer.parseInt(identifier.substring(6)) - 1);
-            final ClassSkill skill = playerData.getBoundSkill(slot);
+			return playerData.hasSkillBound(slot) ? playerData.getBoundSkill(slot).getSkill().getName()
-            if (skill == null) return MMOCore.plugin.configManager.noSkillBoundPlaceholder;
+					: MMOCore.plugin.configManager.noSkillBoundPlaceholder;
-            return (playerData.getCooldownMap().isOnCooldown(skill) ? ChatColor.RED : ChatColor.GREEN) + skill.getSkill().getName();
 		}
 
-        // Returns cooldown of skill bound at given slot
+		else if (identifier.startsWith("profession_experience_"))
-        else if (identifier.startsWith("cooldown_bound_")) {
-            int slot = Math.max(0, Integer.parseInt(identifier.substring(15)));
-            if (playerData.hasSkillBound(slot))
-                return Double.toString(playerData.getCooldownMap().getCooldown(playerData.getBoundSkill(slot)));
-            else return MMOCore.plugin.configManager.noSkillBoundPlaceholder;
-        } else if (identifier.startsWith("profession_experience_"))
 			return MythicLib.plugin.getMMOConfig().decimal.format(
 					playerData.getCollectionSkills().getExperience(identifier.substring(22).replace(" ", "-").replace("_", "-").toLowerCase()));
 
 		else if (identifier.startsWith("profession_next_level_"))
-            return String.valueOf(PlayerData.get(player).getCollectionSkills()
+			return "" + PlayerData.get(player).getCollectionSkills()
-                    .getLevelUpExperience(identifier.substring(22).replace(" ", "-").replace("_", "-").toLowerCase()));
+					.getLevelUpExperience(identifier.substring(22).replace(" ", "-").replace("_", "-").toLowerCase());
 
 		else if (identifier.startsWith("party_count")) {
-            final @Nullable AbstractParty party = playerData.getParty();
+			AbstractParty party = playerData.getParty();
 			return party == null ? "0" : String.valueOf(party.countMembers());
-        } else if (identifier.startsWith("party_member_")) {
+		}
-            final int n = Integer.parseInt(identifier.substring(13)) - 1;
+
-            final @Nullable AbstractParty party = playerData.getParty();
+		else if (identifier.startsWith("profession_"))
-            if (party == null) return ERROR_PLACEHOLDER;
-            if (n >= party.countMembers()) return ERROR_PLACEHOLDER;
-            final @Nullable PlayerData member = party.getMember(n);
-            if (member == null) return ERROR_PLACEHOLDER;
-            return member.getPlayer().getName();
-        } else if (identifier.equals("online_friends")) {
-            int count = 0;
-            for (UUID friendId : playerData.getFriends())
-                if (Bukkit.getPlayer(friendId) != null) count++;
-            return String.valueOf(count);
-        } else if (identifier.startsWith("online_friend_")) {
-            final int n = Integer.parseInt(identifier.substring(14)) - 1;
-            if (n >= playerData.getFriends().size()) return ERROR_PLACEHOLDER;
-            final @Nullable Player friend = Bukkit.getPlayer(playerData.getFriends().get(n));
-            if (friend == null) return ERROR_PLACEHOLDER;
-            return friend.getName();
-        } else if (identifier.startsWith("profession_"))
 			return String
 					.valueOf(playerData.getCollectionSkills().getLevel(identifier.substring(11).replace(" ", "-").replace("_", "-").toLowerCase()));
 
@@ -250,18 +150,23 @@ public class RPGPlaceholders extends PlaceholderExpansion {
 		else if (identifier.equals("mana"))
 			return MythicLib.plugin.getMMOConfig().decimal.format(playerData.getMana());
 
-        else if (identifier.equals("mana_bar"))
+		else if (identifier.equals("mana_bar")) {
 			return playerData.getProfess().getManaDisplay().generateBar(playerData.getMana(), playerData.getStats().getStat("MAX_MANA"));
+		}
 
 		else if (identifier.startsWith("exp_multiplier_")) {
 			String format = identifier.substring(15).toLowerCase().replace("_", "-").replace(" ", "-");
 			Profession profession = format.equals("main") ? null : MMOCore.plugin.professionManager.get(format);
 			return MythicLib.plugin.getMMOConfig().decimal.format(MMOCore.plugin.boosterManager.getMultiplier(profession) * 100);
-        } else if (identifier.startsWith("exp_boost_")) {
+		}
+
+		else if (identifier.startsWith("exp_boost_")) {
 			String format = identifier.substring(10).toLowerCase().replace("_", "-").replace(" ", "-");
 			Profession profession = format.equals("main") ? null : MMOCore.plugin.professionManager.get(format);
 			return MythicLib.plugin.getMMOConfig().decimal.format((MMOCore.plugin.boosterManager.getMultiplier(profession) - 1) * 100);
-        } else if (identifier.equals("stamina"))
+		}
+
+		else if (identifier.equals("stamina"))
 			return MythicLib.plugin.getMMOConfig().decimal.format(playerData.getStamina());
 
 		else if (identifier.equals("stamina_bar")) {
@@ -272,10 +177,14 @@ public class RPGPlaceholders extends PlaceholderExpansion {
 						: ratio >= j - .5 ? MMOCore.plugin.configManager.staminaHalf : MMOCore.plugin.configManager.staminaEmpty)
 						.append(AltChar.listSquare);
 			return format.toString();
-        } else if (identifier.startsWith("stat_")) {
+		}
-            final String stat = UtilityMethods.enumName(identifier.substring(5));
+
-            return StatManager.format(stat, playerData.getMMOPlayerData());
+		else if (identifier.startsWith("stat_")) {
-        } else if (identifier.equals("stellium"))
+			final String stat = identifier.substring(5).toUpperCase();
+			return StatManager.format(identifier.substring(5), playerData.getStats().getStat(stat));
+		}
+
+		else if (identifier.equals("stellium"))
 			return MythicLib.plugin.getMMOConfig().decimal.format(playerData.getStellium());
 
 		else if (identifier.equals("stellium_bar")) {
@@ -284,17 +193,26 @@ public class RPGPlaceholders extends PlaceholderExpansion {
 			for (double j = 1; j < 20; j++)
 				format.append(ratio >= j ? ChatColor.BLUE : ratio >= j - .5 ? ChatColor.AQUA : ChatColor.WHITE).append(AltChar.listSquare);
 			return format.toString();
-        } else if (identifier.equals("quest")) {
+		}
+
+		else if (identifier.equals("quest")) {
 			PlayerQuests data = playerData.getQuestData();
 			return data.hasCurrent() ? data.getCurrent().getQuest().getName() : "None";
-        } else if (identifier.equals("quest_progress")) {
+		}
+
+		else if (identifier.equals("quest_progress")) {
 			PlayerQuests data = playerData.getQuestData();
+
 			return data.hasCurrent() ? MythicLib.plugin.getMMOConfig().decimal
-                    .format((double) data.getCurrent().getObjectiveNumber() / data.getCurrent().getQuest().getObjectives().size() * 100L) : "0";
+					.format( (double) data.getCurrent().getObjectiveNumber() / data.getCurrent().getQuest().getObjectives().size() * 100L) : "0";
-        } else if (identifier.equals("quest_objective")) {
+		}
+
+		else if (identifier.equals("quest_objective")) {
 			PlayerQuests data = playerData.getQuestData();
 			return data.hasCurrent() ? data.getCurrent().getFormattedLore() : "None";
-        } else if (identifier.startsWith("guild_")) {
+		}
+
+		else if (identifier.startsWith("guild_")) {
 			String placeholder = identifier.substring(6);
 			if (playerData.getGuild() == null)
 				return "";