Don't commit if there was an exception

ankihub/db/db_utils.py

@@ -40,9 +40,11 @@ def _execute_inner(self, sql: str, *args, first_row_only=False) -> List:
         except Exception as e:
             LOGGER.info(f"Error while executing SQL: {sql}")
             raise e
-        finally:
+        else:
             if not self._is_used_as_context_manager:
                 self._conn.commit()
+        finally:
+            if not self._is_used_as_context_manager:
                 self._conn.close()
 
         return result
@@ -71,7 +73,9 @@ def __enter__(self):
         return self
 
     def __exit__(self, exc_type, exc_val, exc_tb):
-        self._conn.commit()
+        if exc_type is None:
+            self._conn.commit()
+
         self._conn.close()
         self._is_used_as_context_manager = False
         self._lock_context.__exit__(exc_type, exc_val, exc_tb)