Changeset 9b5a80ff in confclerk_git for src

Timestamp:

09/05/12 21:30:08 (10 years ago)

Author:

Philipp Spitzer <philipp@…>

Branches:

master, qt5

Children:

a8f8b5d

Parents:

1f12b6b

Message:

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

File:

• src/sql/sqlengine.cpp (modified) (1 diff)

src/sql/sqlengine.cpp

@@ -47 +47 @@
 
 void SqlEngine::open() {
+    // we may have to create the directory of the database
     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);