Properly catch and handle database errors

Do not use a singleton for the database.

fix #3203

1 files changed, 18 insertions, 1 deletions

diff --git a/louloulibs/utils/reload.cpp b/louloulibs/utils/reload.cpp
index bddfd86..7125a75 100644
--- a/louloulibs/utils/reload.cpp
+++ b/louloulibs/utils/reload.cpp
@@ -1,11 +1,28 @@
+#include <utils/reload.hpp>
+#include <database/database.hpp>
 #include <config/config.hpp>
+#include <utils/xdg.hpp>
 #include <logger/logger.hpp>
 
+void open_database()
+{
+  const auto db_filename = Config::get("db_name", xdg_data_path("biboumi.sqlite"));
+  log_info("Opening database: ", db_filename);
+  Database::open(db_filename);
+  log_info("database successfully opened.");
+}
+
 void reload_process()
 {
   Config::read_conf();
   // Destroy the logger instance, to be recreated the next time a log
   // line needs to be written
   Logger::instance().reset();
-  log_debug("Configuration and logger reloaded.");
+  log_info("Configuration and logger reloaded.");
+  try {
+      open_database();
+    } catch (const litesql::DatabaseError&) {
+      log_warning("Re-using the previous database.");
+    }
 }
+