new config system part 4

src/main/java/com/gmail/nossr50/DefaultKeys.java

@@ -1,5 +0,0 @@
-package com.gmail.nossr50;
-
-public interface DefaultKeys {
-    boolean validateKeys();
-}

src/main/java/com/gmail/nossr50/config/AdvancedConfig.java

@@ -9,12 +9,11 @@ import net.md_5.bungee.api.ChatColor;
 import java.util.ArrayList;
 import java.util.List;
 
-public class AdvancedConfig extends AutoUpdateConfigLoader {
+public class AdvancedConfig extends ConfigLoaderConfigurable {
     private static AdvancedConfig instance;
 
     private AdvancedConfig() {
-        super("advanced.yml");
-        validate();
+        super(mcMMO.p.getDataFolder(), "advanced.yml");
     }
 
     public static AdvancedConfig getInstance() {
@@ -26,7 +25,7 @@ public class AdvancedConfig extends AutoUpdateConfigLoader {
     }
 
     @Override
-    protected boolean validateKeys() {
+    public List<String> validateKeys() {
         // Validate all the settings!
         List<String> reason = new ArrayList<String>();
 
@@ -637,12 +636,9 @@ public class AdvancedConfig extends AutoUpdateConfigLoader {
             reason.add("Kraken.Attack_Damage should be at least 1!");
         }
 
-        return noErrorsInConfig(reason);
+        return reason;
     }
 
-    @Override
-    protected void loadKeys() {}
-
     /* GENERAL */
     public int getStartingLevel() { return config.getInt("Skills.General.StartingLevel", 1); }
 

src/main/java/com/gmail/nossr50/config/ConfigLoaderConfigurable.java

@@ -1,6 +1,5 @@
 package com.gmail.nossr50.config;
 
-import com.gmail.nossr50.DefaultKeys;
 import com.gmail.nossr50.mcMMO;
 import com.google.common.io.Files;
 import ninja.leaping.configurate.ConfigurationNode;
@@ -10,6 +9,7 @@ import ninja.leaping.configurate.yaml.YAMLConfigurationLoader;
 import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
+import java.util.List;
 
 /**
  * Handles loading and cacheing configuration settings from a configurable compatible config file
@@ -66,7 +66,19 @@ public abstract class ConfigLoaderConfigurable implements DefaultKeys {
             e.printStackTrace();
         }
 
-        validateKeys(); // Validate Keys
+        /*
+         * Print Errors about Keys
+         */
+
+        List<String> validKeyErrors = validateKeys(); // Validate Keys
+
+        if(validKeyErrors.size() > 0)
+        {
+            for(String error : validKeyErrors)
+            {
+                mcMMO.p.getLogger().severe(error);
+            }
+        }
     }
 
     /**

src/main/java/com/gmail/nossr50/config/DefaultKeys.java

@@ -0,0 +1,7 @@
+package com.gmail.nossr50.config;
+
+import java.util.List;
+
+public interface DefaultKeys {
+    List<String> validateKeys();
+}