Append _F or _M to username in eAthena network layer

Abstracts it away and out of main.cpp and register.cpp.

7 files changed, 57 insertions, 48 deletions

diff --git a/src/net/ea/charserverhandler.cpp b/src/net/ea/charserverhandler.cpp
index 2973fc4c..3402b5fc 100644
--- a/src/net/ea/charserverhandler.cpp
+++ b/src/net/ea/charserverhandler.cpp
@@ -170,8 +170,7 @@ void CharServerHandler::handleMessage(MessageIn &msg)
 LocalPlayer *CharServerHandler::readPlayerData(MessageIn &msg, int &slot)
 {
     LocalPlayer *tempPlayer = new LocalPlayer(mLoginData->account_ID, 0, NULL);
-    tempPlayer->setGender(
-            (mLoginData->sex == 0) ? GENDER_FEMALE : GENDER_MALE);
+    tempPlayer->setGender(mLoginData->sex);
 
     tempPlayer->mCharId = msg.readInt32();
     tempPlayer->setXp(msg.readInt32());
@@ -218,7 +217,8 @@ void CharServerHandler::setCharCreateDialog(CharCreateDialog *window)
 {
     mCharCreateDialog = window;
 
-    if (!mCharCreateDialog) return;
+    if (!mCharCreateDialog)
+        return;
 
     std::vector<std::string> attributes;
     attributes.push_back(_("Strength:"));
@@ -229,7 +229,7 @@ void CharServerHandler::setCharCreateDialog(CharCreateDialog *window)
     attributes.push_back(_("Luck:"));
 
     mCharCreateDialog->setAttributes(attributes, 30, 1, 9);
-    mCharCreateDialog->setFixedGender(true);
+    mCharCreateDialog->setFixedGender(true, mLoginData->sex);
 }
 
 void CharServerHandler::connect(LoginData *loginData)
@@ -243,7 +243,7 @@ void CharServerHandler::connect(LoginData *loginData)
     // [Fate] The next word is unused by the old char server, so we squeeze in
     //        tmw client version information
     outMsg.writeInt16(CLIENT_PROTOCOL_VERSION);
-    outMsg.writeInt8(loginData->sex);
+    outMsg.writeInt8((loginData->sex == GENDER_MALE) ? 1 : 0);
 
     // We get 4 useless bytes before the real answer comes in (what are these?)
     mNetwork->skip(4);
diff --git a/src/net/ea/loginhandler.cpp b/src/net/ea/loginhandler.cpp
index 1031ee05..9c34c4cd 100644
--- a/src/net/ea/loginhandler.cpp
+++ b/src/net/ea/loginhandler.cpp
@@ -81,7 +81,7 @@ void LoginHandler::handleMessage(MessageIn &msg)
             mLoginData->account_ID = msg.readInt32();
             mLoginData->session_ID2 = msg.readInt32();
             msg.skip(30);                           // unknown
-            mLoginData->sex = msg.readInt8();
+            mLoginData->sex = msg.readInt8() ? GENDER_MALE : GENDER_FEMALE;
 
             for (int i = 0; i < n_server; i++)
             {
@@ -146,18 +146,7 @@ void LoginHandler::handleMessage(MessageIn &msg)
 void LoginHandler::loginAccount(LoginData *loginData)
 {
     mLoginData = loginData;
-    MessageOut outMsg(0x0064);
-    outMsg.writeInt32(0); // client version
-    outMsg.writeString(loginData->username, 24);
-    outMsg.writeString(loginData->password, 24);
-
-    /*
-     * eAthena calls the last byte "client version 2", but it isn't used at
-     * at all. We're retasking it, with bit 0 to indicate whether the client
-     * can handle the 0x63 "update host" packet. Clients prior to 0.0.25 send
-     * 0 here.
-     */
-    outMsg.writeInt8(0x01);
+    sendLoginRegister(loginData->username, loginData->password);
 }
 
 void LoginHandler::changeEmail(const std::string &email)
@@ -166,8 +155,8 @@ void LoginHandler::changeEmail(const std::string &email)
 }
 
 void LoginHandler::changePassword(const std::string &username,
-                   const std::string &oldPassword,
-                   const std::string &newPassword)
+                                  const std::string &oldPassword,
+                                  const std::string &newPassword)
 {
     // TODO
 }
@@ -177,17 +166,37 @@ void LoginHandler::chooseServer(int server)
     // TODO
 }
 
-void LoginHandler::registerAccount(const std::string &username,
-                   const std::string &password,
-                   const std::string &email = "")
+void LoginHandler::registerAccount(LoginData *loginData)
 {
-    // TODO
+    mLoginData = loginData;
+
+    std::string username = loginData->username;
+    username.append((loginData->sex == GENDER_FEMALE) ? "_F" : "_M");
+
+    sendLoginRegister(username, loginData->password);
 }
 
 void LoginHandler::unregisterAccount(const std::string &username,
-                   const std::string &password)
+                                     const std::string &password)
 {
     // TODO
 }
 
