From aff1108836410f47e7df0f1c8cc0838cceaabfe0 Mon Sep 17 00:00:00 2001 From: Florian Bruhin Date: Sat, 1 Sep 2018 12:09:01 +0200 Subject: [PATCH] sql.Query: Make sure transaction/commit succeed --- qutebrowser/misc/sql.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/qutebrowser/misc/sql.py b/qutebrowser/misc/sql.py index 53c1275be..eb31834e5 100644 --- a/qutebrowser/misc/sql.py +++ b/qutebrowser/misc/sql.py @@ -201,11 +201,14 @@ class Query: raise SqlError("Missing bound values!") db = QSqlDatabase.database() - db.transaction() + ok = db.transaction() + self._check_ok('transaction', ok) ok = self.query.execBatch() self._check_ok('execBatch', ok) - db.commit() + + ok = db.commit() + self._check_ok('commit', ok) def value(self): """Return the result of a single-value query (e.g. an EXISTS)."""