* Changed Serializer::Deserialize to a non-static method.

* Fixed default config initialization.
* Added Firething call to delete stored config.

Author: ed7coyne

src/thing/Config.cpp

@@ -5,7 +5,7 @@
 namespace thing {
 
 ConfigJsonSerializer::ConfigJsonSerializer(const Config& config) {
-  JsonObject& root = json_.createObject();
+  JsonObject& root = buffer_.createObject();
   root_ = &root;
   root["host"] = config.host.c_str();
   root["auth"] = config.auth.c_str();
@@ -23,6 +23,10 @@ ConfigJsonSerializer::ConfigJsonSerializer(const Config& config) {
   pins_root["config_mode_button"] = config.pins.config_mode_button;
 }
 
+ConfigJsonSerializer::ConfigJsonSerializer(char* serialized_config) {
+  root_ = &(buffer_.parseObject(serialized_config));
+}
+
 int ConfigJsonSerializer::content_length() const {
   return root_->measureLength();
 }
@@ -35,26 +39,20 @@ void ConfigJsonSerializer::SerializeTo(Stream* output) {
   output->print(buffer);
 }
 
-Config ConfigJsonSerializer::Deserialize(char* string) {
-  Config config;
-
-  DynamicJsonBuffer jsonBuffer;
-  JsonObject& root = jsonBuffer.parseObject(string);
-  config.host = root["host"].asString();
-  config.auth = root["auth"].asString();
-  config.path = root["path"].asString();
-  config.wifi_ssid = root["wifi_ssid"].asString();
-  config.wifi_key = root["wifi_key"].asString();
-  config.analog_activation_threshold = root["activation_threshold"];
-  config.wifi_connect_attempts = root["wifi_connect_attempts"];
-
-  config.pins.digital_in = root["pins"]["digital_in"];
-  config.pins.digital_out = root["pins"]["digital_out"];
-  config.pins.analog_in = root["pins"]["analog_in"];
-  config.pins.analog_out = root["pins"]["analog_out"];
-  config.pins.config_mode_button = root["pins"]["config_mode_button"];
-
-  return config;
+void ConfigJsonSerializer::DeserializeTo(Config* config) {
+  config->host = root()["host"].asString();
+  config->auth = root()["auth"].asString();
+  config->path = root()["path"].asString();
+  config->wifi_ssid = root()["wifi_ssid"].asString();
+  config->wifi_key = root()["wifi_key"].asString();
+  config->analog_activation_threshold = root()["activation_threshold"];
+  config->wifi_connect_attempts = root()["wifi_connect_attempts"];
+
+  config->pins.digital_in = root()["pins"]["digital_in"];
+  config->pins.digital_out = root()["pins"]["digital_out"];
+  config->pins.analog_in = root()["pins"]["analog_in"];
+  config->pins.analog_out = root()["pins"]["analog_out"];
+  config->pins.config_mode_button = root()["pins"]["config_mode_button"];
 }
 
 };

src/thing/Config.h

@@ -35,13 +35,18 @@ struct Config {
 class ConfigJsonSerializer {
  public:
   ConfigJsonSerializer(const Config& config);
+
+  // We need a mutable char array here, otherwise a copy will be made.
+  ConfigJsonSerializer(char* config);
+
   int content_length() const;
   void SerializeTo(Stream* output);
+  void DeserializeTo(Config* config);
 
-  // We need a mutable char array here, otherwise a copy will be made.
-  static Config Deserialize(char* string);
  private:
-  DynamicJsonBuffer json_;
+  JsonObject& root() {return *root_;}
+
+  DynamicJsonBuffer buffer_;
   JsonObject* root_;
 };
 

src/thing/FireThing.cpp

@@ -12,11 +12,14 @@ Config kDefaultConfig = {
   "",  // wifi ssid
   "",  // wifi key
   0.1,  // analog activation threshold
+  2, // wifi connect attempts
+  {
   D1,  // digital in
   BUILTIN_LED,  // digital out
   A0,  // analog in
   D1, // analog out
   D0, // config mode button
+  }
 };
 
 const char kStorageFilename[] = "fthing.cfg";
@@ -32,8 +35,7 @@ bool FireThing::Setup() {
     return false;
   }
   SetPinModes(config);
-
-  if (digitalRead(config.pins.config_mode_button) || !ConnectToWiFi(config)) {
+  if (digitalRead(config.pins.config_mode_button) == HIGH || !ConnectToWiFi(config)) {
     wifi_.StartAP();
   }
 
@@ -54,9 +56,20 @@ void FireThing::Loop() {
   transcriber_.Loop();
 }
 
+bool FireThing::DeleteStoredConfig() {
+  if (!SPIFFS.begin()) {
+    debug_("Failed to mount FS.");
+    return false;
+  }
+  bool success = SPIFFS.remove(kStorageFilename);
+  SPIFFS.end();
+  return success;
+}
+
 bool FireThing::ConnectToWiFi(const Config& config) {
   debug_("Connecting to wifi:");
   debug_(config.wifi_ssid.c_str());
+  // TODO we should probably not print the key to serial.
   debug_(config.wifi_key.c_str());
   if (wifi_.Connect(config.wifi_ssid, config.wifi_key)) {
     debug_("Connected");
@@ -93,7 +106,8 @@ bool FireThing::ReadConfigFromStorage(Config* config) {
     }
     char buffer[cfg.size()];
     cfg.readBytes(buffer, cfg.size());
-    *config = ConfigJsonSerializer::Deserialize(buffer);
+    ConfigJsonSerializer serializer(buffer);
+    serializer.DeserializeTo(config);
     debug_("Config read from disk.");
   }
 

src/thing/FireThing.h

@@ -19,6 +19,9 @@ class FireThing {
 
   void SetDebugHandler(std::function<void(const char* message)> debug);
 
+  // Called to delete the currently stored config from the filesystem.
+  bool DeleteStoredConfig();
+
  private:
   bool ReadConfigFromStorage(Config* config);
   bool WriteConfigToStorage(const Config& config);

src/thing/Portal.cpp

@@ -132,8 +132,11 @@ void Portal::Start(const Config& config) {
         buffer = (char*)malloc(config.length()+1);
         memcpy(buffer, config.c_str(), config.length()+1);
       }
-      config_ = ConfigJsonSerializer::Deserialize(buffer);
-      free(buffer);
+      { // Scoped because serializer is invalid after free().
+        ConfigJsonSerializer serializer(buffer);
+        serializer.DeserializeTo(&config_);
+        free(buffer);
+      }
 
       callback_(config_);
       server_.send(200, "text/plain", "");