+void LoginHandler::sendLoginRegister(const std::string &username,
+                                     const std::string &password)
+{
+    MessageOut outMsg(0x0064);
+    outMsg.writeInt32(0); // client version
+    outMsg.writeString(username, 24);
+    outMsg.writeString(password, 24);
+
+    /*
+     * eAthena calls the last byte "client version 2", but it isn't used at
+     * at all. We're retasking it, with bit 0 to indicate whether the client
+     * can handle the 0x63 "update host" packet. Clients prior to 0.0.25 send
+     * 0 here.
+     */
+    outMsg.writeInt8(0x01);
+}
+
 } // namespace EAthena
diff --git a/src/net/ea/loginhandler.h b/src/net/ea/loginhandler.h
index cdc59e4a..93f21754 100644
--- a/src/net/ea/loginhandler.h
+++ b/src/net/ea/loginhandler.h
@@ -43,19 +43,20 @@ class LoginHandler : public MessageHandler, public Net::LoginHandler
         void changeEmail(const std::string &email);
 
         void changePassword(const std::string &username,
-                           const std::string &oldPassword,
-                           const std::string &newPassword);
+                            const std::string &oldPassword,
+                            const std::string &newPassword);
 
         void chooseServer(int server);
 
-        void registerAccount(const std::string &username,
-                           const std::string &password,
-                           const std::string &email);
+        void registerAccount(LoginData *loginData);
 
         void unregisterAccount(const std::string &username,
-                           const std::string &password);
+                               const std::string &password);
 
     private:
+        void sendLoginRegister(const std::string &username,
+                               const std::string &password);
+
         LoginData *mLoginData;
         std::string mUpdateHost;
 };
diff --git a/src/net/ea/maphandler.cpp b/src/net/ea/maphandler.cpp
index de60059f..6b061798 100644
--- a/src/net/ea/maphandler.cpp
+++ b/src/net/ea/maphandler.cpp
@@ -87,7 +87,7 @@ void MapHandler::connect(LoginData *loginData)
     outMsg.writeInt32(player_node->mCharId);
     outMsg.writeInt32(loginData->session_ID1);
     outMsg.writeInt32(loginData->session_ID2);
-    outMsg.writeInt8(loginData->sex);
+    outMsg.writeInt8((loginData->sex == GENDER_MALE) ? 1 : 0);
 
     // We get 4 useless bytes before the real answer comes in (what are these?)
     mNetwork->skip(4);
diff --git a/src/net/loginhandler.h b/src/net/loginhandler.h
index fcdee0da..fdd57689 100644
--- a/src/net/loginhandler.h
+++ b/src/net/loginhandler.h
@@ -27,6 +27,7 @@
 #include <iosfwd>
 
 namespace Net {
+
 class LoginHandler
 {
     public:
@@ -35,18 +36,17 @@ class LoginHandler
         virtual void changeEmail(const std::string &email) = 0;
 
         virtual void changePassword(const std::string &username,
-                           const std::string &oldPassword,
-                           const std::string &newPassword) = 0;
+                                    const std::string &oldPassword,
+                                    const std::string &newPassword) = 0;
 
         virtual void chooseServer(int server) = 0;
 
-        virtual void registerAccount(const std::string &username,
-                           const std::string &password,
-                           const std::string &email) = 0;
+        virtual void registerAccount(LoginData *loginData) = 0;
 
         virtual void unregisterAccount(const std::string &username,
-                           const std::string &password) = 0;
+                                       const std::string &password) = 0;
 };
-}
+
+} // namespace Net
 
 #endif // LOGINHANDLER_H
diff --git a/src/net/tmwserv/loginhandler.cpp b/src/net/tmwserv/loginhandler.cpp
index ef23546a..c8db624c 100644
--- a/src/net/tmwserv/loginhandler.cpp
+++ b/src/net/tmwserv/loginhandler.cpp
@@ -258,12 +258,13 @@ void LoginHandler::chooseServer(int server)
     // TODO
 }
 
-void LoginHandler::registerAccount(const std::string &username,
-                        const std::string &password, const std::string &email)
+void LoginHandler::registerAccount(LoginData *loginData)
 {
     Net::AccountServer::registerAccount(accountServerConnection,
             0, // client version
-            username, password, email);
+            loginData->username,
+            loginData->password,
+            loginData->email);
 }
 
 void LoginHandler::unregisterAccount(const std::string &username,
diff --git a/src/net/tmwserv/loginhandler.h b/src/net/tmwserv/loginhandler.h
index 93d55396..f3bc0c6d 100644
--- a/src/net/tmwserv/loginhandler.h
+++ b/src/net/tmwserv/loginhandler.h
@@ -41,17 +41,15 @@ class LoginHandler : public MessageHandler, public Net::LoginHandler
         void changeEmail(const std::string &email);
 
         void changePassword(const std::string &username,
-                           const std::string &oldPassword,
-                           const std::string &newPassword);
+                            const std::string &oldPassword,
+                            const std::string &newPassword);
 
         void chooseServer(int server);
 
-        void registerAccount(const std::string &username,
-                           const std::string &password,
-                           const std::string &email);
+        void registerAccount(LoginData *loginData);
 
         void unregisterAccount(const std::string &username,
-                           const std::string &password);
+                               const std::string &password);
 
     private:
         void handleLoginResponse(MessageIn &msg);