fix save values using byRef parameters

Author: gerdlanger

.gitignore

@@ -258,4 +258,7 @@ paket-files/
 
 # Python Tools for Visual Studio (PTVS)
 __pycache__/
-*.pyc
+*.pyc
+
+
+*.cpp__

src/ConfigTool.cpp

@@ -5,9 +5,7 @@
 
 #include "ConfigTool.h"
 #include <FS.h>
-#include <ArduinoJson.h>
 #include <SPIFFS.h>
-#include <map>
 
 void ConfigTool::load() {
 	if (!SPIFFS.begin(true)) {
@@ -21,7 +19,7 @@ void ConfigTool::load() {
 	DynamicJsonDocument root(ConfigSize);
 	deserializeJson(root, f.readStringUntil('\n'));
 	for (auto item : variables_) {
-		item.second->deserialize(root);
+		item.second->deserialize(&root);
 	}
 
 	f.close();
@@ -31,12 +29,13 @@ void ConfigTool::save() {
 	DynamicJsonDocument root(ConfigSize);
 
 	for (auto item : variables_) {
-		item.second->serialize(root);
+		item.second->serialize(&root);
 	}
-
-	SPIFFS.begin();
+	
+	SPIFFS.begin(true);
 	File f = SPIFFS.open("/config.json", "w");
 	serializeJson(root, f);
+	f.close();
 }
 
 void ConfigTool::reset() {

src/ConfigTool.h

@@ -5,15 +5,14 @@
 #ifndef _ConfigTool_h
 #define _ConfigTool_h
 
-#include <Arduino.h>
 #include <map>
 #include <ArduinoJson.h>
 
 
 struct BaseVar {
 	String name;
-	virtual void serialize(JsonDocument) = 0;
-	virtual void deserialize(JsonDocument) = 0;
+	virtual void serialize(JsonDocument*) = 0;
+	virtual void deserialize(JsonDocument*) = 0;
 	virtual void reset() = 0;
 	virtual String toString() = 0;
 	virtual void fromString(String) = 0;
@@ -23,9 +22,9 @@ template <typename T>
 struct ConfigVar : BaseVar {
 	ConfigVar(String n, T* p) {};
 
-	void deserialize(JsonDocument json) {};
+	void deserialize(JsonDocument* json) {};
 
-	void serialize(JsonDocument json) {};
+	void serialize(JsonDocument* json) {};
 
 	void reset() {};
 
@@ -44,12 +43,12 @@ struct ConfigVar<String> : BaseVar {
 		defaultValue = *p;
 	};
 
-	void deserialize(JsonDocument json) {
-		*pointer = String{ json[name] | defaultValue };
+	void deserialize(JsonDocument* json) {
+		*pointer = String{ (*json)[name] | defaultValue };
 	}
 
-	void serialize(JsonDocument json) {
-		json[name] = *pointer;
+	void serialize(JsonDocument* json) {
+		(*json)[name] = *pointer;
 	}
 
 	void reset() {
@@ -75,14 +74,14 @@ struct ConfigVar<bool> : BaseVar {
 		defaultValue = *p;
 	};
 
-	void deserialize(JsonDocument json) {
-		if (!json[name].isNull()) {
-			*pointer = json[name];
+	void deserialize(JsonDocument* json) {
+		if (!(*json)[name].isNull()) {
+			*pointer = (*json)[name];
 		}
 	}
 
-	void serialize(JsonDocument json) {
-		json[name] =  *pointer;
+	void serialize(JsonDocument* json) {
+		(*json)[name] =  *pointer;
 	}
 
 	void reset() {
@@ -108,14 +107,14 @@ struct ConfigVar<int> : BaseVar {
 		defaultValue = *p;
 	};
 
-	void deserialize(JsonDocument json) {
-		if (!json[name].isNull()) {
-			*pointer = json[name];
+	void deserialize(JsonDocument* json) {
+		if (!(*json)[name].isNull()) {
+			*pointer = (*json)[name];
 		}
 	}
 
-	void serialize(JsonDocument json) {
-		json[name] = *pointer;
+	void serialize(JsonDocument* json) {
+		(*json)[name] = *pointer;
 	}
 
 	void reset() {