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/Bookmark2Tag.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/Bookmark2Tag.php')
-rw-r--r-- | src/SemanticScuttle/Service/Bookmark2Tag.php | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/src/SemanticScuttle/Service/Bookmark2Tag.php b/src/SemanticScuttle/Service/Bookmark2Tag.php index 2b95bd8..232fc80 100644 --- a/src/SemanticScuttle/Service/Bookmark2Tag.php +++ b/src/SemanticScuttle/Service/Bookmark2Tag.php @@ -208,7 +208,16 @@ class SemanticScuttle_Service_Bookmark2Tag extends SemanticScuttle_DbService 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 attaching tags', + '', __LINE__, __FILE__, $sql, $this->db + ); + return false; + } + return true; } |