Extract shared logic from npchandler and playerhandler netcode to ea namespace.

4 files changed, 60 insertions, 704 deletions

diff --git a/src/net/tmwa/npchandler.cpp b/src/net/tmwa/npchandler.cpp
index 9bcc640a1..d6549e39f 100644
--- a/src/net/tmwa/npchandler.cpp
+++ b/src/net/tmwa/npchandler.cpp
@@ -22,13 +22,11 @@
 
 #include "net/tmwa/npchandler.h"
 
-#include "actorspritemanager.h"
 #include "localplayer.h"
 
 #include "gui/npcdialog.h"
 
 #include "net/messagein.h"
-#include "net/messageout.h"
 #include "net/net.h"
 #include "net/npchandler.h"
 
@@ -36,8 +34,6 @@
 
 #include "net/ea/eaprotocol.h"
 
-#include <SDL_types.h>
-
 #include "debug.h"
 
 extern Net::NpcHandler *npcHandler;
@@ -63,82 +59,33 @@ NpcHandler::NpcHandler()
 
 void NpcHandler::handleMessage(Net::MessageIn &msg)
 {
-    if (msg.getId() == SMSG_NPC_CHOICE || msg.getId() == SMSG_NPC_MESSAGE)
-        msg.readInt16();  // length
-
-    int npcId = msg.readInt32();
-    NpcDialogs::iterator diag = mNpcDialogs.find(npcId);
-    NpcDialog *dialog = 0;
-
-    if (diag == mNpcDialogs.end())
-    {
-        // Empty dialogs don't help
-        if (msg.getId() == SMSG_NPC_CLOSE)
-        {
-            closeDialog(npcId);
-            return;
-        }
-        else if (msg.getId() == SMSG_NPC_NEXT)
-        {
-            nextDialog(npcId);
-            return;
-        }
-        else
-        {
-            dialog = new NpcDialog(npcId);
-            Wrapper wrap;
-            wrap.dialog = dialog;
-            mNpcDialogs[npcId] = wrap;
-        }
-    }
-    else
-    {
-        dialog = diag->second.dialog;
-    }
+    getNpc(msg, msg.getId() == SMSG_NPC_CHOICE
+        || msg.getId() == SMSG_NPC_MESSAGE);
 
     switch (msg.getId())
     {
         case SMSG_NPC_CHOICE:
-            if (dialog)
-            {
-                dialog->choiceRequest();
-                dialog->parseListItems(msg.readString(msg.getLength() - 8));
-            }
-            else
-            {
-                msg.readString(msg.getLength() - 8);
-            }
+            processNpcChoice(msg);
             break;
 
         case SMSG_NPC_MESSAGE:
-            if (dialog)
-                dialog->addText(msg.readString(msg.getLength() - 8));
-            else
-                msg.readString(msg.getLength() - 8);
+            processNpcMessage(msg);
             break;
 
         case SMSG_NPC_CLOSE:
-            // Show the close button
-            if (dialog)
-                dialog->showCloseButton();
+            processNpcClose(msg);
             break;
 
         case SMSG_NPC_NEXT:
-            // Show the next button
-            if (dialog)
-                dialog->showNextButton();
+            processNpcNext(msg);
             break;
 
         case SMSG_NPC_INT_INPUT:
-            // Request for an integer
-            if (dialog)
-                dialog->integerRequest(0);
+            processNpcIntInput(msg);
             break;
 
         case SMSG_NPC_STR_INPUT:
-            // Request for a string
-            if (dialog)
-                dialog->textRequest("");
+            processNpcStrInput(msg);
             break;
 
         default:
@@ -147,6 +94,8 @@ void NpcHandler::handleMessage(Net::MessageIn &msg)
 
     if (player_node && player_node->getCurrentAction() != Being::SIT)
         player_node->setAction(Being::STAND);
+    
+    mDialog = 0;
 }
 
 void NpcHandler::talk(int npcId)
@@ -199,18 +148,6 @@ void NpcHandler::stringInput(int npcId, const std::string &value)
     outMsg.writeInt8(0); // Prevent problems with string reading
 }
 
