Castopod
/
kasts
mirror of https://github.com/KDE/kasts.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Use transactions to do database migration 

This was already implemented for 2 -> 3.
Now also for 1 -> 2.

Additionally, remove unnecessary whitespace in debug statement
This commit is contained in:
Bart De Vries 2021-07-06 10:54:41 +02:00
parent 104cafbd41
commit a5a24fb08c
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/database.cpp
View File

@ -71,9 +71,11 @@ bool Database::migrateTo1()
bool Database::migrateTo2() bool Database::migrateTo2()
{ {
qDebug() << "Migrating database to version 2"; qDebug() << "Migrating database to version 2";
TRUE_OR_RETURN(execute(QStringLiteral("BEGIN TRANSACTION;")));
TRUE_OR_RETURN(execute(QStringLiteral("DROP TABLE Errors;"))); TRUE_OR_RETURN(execute(QStringLiteral("DROP TABLE Errors;")));
TRUE_OR_RETURN(execute(QStringLiteral("CREATE TABLE IF NOT EXISTS Errors (type INTEGER, url TEXT, id TEXT, code INTEGER, message TEXT, date INTEGER);"))); TRUE_OR_RETURN(execute(QStringLiteral("CREATE TABLE IF NOT EXISTS Errors (type INTEGER, url TEXT, id TEXT, code INTEGER, message TEXT, date INTEGER);")));
TRUE_OR_RETURN(execute(QStringLiteral("PRAGMA user_version = 2;"))); TRUE_OR_RETURN(execute(QStringLiteral("PRAGMA user_version = 2;")));
TRUE_OR_RETURN(execute(QStringLiteral("COMMIT;")));
return true; return true;
} }
@ -121,7 +123,7 @@ int Database::version()
if (query.next()) { if (query.next()) {
bool ok; bool ok;
int value = query.value(0).toInt(&ok); int value = query.value(0).toInt(&ok);
qDebug() << "Database version " << value; qDebug() << "Database version" << value;
if (ok) if (ok)
return value; return value;
} else { } else {