From 09944739cc5e05d1ff352c165f0d6e5b19c941a9 Mon Sep 17 00:00:00 2001 From: mgrojo Date: Mon, 20 May 2019 20:29:54 +0200 Subject: [PATCH] Log transaction and savepoints statements executed by application These statements have implications for the user, so it is better to log them. See #1859 and #1901 --- src/sqlitedb.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/sqlitedb.cpp b/src/sqlitedb.cpp index 842e5a8e..f7bfc2dd 100644 --- a/src/sqlitedb.cpp +++ b/src/sqlitedb.cpp @@ -525,7 +525,7 @@ bool DBBrowserDB::setSavepoint(const QString& pointname) return true; QString query = QString("SAVEPOINT %1;").arg(sqlb::escapeIdentifier(pointname)); - executeSQL(query, false, false); + executeSQL(query, false, true); savepointList.append(pointname); emit dbChanged(getDirty()); @@ -543,7 +543,7 @@ bool DBBrowserDB::releaseSavepoint(const QString& pointname) return true; QString query = QString("RELEASE %1;").arg(sqlb::escapeIdentifier(pointname)); - if(!executeSQL(query, false, false)) + if(!executeSQL(query, false, true)) return false; // SQLite releases all savepoints that were created between // creation of given savepoint and releasing of it, @@ -561,9 +561,9 @@ bool DBBrowserDB::revertToSavepoint(const QString& pointname) return false; QString query = QString("ROLLBACK TO SAVEPOINT %1;").arg(sqlb::escapeIdentifier(pointname)); - executeSQL(query, false, false); + executeSQL(query, false, true); query = QString("RELEASE %1;").arg(sqlb::escapeIdentifier(pointname)); - executeSQL(query, false, false); + executeSQL(query, false, true); // SQLite releases all savepoints that were created between // creation of given savepoint and releasing of it, // so we should too @@ -589,7 +589,7 @@ bool DBBrowserDB::releaseAllSavepoints() // When still in a transaction, commit that too if(sqlite3_get_autocommit(_db) == 0) - executeSQL("COMMIT;", false, false); + executeSQL("COMMIT;", false, true); return true; }