-void NpcHandler::sendLetter(int npcId A_UNUSED,
-                            const std::string &recipient A_UNUSED,
-                            const std::string &text A_UNUSED)
-{
-    // TODO
-}
-
-void NpcHandler::startShopping(int beingId A_UNUSED)
-{
-    // TODO
-}
-
 void NpcHandler::buy(int beingId)
 {
     MessageOut outMsg(CMSG_NPC_BUY_SELL_REQUEST);
@@ -253,21 +190,42 @@ void NpcHandler::sellItem(int beingId A_UNUSED, int itemId, int amount)
     outMsg.writeInt16(static_cast<Sint16>(amount));
 }
 
-void NpcHandler::endShopping(int beingId A_UNUSED)
+int NpcHandler::getNpc(Net::MessageIn &msg, bool haveLength)
 {
-    // TODO
-}
+    if (haveLength)
+        msg.readInt16();  // length
 
-void NpcHandler::clearDialogs()
-{
-    NpcDialogs::iterator it = mNpcDialogs.begin();
-    NpcDialogs::iterator it_end = mNpcDialogs.end();
-    while (it != it_end)
+    const int npcId = msg.readInt32();
+
+    NpcDialogs::iterator diag = mNpcDialogs.find(npcId);
+    mDialog = 0;
+
+    if (diag == mNpcDialogs.end())
+    {
+        // Empty dialogs don't help
+        if (msg.getId() == SMSG_NPC_CLOSE)
+        {
+            closeDialog(npcId);
+            return npcId;
+        }
+        else if (msg.getId() == SMSG_NPC_NEXT)
+        {
+            nextDialog(npcId);
+            return npcId;
+        }
+        else
+        {
+            mDialog = new NpcDialog(npcId);
+            Wrapper wrap;
+            wrap.dialog = mDialog;
+            mNpcDialogs[npcId] = wrap;
+        }
+    }
+    else
     {
-        delete (*it).second.dialog;
-        ++ it;
+        mDialog = diag->second.dialog;
     }
-    mNpcDialogs.clear();
+    return npcId;
 }
 
 } // namespace TmwAthena
diff --git a/src/net/tmwa/npchandler.h b/src/net/tmwa/npchandler.h
index 1b4455854..624f64cbc 100644
--- a/src/net/tmwa/npchandler.h
+++ b/src/net/tmwa/npchandler.h
@@ -26,6 +26,8 @@
 #include "net/net.h"
 #include "net/npchandler.h"
 
+#include "net/ea/npchandler.h"
+
 #include "net/tmwa/messagehandler.h"
 
 #include <map>
@@ -41,7 +43,7 @@ class NpcDialog;
 namespace TmwAthena
 {
 
-class NpcHandler : public MessageHandler, public Net::NpcHandler
+class NpcHandler : public MessageHandler, public Ea::NpcHandler
 {
     public:
         NpcHandler();
@@ -60,11 +62,6 @@ class NpcHandler : public MessageHandler, public Net::NpcHandler
 
         void stringInput(int npcId, const std::string &value);
 
-        void sendLetter(int npcId, const std::string &recipient,
-                        const std::string &text);
-
-        void startShopping(int beingId);
-
         void buy(int beingId);
 
         void sell(int beingId);
@@ -73,17 +70,7 @@ class NpcHandler : public MessageHandler, public Net::NpcHandler
 
         void sellItem(int beingId, int itemId, int amount);
 
-        void endShopping(int beingId);
-
-        void clearDialogs();
-
-    private:
-        typedef struct
-        {
-            NpcDialog* dialog;
-        } Wrapper;
-        typedef std::map<int, Wrapper> NpcDialogs;
-        NpcDialogs mNpcDialogs;
+        int getNpc(Net::MessageIn &msg, bool haveLength);
 };
 
 } // namespace TmwAthena
