Only save config if changed, number conversion fixes

And do save config if changed
#83

BuildConfigUpdater/BuildConfigUpdater.iml

@@ -18,6 +18,7 @@
     <orderEntry type="inheritedJdk" />
     <orderEntry type="sourceFolder" forTests="false" />
     <orderEntry type="module" module-name="ButtonCore (1) (com.github.TBMCPlugins.ButtonCore)" />
+    <orderEntry type="module" module-name="ButtonCore (1) (com.github.TBMCPlugins.ButtonCore)" />
     <orderEntry type="library" name="Maven: org.reflections:reflections:0.9.10" level="project" />
     <orderEntry type="library" name="Maven: com.google.code.findbugs:annotations:2.0.1" level="project" />
     <orderEntry type="library" name="Maven: org.javassist:javassist:3.20.0-GA" level="project" />

ButtonCore/src/main/java/buttondevteam/lib/architecture/ConfigData.java

@@ -15,7 +15,6 @@ import org.bukkit.scheduler.BukkitTask;
 import java.lang.reflect.Array;
 import java.util.HashMap;
 import java.util.List;
-import java.util.Objects;
 import java.util.function.BiFunction;
 import java.util.function.Function;
 
@@ -49,10 +48,6 @@ public class ConfigData<T> {
 	 * The config value should not change outside this instance
 	 */
 	private T value;
-	/**
-	 * Whether the default value is saved in the yaml
-	 */
-	private boolean saved = false;
 
 	//This constructor is needed because it sets the getter and setter
 	ConfigData(ConfigurationSection config, String path, T def, Object primitiveDef, Function<Object, T> getter, Function<T, Object> setter, Runnable saveAction) {
@@ -85,22 +80,25 @@ public class ConfigData<T> {
 	@SuppressWarnings("unchecked")
 	public T get() {
 		if (value != null) return value; //Speed things up
-		Object val = config == null ? null : config.get(path); //config==null: testing
+		Object val;
-		if (val == null) {
+		if (config == null || !config.isSet(path)) { //Call set() if config == null
 			val = primitiveDef;
-		}
-		if (!saved && Objects.equals(val, primitiveDef)) { //String needs .equals()
 			if (def == null && config != null) //In Discord's case def may be null
-				config.set(path, primitiveDef);
+				setInternal(primitiveDef);
 			else
 				set(def); //Save default value - def is always set
-			saved = true;
+		} else
-		}
+			val = config.get(path); //config==null: testing
-		BiFunction<Object, Object, Object> convert=(_val, _def) -> {
+		if (val == null) //If it's set to null explicitly
-			if (_val instanceof Number && _def != null)
+			val = primitiveDef;
-				_val = ChromaUtils.convertNumber((Number) _val,
+		BiFunction<Object, Object, Object> convert = (_val, _def) -> {
-					(Class<? extends Number>) _def.getClass());
+			if (_def instanceof Number) //If we expect a number
-			if (_val instanceof List && _def != null && _def.getClass().isArray())
+				if (_val instanceof Number)
+					_val = ChromaUtils.convertNumber((Number) _val,
+						(Class<? extends Number>) _def.getClass());
+				else
+					_val = _def; //If we didn't get a number, return default (which is a number)
+			else if (_val instanceof List && _def != null && _def.getClass().isArray())
 				_val = ((List<T>) _val).toArray((T[]) Array.newInstance(_def.getClass().getComponentType(), 0));
 			return _val;
 		};
@@ -121,20 +119,23 @@ public class ConfigData<T> {
 		if (setter != null && value != null)
 			val = setter.apply(value);
 		else val = value;
-		if (config != null) {
+		if (config != null)
-			config.set(path, val);
+			setInternal(val);
-			if (!saveTasks.containsKey(config.getRoot())) {
+		this.value = value;
-				synchronized (saveTasks) {
+	}
-					saveTasks.put(config.getRoot(), new SaveTask(Bukkit.getScheduler().runTaskLaterAsynchronously(MainPlugin.Instance, () -> {
+
-						synchronized (saveTasks) {
+	private void setInternal(Object val) {
-							saveTasks.remove(config.getRoot());
+		config.set(path, val);
-							saveAction.run();
+		if (!saveTasks.containsKey(config.getRoot())) {
-						}
+			synchronized (saveTasks) {
-					}, 100), saveAction));
+				saveTasks.put(config.getRoot(), new SaveTask(Bukkit.getScheduler().runTaskLaterAsynchronously(MainPlugin.Instance, () -> {
-				}
+					synchronized (saveTasks) {
+						saveTasks.remove(config.getRoot());
+						saveAction.run();
+					}
+				}, 100), saveAction));
 			}
 		}
-		this.value = value;
 	}
 
 	@AllArgsConstructor