Have the PerformTransaction class automatically handle nesting

No need to fiddle around with "startTransaction" booleans now that the
helper class is a little more intelligent. When the database is already
performing a transaction, no new one will be started.

1 files changed, 3 insertions, 4 deletions

diff --git a/src/account-server/storage.hpp b/src/account-server/storage.hpp
index 8651d20e..27d12997 100644
--- a/src/account-server/storage.hpp
+++ b/src/account-server/storage.hpp
@@ -74,8 +74,8 @@ class Storage
 
         void banCharacter(int id, int duration);
 
-        void delCharacter(int charId, bool startTransaction) const;
-        void delCharacter(Character *character, bool startTransaction) const;
+        void delCharacter(int charId) const;
+        void delCharacter(Character *character) const;
 
         void checkBannedAccounts();
 
@@ -83,8 +83,7 @@ class Storage
         bool doesEmailAddressExist(const std::string &email);
         bool doesCharacterNameExist(const std::string &name);
 
-        bool updateCharacter(Character *ptr,
-                             bool startTransaction = true);
+        bool updateCharacter(Character *ptr);
 
         void flushSkill(const Character *character, int skill_id);