diff --git a/src/net/tmwa/playerhandler.cpp b/src/net/tmwa/playerhandler.cpp
index 18bed203f..f20ea64a0 100644
--- a/src/net/tmwa/playerhandler.cpp
+++ b/src/net/tmwa/playerhandler.cpp
@@ -22,129 +22,16 @@
 
 #include "net/tmwa/playerhandler.h"
 
-#include "event.h"
-#include "game.h"
-#include "localplayer.h"
 #include "log.h"
-#include "party.h"
-#include "playerinfo.h"
-#include "units.h"
-
-#include "gui/gui.h"
-#include "gui/okdialog.h"
-#include "gui/skilldialog.h"
-#include "gui/statuswindow.h"
-#include "gui/viewport.h"
 
 #include "net/messagein.h"
-#include "net/messageout.h"
 
 #include "net/tmwa/protocol.h"
 #include "net/tmwa/npchandler.h"
 #include "net/tmwa/inventoryhandler.h"
 
-#include "utils/stringutils.h"
-#include "utils/gettext.h"
-
 #include "debug.h"
 
-extern OkDialog *weightNotice;
-extern OkDialog *deathNotice;
-extern int weightNoticeTime;
-
-// Max. distance we are willing to scroll after a teleport;
-// everything beyond will reset the port hard.
-static const int MAP_TELEPORT_SCROLL_DISTANCE = 8;
-
-// TODO Move somewhere else
-namespace
-{
-    /**
-     * Listener used for handling the overweigth message.
-     */
-    struct WeightListener : public gcn::ActionListener
-    {
-        void action(const gcn::ActionEvent &event A_UNUSED)
-        {
-            weightNotice = NULL;
-        }
-    } weightListener;
-
-    /**
-     * Listener used for handling death message.
-     */
-    struct DeathListener : public gcn::ActionListener
-    {
-        void action(const gcn::ActionEvent &event A_UNUSED)
-        {
-            Net::getPlayerHandler()->respawn();
-            deathNotice = NULL;
-
-            Client::closeDialogs();
-
-            if (viewport)
-                viewport->closePopupMenu();
-
-            TmwAthena::NpcHandler *handler =
-                    static_cast<TmwAthena::NpcHandler*>(Net::getNpcHandler());
-            if (handler)
-                handler->clearDialogs();
-            if (player_node)
-                player_node->respawn();
-        }
-    } deathListener;
-
-} // anonymous namespace
-
-static const char *randomDeathMessage()
-{
-    static char const *const deadMsg[] =
-    {
-        N_("You are dead."),
-        N_("We regret to inform you that your character was killed in "
-           "battle."),
-        N_("You are not that alive anymore."),
-        N_("The cold hands of the grim reaper are grabbing for your soul."),
-        N_("Game Over!"),
-        N_("Insert coin to continue."),
-        N_("No, kids. Your character did not really die. It... "
-           "err... went to a better place."),
-        N_("Your plan of breaking your enemies weapon by "
-           "bashing it with your throat failed."),
-        N_("I guess this did not run too well."),
-        // NetHack reference:
-        N_("Do you want your possessions identified?"),
-        // Secret of Mana reference:
-        N_("Sadly, no trace of you was ever found..."),
-        // Final Fantasy VI reference:
-        N_("Annihilated."),
-        // Earthbound reference:
-        N_("Looks like you got your head handed to you."),
-        // Leisure Suit Larry 1 reference:
-        N_("You screwed up again, dump your body down the tubes "
-           "and get you another one."),
-        // Monty Python references (Dead Parrot sketch mostly):
-        N_("You're not dead yet. You're just resting."),
-        N_("You are no more."),
-        N_("You have ceased to be."),
-        N_("You've expired and gone to meet your maker."),
-        N_("You're a stiff."),
-        N_("Bereft of life, you rest in peace."),
-        N_("If you weren't so animated, you'd be pushing up the daisies."),
-        N_("Your metabolic processes are now history."),
-        N_("You're off the twig."),
-        N_("You've kicked the bucket."),
-        N_("You've shuffled off your mortal coil, run down the "
-           "curtain and joined the bleedin' choir invisibile."),
-        N_("You are an ex-player."),
-        N_("You're pining for the fjords.")
-    };
-
-    const int random = static_cast<int>(rand() % (sizeof(deadMsg)
-        / sizeof(deadMsg[0])));
-    return gettext(deadMsg[random]);
-}
-
 extern Net::PlayerHandler *playerHandler;
 
 namespace TmwAthena
