Introduced a helper class for performing transactions

Avoids having to remember to call rollbackTransaction and makes
transactions exception-safe (since the destructor of PerformTransaction
will be called when an exception is thrown).

1 files changed, 2 insertions, 2 deletions

diff --git a/src/account-server/serverhandler.cpp b/src/account-server/serverhandler.cpp
index 9ca9e281..b81be075 100644
--- a/src/account-server/serverhandler.cpp
+++ b/src/account-server/serverhandler.cpp
@@ -553,7 +553,7 @@ void GameServerHandler::sendPartyChange(Character *ptr, int partyId)
 void GameServerHandler::syncDatabase(MessageIn &msg)
 {
     // It is safe to perform the following updates in a transaction
-    storage->database()->beginTransaction();
+    dal::PerformTransaction transaction(storage->database());
 
     int msgType = msg.readByte();
     while (msgType != SYNC_END_OF_BUFFER && msg.getUnreadLength() > 0)
@@ -602,5 +602,5 @@ void GameServerHandler::syncDatabase(MessageIn &msg)
         msgType = msg.readByte();
     }
 
-    storage->database()->commitTransaction();
+    transaction.commit();
 }