Bugfix for storing transaction messages for new created characters.

First flush the new character to the database to get it's new DatabaseID.
Calling ->getDatabaseID() before flush() will give a wrong, uninitialized
value.

1 files changed, 3 insertions, 2 deletions

diff --git a/src/account-server/accounthandler.cpp b/src/account-server/accounthandler.cpp
index d04671e2..09e849bd 100644
--- a/src/account-server/accounthandler.cpp
+++ b/src/account-server/accounthandler.cpp
@@ -607,15 +607,16 @@ static void handleCharacterCreateMessage(AccountClient &computer, MessageIn &msg
             LOG_INFO("Character " << name << " was created for "
                      << acc->getName() << "'s account.");
 
+            storage->flush(acc); // flush changes
+
             // log transaction
             Transaction trans;
             trans.mCharacterId = newCharacter->getDatabaseID();
             trans.mAction = TRANS_CHAR_CREATE;
             trans.mMessage = acc->getName() + " created character ";
-            trans.mMessage.append(" called " + name);
+            trans.mMessage.append("called " + name);
             storage->addTransaction(trans);
 
-            storage->flush(acc); // flush changes
             reply.writeByte(ERRMSG_OK);
             computer.send(reply);