@@ -174,469 +61,40 @@ void PlayerHandler::handleMessage(Net::MessageIn &msg)
     switch (msg.getId())
     {
         case SMSG_WALK_RESPONSE:
-            /*
-             * This client assumes that all walk messages succeed,
-             * and that the server will send a correction notice
-             * otherwise.
-             */
-            Uint16 srcX, srcY, dstX, dstY;
-            msg.readInt32();    //tick
-            msg.readCoordinatePair(srcX, srcY, dstX, dstY);
-            if (player_node)
-                player_node->setRealPos(dstX, dstY);
-//            if (debugChatTab)
-//                debugChatTab->chatLog("move resp: " + toString((int)srcX) + "," + toString((int)srcY) + " "
-//             + toString((int)dstX) + "," + toString((int)dstY));
-
+            processWalkResponse(msg);
             break;
 
         case SMSG_PLAYER_WARP:
-            {
-                std::string mapPath = msg.readString(16);
-                int x = msg.readInt16();
-                int y = msg.readInt16();
-
-                logger->log("Warping to %s (%d, %d)", mapPath.c_str(), x, y);
-
-                if (!player_node)
-                    logger->log1("SMSG_PLAYER_WARP player_node null");
-
-                /*
-                 * We must clear the local player's target *before* the call
-                 * to changeMap, as it deletes all beings.
-                 */
-                if (player_node)
-                    player_node->stopAttack();
-
-                Game *game = Game::instance();
-
-                const std::string &currentMapName = game->getCurrentMapName();
-                bool sameMap = (currentMapName == mapPath);
-
-                // Switch the actual map, deleting the previous one if necessary
-                mapPath = mapPath.substr(0, mapPath.rfind("."));
-                game->changeMap(mapPath);
-
-                float scrollOffsetX = 0.0f;
-                float scrollOffsetY = 0.0f;
-
-                if (player_node)
-                {
-                    Map *map = game->getCurrentMap();
-                    if (map)
-                    {
-                        if (x >= map->getWidth())
-                            x = map->getWidth() - 1;
-                        if (y >= map->getHeight())
-                            y = map->getHeight() - 1;
-                        if (x < 0)
-                            x = 0;
-                        if (y < 0)
-                            y = 0;
-                        /* Scroll if neccessary */
-                        if (!sameMap
-                            || (abs(x - player_node->getTileX())
-                            > MAP_TELEPORT_SCROLL_DISTANCE)
-                            || (abs(y - player_node->getTileY())
-                            > MAP_TELEPORT_SCROLL_DISTANCE))
-                        {
-                            scrollOffsetX = static_cast<float>((x
-                                - player_node->getTileX())
-                                * map->getTileWidth());
-                            scrollOffsetY = static_cast<float>((y
-                                - player_node->getTileY())
-                                * map->getTileHeight());
-                        }
-                    }
-
-                    player_node->setAction(Being::STAND);
-                    player_node->setTileCoords(x, y);
-                    player_node->navigateClean();
-//                    player_node->updateNavigateList();
-                }
-
-                logger->log("Adjust scrolling by %d:%d",
-                    static_cast<int>(scrollOffsetX),
-                    static_cast<int>(scrollOffsetY));
-
-                if (viewport)
-                    viewport->scrollBy(scrollOffsetX, scrollOffsetY);
-            }
+            processPlayerWarp(msg);
             break;
 
         case SMSG_PLAYER_STAT_UPDATE_1:
-            {
-                int type = msg.readInt16();
-                int value = msg.readInt32();
-                if (!player_node)
-                    return;
-
-                switch (type)
-                {
-                    case 0x0000:
-                        player_node->setWalkSpeed(Vector(static_cast<float>(
-                            value), static_cast<float>(value), 0));
-                        PlayerInfo::setStatBase(WALK_SPEED, value);
-                        PlayerInfo::setStatMod(WALK_SPEED, 0);
-                    break;
-                    case 0x0004: break; // manner
-                    case 0x0005:
-                        PlayerInfo::setAttribute(HP, value);
-                        if (player_node->isInParty() && Party::getParty(1))
-                        {
-                            PartyMember *m = Party::getParty(1)
-                                ->getMember(player_node->getId());
-                            if (m)
-                            {
-                                m->setHp(value);
-                                m->setMaxHp(PlayerInfo::getAttribute(MAX_HP));
-                            }
-                        }
-                        break;
-                    case 0x0006:
-                        PlayerInfo::setAttribute(MAX_HP, value);
-
-                        if (player_node->isInParty() && Party::getParty(1))
-                        {
-                            PartyMember *m = Party::getParty(1)->getMember(
-                                player_node->getId());
-                            if (m)
-                            {
-                                m->setHp(PlayerInfo::getAttribute(HP));
-                                m->setMaxHp(value);
-                            }
-                        }
-                        break;
-                    case 0x0007:
-                        PlayerInfo::setAttribute(MP, value);
-                        break;
-                    case 0x0008:
-                        PlayerInfo::setAttribute(MAX_MP, value);
-                        break;
-                    case 0x0009:
-                        PlayerInfo::setAttribute(CHAR_POINTS, value);
-                        break;
-                    case 0x000b:
-                        PlayerInfo::setAttribute(LEVEL, value);
-                        if (player_node)
-                        {
-                            player_node->setLevel(value);
-                            player_node->updateName();
-                        }
-                        break;
-                    case 0x000c:
-                        PlayerInfo::setAttribute(SKILL_POINTS, value);
-                        if (skillDialog)
-                            skillDialog->update();
-                        break;
-                    case 0x0018:
-                        if (!weightNotice)
-                        {
-                            const int max
-                                = PlayerInfo::getAttribute(MAX_WEIGHT) / 2;
-                            const int total
-                                = PlayerInfo::getAttribute(TOTAL_WEIGHT);
-                            if (value >= max && total < max)
-                            {
-                                weightNoticeTime = cur_time + 10;
-                                weightNotice = new OkDialog(_("Message"),
-                                    _("You are carrying more than "
-                                    "half your weight. You are "
-                                    "unable to regain health."), false);
-                                weightNotice->addActionListener(
-                                    &weightListener);
-                            }
-                            else if (value < max && total >= max)
-                            {
-                                weightNoticeTime = cur_time + 10;
-                                weightNotice = new OkDialog(_("Message"),
-                                    _("You are carrying less than "
-                                    "half your weight. You "
-                                    "can regain health."), false);
-                                weightNotice->addActionListener(
-                                    &weightListener);
-                            }
-                        }
-                        PlayerInfo::setAttribute(TOTAL_WEIGHT, value);
-                        break;
-                    case 0x0019:
-                        PlayerInfo::setAttribute(MAX_WEIGHT, value);
-                        break;
-
-                    case 0x0029:
-                        PlayerInfo::setStatBase(ATK, value);
-                        break;
-                    case 0x002a:
-                        PlayerInfo::setStatMod(ATK, value);
-                        break;
-
-                    case 0x002b:
-                        PlayerInfo::setStatBase(MATK, value);
-                        break;
-                    case 0x002c:
-                        PlayerInfo::setStatMod(MATK, value);
-                        break;
-
-                    case 0x002d:
-                        PlayerInfo::setStatBase(DEF, value);
-                        break;
-                    case 0x002e:
-                        PlayerInfo::setStatMod(DEF, value);
-                        break;
-
-                    case 0x002f:
-                        PlayerInfo::setStatBase(MDEF, value);
-                        break;
-                    case 0x0030:
-                        PlayerInfo::setStatMod(MDEF, value);
-                        break;
-
-                    case 0x0031:
-                        PlayerInfo::setStatBase(HIT, value);
-                        break;
-
-                    case 0x0032:
-                        PlayerInfo::setStatBase(FLEE, value);
-                        break;
-                    case 0x0033:
-                        PlayerInfo::setStatMod(FLEE, value);
-                        break;
-
-                    case 0x0034:
-                        PlayerInfo::setStatBase(CRIT, value);
-                        break;
-
-                    case 0x0035:
-                        player_node->setAttackSpeed(value);
-                        PlayerInfo::setStatBase(ATTACK_SPEED, value);
-                        PlayerInfo::setStatMod(ATTACK_SPEED, 0);
-                        break;
-
-                    case 0x0037:
-                        PlayerInfo::setStatBase(JOB, value);
-                        break;
-
-                    case 500:
-                        player_node->setGMLevel(value);
-                        break;
-
-                    default:
-                        logger->log("QQQQ PLAYER_STAT_UPDATE_1 "
-                                    + toString(type) + "," + toString(value));
-                        break;
-                }
-
-                if (PlayerInfo::getAttribute(HP) == 0 && !deathNotice)
-                {
-                    deathNotice = new OkDialog(_("Message"),
-                                               randomDeathMessage(),
-                                               false);
-                    deathNotice->addActionListener(&deathListener);
-                    player_node->setAction(Being::DEAD);
-                }
-            }
+            processPlayerStatUpdate1(msg);
             break;
 
         case SMSG_PLAYER_STAT_UPDATE_2:
-        {
-            int type = msg.readInt16();
-            switch (type)
-            {
-                case 0x0001:
-                    PlayerInfo::setAttribute(EXP, msg.readInt32());
-                    break;
-                case 0x0002:
-                    PlayerInfo::setStatExperience(JOB, msg.readInt32(),
-                    PlayerInfo::getStatExperience(JOB).second);
-                    break;
-                case 0x0014:
-                {
-                        int oldMoney = PlayerInfo::getAttribute(MONEY);
-                        int newMoney = msg.readInt32();
-                        if (newMoney > oldMoney)
-                        {
-                            SERVER_NOTICE(strprintf(_("You picked up %s."),
-                                          Units::formatCurrency(newMoney -
-                                          oldMoney).c_str()))
-                        }
-                        else if (newMoney < oldMoney)
-                        {
-                            SERVER_NOTICE(strprintf(_("You spent %s."),
-                                          Units::formatCurrency(oldMoney -
-                                          newMoney).c_str()))
-                        }
-
-                        PlayerInfo::setAttribute(MONEY, newMoney);
-                    break;
-                }
-                case 0x0016:
-                    PlayerInfo::setAttribute(EXP_NEEDED, msg.readInt32());
-                    break;
-                case 0x0017:
-                    PlayerInfo::setStatExperience(JOB,
-                        PlayerInfo::getStatExperience(JOB).first,
-                        msg.readInt32());
-                    break;
-                default:
-                    logger->log("QQQQ PLAYER_STAT_UPDATE_2 " + toString(type));
-                    break;
-            }
+            processPlayerStatUpdate2(msg);
             break;
-        }
+
         case SMSG_PLAYER_STAT_UPDATE_3: // Update a base attribute
-            {
-                int type = msg.readInt32();
-                int base = msg.readInt32();
-                int bonus = msg.readInt32();
-
-                PlayerInfo::setStatBase(type, base, false);
-                PlayerInfo::setStatMod(type, bonus);
-            }
+            processPlayerStatUpdate3(msg);
             break;
 
         case SMSG_PLAYER_STAT_UPDATE_4: // Attribute increase ack
-            {
-                int type = msg.readInt16();
-                int ok = msg.readInt8();
-                int value = msg.readInt8();
-
-                if (ok != 1)
-                {
-                    int oldValue = PlayerInfo::getStatBase(type);
-                    int points = PlayerInfo::getAttribute(CHAR_POINTS);
-                    points += oldValue - value;
-                    PlayerInfo::setAttribute(CHAR_POINTS, points);
-                    SERVER_NOTICE(_("Cannot raise skill!"))
-                }
-
-                PlayerInfo::setStatBase(type, value);
-            }
+            processPlayerStatUpdate4(msg);
             break;
 
         // Updates stats and status points
         case SMSG_PLAYER_STAT_UPDATE_5:
-            PlayerInfo::setAttribute(CHAR_POINTS, msg.readInt16());
-            {
-                int val = msg.readInt8();
-                PlayerInfo::setStatBase(STR, val);
-                if (statusWindow)
-                    statusWindow->setPointsNeeded(STR, msg.readInt8());
-                else
-                    msg.readInt8();
-
-                val = msg.readInt8();
-                PlayerInfo::setStatBase(AGI, val);
-                if (statusWindow)
-                    statusWindow->setPointsNeeded(AGI, msg.readInt8());
-                else
-                    msg.readInt8();
-
-                val = msg.readInt8();
-                PlayerInfo::setStatBase(VIT, val);
-                if (statusWindow)
-                    statusWindow->setPointsNeeded(VIT, msg.readInt8());
-                else
-                    msg.readInt8();
-
-                val = msg.readInt8();
-                PlayerInfo::setStatBase(INT, val);
-                if (statusWindow)
-                    statusWindow->setPointsNeeded(INT, msg.readInt8());
-                else
-                    msg.readInt8();
-
-                val = msg.readInt8();
-                PlayerInfo::setStatBase(DEX, val);
-                if (statusWindow)
-                    statusWindow->setPointsNeeded(DEX, msg.readInt8());
-                else
-                    msg.readInt8();
-
-                val = msg.readInt8();
-                PlayerInfo::setStatBase(LUK, val);
-                if (statusWindow)
-                    statusWindow->setPointsNeeded(LUK, msg.readInt8());
-                else
-                    msg.readInt8();
-
-                PlayerInfo::setStatBase(ATK, msg.readInt16(), false);
-                PlayerInfo::setStatMod(ATK, msg.readInt16());
-
-                val = msg.readInt16();
-                PlayerInfo::setStatBase(MATK, val, false);
-
-                val = msg.readInt16();
-                PlayerInfo::setStatMod(MATK, val);
-
-                PlayerInfo::setStatBase(DEF, msg.readInt16(), false);
-                PlayerInfo::setStatMod(DEF, msg.readInt16());
-
-                PlayerInfo::setStatBase(MDEF, msg.readInt16(), false);
-                PlayerInfo::setStatMod(MDEF, msg.readInt16());
-
-                PlayerInfo::setStatBase(HIT, msg.readInt16());
-
-                PlayerInfo::setStatBase(FLEE, msg.readInt16(), false);
-                PlayerInfo::setStatMod(FLEE, msg.readInt16());
-
-                PlayerInfo::setStatBase(CRIT, msg.readInt16());
-            }
-
-            msg.readInt16();  // manner
+            processPlayerStatUpdate5(msg);
             break;
 
         case SMSG_PLAYER_STAT_UPDATE_6:
-        {
-            int type = msg.readInt16();
-            if (statusWindow)
-            {
-                switch (type)
-                {
-                    case 0x0020:
-                        statusWindow->setPointsNeeded(STR, msg.readInt8());
-                        break;
-                    case 0x0021:
-                        statusWindow->setPointsNeeded(AGI, msg.readInt8());
-                        break;
-                    case 0x0022:
-                        statusWindow->setPointsNeeded(VIT, msg.readInt8());
-                        break;
-                    case 0x0023:
-                        statusWindow->setPointsNeeded(INT, msg.readInt8());
-                        break;
-                    case 0x0024:
-                        statusWindow->setPointsNeeded(DEX, msg.readInt8());
-                        break;
-                    case 0x0025:
-                        statusWindow->setPointsNeeded(LUK, msg.readInt8());
-                        break;
-                    default:
-                        logger->log("QQQQ PLAYER_STAT_UPDATE_6 "
-                                    + toString(type));
-                        break;
-                }
-            }
+            processPlayerStatUpdate6(msg);
             break;
-        }
+
         case SMSG_PLAYER_ARROW_MESSAGE:
-            {
-                int type = msg.readInt16();
-
-                switch (type)
-                {
-                    case 0:
-                        {
-                            SERVER_NOTICE(_("Equip arrows first."))
-                        }
-                        break;
-                    case 3:
-                        // arrows equiped
-                        break;
-                    default:
-                        logger->log("QQQQ 0x013b: Unhandled message %i", type);
-                        break;
-                }
-            }
+            processPlayerArrowMessage(msg);
             break;
 
         default:
@@ -675,11 +133,6 @@ void PlayerHandler::increaseAttribute(int attr)
     }
 }
 
