diff options
author | 2024-06-11 00:11:58 -0700 | |
---|---|---|
committer | 2024-06-11 00:11:58 -0700 | |
commit | 85cca718e308779a56dbf302140ce9088fc0727a (patch) | |
tree | ac0db2fa23482c6ef5caedd21079bf2f8c9be368 /src/SemanticScuttle/Service/Tag.php | |
parent | dec3464bffa14245b2cea076f2aac3a55283230d (diff) | |
download | scuttle-85cca718e308779a56dbf302140ce9088fc0727a.tar.gz scuttle-85cca718e308779a56dbf302140ce9088fc0727a.zip |
Throw an error when changes can't be committed to the databasesqlite-fixes
I was testing and another process had the database open, so it couldn't commit
changes. This wasn't apparent from the UI because it would just silently assume
a commit went fine.
Diffstat (limited to 'src/SemanticScuttle/Service/Tag.php')
-rw-r--r-- | src/SemanticScuttle/Service/Tag.php | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/SemanticScuttle/Service/Tag.php b/src/SemanticScuttle/Service/Tag.php index 17acae7..087a004 100644 --- a/src/SemanticScuttle/Service/Tag.php +++ b/src/SemanticScuttle/Service/Tag.php @@ -119,7 +119,13 @@ class SemanticScuttle_Service_Tag extends SemanticScuttle_DbService message_die(GENERAL_ERROR, 'Could not delete bookmarks', '', __LINE__, __FILE__, $query, $this->db); return false; } - $this->db->sql_transaction('commit'); + + if (!$this->db->sql_transaction('commit')) { + $this->db->sql_transaction('rollback'); + message_die(GENERAL_ERROR, 'Could not commit deleting bookmarks', '', __LINE__, __FILE__, $query, $this->db); + return false; + } + return true; } |