Added a second possibility to create the directory and removed the TODO.

[toast/confclerk.git] / src / sql / sqlengine.cpp

diff --git a/src/sql/sqlengine.cpp b/src/sql/sqlengine.cpp
index a0771d7d1bc36195c25aa45caeabba0026e35dae..62d27e3d534be0f2ea51ef9060f71b2fb1c68a92 100644 (file)
--- a/src/sql/sqlengine.cpp
+++ b/src/sql/sqlengine.cpp
@@ -46,8 +46,18 @@ SqlEngine::~SqlEngine() {
 
 
 void SqlEngine::open() {
 
 
 void SqlEngine::open() {

+    // we may have to create the directory of the database

     QFileInfo dbFilenameInfo(dbFilename);
     QFileInfo dbFilenameInfo(dbFilename);

-    dbFilenameInfo.absoluteDir().mkpath("."); // TODO: handle error
+
+    // First possibility (works with Linux, not tested on Windows:
+    // QDir cwd;
+    // cwd.mkpath(dbFilenameInfo.absolutePath());
+
+    // Second possibility (works on Windows)
+    dbFilenameInfo.absoluteDir().mkpath(".");
+
+    // We don't have to handle errors because in worst case, opening the database will fail
+    // and db.isOpen() returns false.

     db = QSqlDatabase::addDatabase("QSQLITE");
     db.setDatabaseName(dbFilename);
     db.open();
     db = QSqlDatabase::addDatabase("QSQLITE");
     db.setDatabaseName(dbFilename);
     db.open();