-void PlayerHandler::decreaseAttribute(int attr A_UNUSED)
-{
-    // Supported by eA?
-}
-
 void PlayerHandler::increaseSkill(unsigned short skillId)
 {
     if (PlayerInfo::getAttribute(SKILL_POINTS) <= 0)
@@ -748,37 +201,4 @@ void PlayerHandler::respawn()
     outMsg.writeInt8(0);
 }
 
-void PlayerHandler::ignorePlayer(const std::string &player A_UNUSED,
-                                 bool ignore A_UNUSED)
-{
-    // TODO
-}
-
-void PlayerHandler::ignoreAll(bool ignore A_UNUSED)
-{
-    // TODO
-}
-
-bool PlayerHandler::canUseMagic() const
-{
-    return PlayerInfo::getStatEffective(MATK) > 0;
-}
-
-bool PlayerHandler::canCorrectAttributes() const
-{
-    return false;
-}
-
-int PlayerHandler::getJobLocation() const
-{
-    return JOB;
-}
-
-Vector PlayerHandler::getDefaultWalkSpeed() const
-{
-    // Return an normalized speed for any side
-    // as the offset is calculated elsewhere.
-    return Vector(150, 150, 0);
-}
-
 } // namespace TmwAthena
diff --git a/src/net/tmwa/playerhandler.h b/src/net/tmwa/playerhandler.h
index fc4230856..4fd108e94 100644
--- a/src/net/tmwa/playerhandler.h
+++ b/src/net/tmwa/playerhandler.h
@@ -26,6 +26,8 @@
 #include "net/net.h"
 #include "net/playerhandler.h"
 
+#include "net/ea/playerhandler.h"
+
 #include "net/tmwa/messagehandler.h"
 
 #ifdef __GNUC__
@@ -37,7 +39,7 @@
 namespace TmwAthena
 {
 
-class PlayerHandler : public MessageHandler, public Net::PlayerHandler
+class PlayerHandler : public MessageHandler, public Ea::PlayerHandler
 {
     public:
         PlayerHandler();
@@ -49,7 +51,6 @@ class PlayerHandler : public MessageHandler, public Net::PlayerHandler
         void emote(Uint8 emoteId);
 
         void increaseAttribute(int attr);
-        void decreaseAttribute(int attr);
         void increaseSkill(unsigned short skillId);
 
         void pickUp(FloorItem *floorItem);
@@ -58,16 +59,6 @@ class PlayerHandler : public MessageHandler, public Net::PlayerHandler
         void changeAction(Being::Action action);
 
         void respawn();
-
-        void ignorePlayer(const std::string &player, bool ignore);
-        void ignoreAll(bool ignore);
-
-        bool canUseMagic() const;
-        bool canCorrectAttributes() const;
-
-        int getJobLocation() const;
-
-        Vector getDefaultWalkSpeed() const;
 };
 
 } // namespace TmwAthena