Move player attributes into separate file.

3 files changed, 33 insertions, 30 deletions

diff --git a/src/net/ea/buysellhandler.cpp b/src/net/ea/buysellhandler.cpp
index d55a3cea6..28ec4497e 100644
--- a/src/net/ea/buysellhandler.cpp
+++ b/src/net/ea/buysellhandler.cpp
@@ -27,6 +27,7 @@
 #include "notifications.h"
 #include "notifymanager.h"
 
+#include "being/attributes.h"
 #include "being/playerinfo.h"
 
 #include "gui/windows/chatwindow.h"
@@ -144,7 +145,7 @@ void BuySellHandler::processNpcSell(Net::MessageIn &msg,
     if (n_items > 0)
     {
         SellDialog *const dialog = new SellDialog(mNpcId);
-        dialog->setMoney(PlayerInfo::getAttribute(PlayerInfo::MONEY));
+        dialog->setMoney(PlayerInfo::getAttribute(Attributes::MONEY));
 
         for (int k = 0; k < n_items; k++)
         {
@@ -176,7 +177,7 @@ void BuySellHandler::processNpcBuyResponse(Net::MessageIn &msg) const
         // Reset player money since buy dialog already assumed purchase
         // would go fine
         if (mBuyDialog)
-            mBuyDialog->setMoney(PlayerInfo::getAttribute(PlayerInfo::MONEY));
+            mBuyDialog->setMoney(PlayerInfo::getAttribute(Attributes::MONEY));
         NotifyManager::notify(NotifyManager::BUY_FAILED);
     }
 }
diff --git a/src/net/ea/inventoryhandler.cpp b/src/net/ea/inventoryhandler.cpp
index 2f5ec35ce..46f87f089 100644
--- a/src/net/ea/inventoryhandler.cpp
+++ b/src/net/ea/inventoryhandler.cpp
@@ -25,6 +25,7 @@
 #include "notifications.h"
 #include "notifymanager.h"
 
+#include "being/attributes.h"
 #include "being/localplayer.h"
 
 #include "net/messagein.h"
@@ -575,8 +576,8 @@ void InventoryHandler::processPlayerAttackRange(Net::MessageIn &msg)
     const int range = msg.readInt16();
     if (player_node)
         player_node->setAttackRange(range);
-    PlayerInfo::setStatBase(PlayerInfo::ATTACK_RANGE, range);
-    PlayerInfo::setStatMod(PlayerInfo::ATTACK_RANGE, 0);
+    PlayerInfo::setStatBase(Attributes::ATTACK_RANGE, range);
+    PlayerInfo::setStatMod(Attributes::ATTACK_RANGE, 0);
 }
 
 void InventoryHandler::processPlayerArrowEquip(Net::MessageIn &msg)
diff --git a/src/net/ea/playerhandler.cpp b/src/net/ea/playerhandler.cpp
index 475df2db0..1b324e376 100644
--- a/src/net/ea/playerhandler.cpp
+++ b/src/net/ea/playerhandler.cpp
@@ -29,6 +29,7 @@
 #include "notifymanager.h"
 #include "units.h"
 
+#include "being/attributes.h"
 #include "being/localplayer.h"
 
 #include "gui/viewport.h"
@@ -233,13 +234,13 @@ void PlayerHandler::processPlayerStatUpdate1(Net::MessageIn &msg)
         case 0x0000:
             player_node->setWalkSpeed(Vector(static_cast<float>(
                 value), static_cast<float>(value), 0));
-            PlayerInfo::setStatBase(PlayerInfo::WALK_SPEED, value);
-            PlayerInfo::setStatMod(PlayerInfo::WALK_SPEED, 0);
+            PlayerInfo::setStatBase(Attributes::WALK_SPEED, value);
+            PlayerInfo::setStatMod(Attributes::WALK_SPEED, 0);
         break;
         case 0x0004:
             break;  // manner
         case 0x0005:
-            PlayerInfo::setAttribute(PlayerInfo::HP, value);
+            PlayerInfo::setAttribute(Attributes::HP, value);
             if (player_node->isInParty() && Party::getParty(1))
             {
                 PartyMember *const m = Party::getParty(1)
@@ -247,12 +248,12 @@ void PlayerHandler::processPlayerStatUpdate1(Net::MessageIn &msg)
                 if (m)
                 {
                     m->setHp(value);
-                    m->setMaxHp(PlayerInfo::getAttribute(PlayerInfo::MAX_HP));
+                    m->setMaxHp(PlayerInfo::getAttribute(Attributes::MAX_HP));
                 }
             }
             break;
         case 0x0006:
-            PlayerInfo::setAttribute(PlayerInfo::MAX_HP, value);
+            PlayerInfo::setAttribute(Attributes::MAX_HP, value);
 
             if (player_node->isInParty() && Party::getParty(1))
             {
@@ -260,22 +261,22 @@ void PlayerHandler::processPlayerStatUpdate1(Net::MessageIn &msg)
                     player_node->getId());
                 if (m)
                 {
-                    m->setHp(PlayerInfo::getAttribute(PlayerInfo::HP));
+                    m->setHp(PlayerInfo::getAttribute(Attributes::HP));
                     m->setMaxHp(value);
                 }
             }
             break;
         case 0x0007:
-            PlayerInfo::setAttribute(PlayerInfo::MP, value);
+            PlayerInfo::setAttribute(Attributes::MP, value);
             break;
         case 0x0008:
-            PlayerInfo::setAttribute(PlayerInfo::MAX_MP, value);
+            PlayerInfo::setAttribute(Attributes::MAX_MP, value);
             break;
         case 0x0009:
-            PlayerInfo::setAttribute(PlayerInfo::CHAR_POINTS, value);
+            PlayerInfo::setAttribute(Attributes::CHAR_POINTS, value);
             break;
         case 0x000b:
-            PlayerInfo::setAttribute(PlayerInfo::LEVEL, value);
+            PlayerInfo::setAttribute(Attributes::LEVEL, value);
             if (player_node)
             {
                 player_node->setLevel(value);
@@ -283,7 +284,7 @@ void PlayerHandler::processPlayerStatUpdate1(Net::MessageIn &msg)
             }
             break;
         case 0x000c:
-            PlayerInfo::setAttribute(PlayerInfo::SKILL_POINTS, value);
+            PlayerInfo::setAttribute(Attributes::SKILL_POINTS, value);
             if (skillDialog)
                 skillDialog->update();
             break;
@@ -291,9 +292,9 @@ void PlayerHandler::processPlayerStatUpdate1(Net::MessageIn &msg)
             if (!weightNotice && config.getBoolValue("weightMsg"))
             {
                 const int max = PlayerInfo::getAttribute(
-                    PlayerInfo::MAX_WEIGHT) / 2;
+                    Attributes::MAX_WEIGHT) / 2;
                 const int total = PlayerInfo::getAttribute(
-                    PlayerInfo::TOTAL_WEIGHT);
+                    Attributes::TOTAL_WEIGHT);
                 if (value >= max && total < max)
                 {
                     weightNoticeTime = cur_time + 5;
@@ -319,10 +320,10 @@ void PlayerHandler::processPlayerStatUpdate1(Net::MessageIn &msg)
                         &weightListener);
                 }
             }
-            PlayerInfo::setAttribute(PlayerInfo::TOTAL_WEIGHT, value);
+            PlayerInfo::setAttribute(Attributes::TOTAL_WEIGHT, value);
             break;
         case 0x0019:
-            PlayerInfo::setAttribute(PlayerInfo::MAX_WEIGHT, value);
+            PlayerInfo::setAttribute(Attributes::MAX_WEIGHT, value);
             break;
 
         case 0x0029:
@@ -372,8 +373,8 @@ void PlayerHandler::processPlayerStatUpdate1(Net::MessageIn &msg)
 
         case 0x0035:
             player_node->setAttackSpeed(value);
-            PlayerInfo::setStatBase(PlayerInfo::ATTACK_DELAY, value);
-            PlayerInfo::setStatMod(PlayerInfo::ATTACK_DELAY, 0);
+            PlayerInfo::setStatBase(Attributes::ATTACK_DELAY, value);
+            PlayerInfo::setStatMod(Attributes::ATTACK_DELAY, 0);
             PlayerInfo::updateAttrs();
             break;
 
@@ -391,7 +392,7 @@ void PlayerHandler::processPlayerStatUpdate1(Net::MessageIn &msg)
             break;
     }
 
-    if (PlayerInfo::getAttribute(PlayerInfo::HP) == 0 && !deathNotice)
+    if (PlayerInfo::getAttribute(Attributes::HP) == 0 && !deathNotice)
     {
         // TRANSLATORS: message header
         deathNotice = new OkDialog(_("Message"),
@@ -413,7 +414,7 @@ void PlayerHandler::processPlayerStatUpdate2(Net::MessageIn &msg)
     switch (type)
     {
         case 0x0001:
-            PlayerInfo::setAttribute(PlayerInfo::EXP, msg.readInt32());
+            PlayerInfo::setAttribute(Attributes::EXP, msg.readInt32());
             break;
         case 0x0002:
             PlayerInfo::setStatExperience(EA_JOB, msg.readInt32(),
@@ -421,7 +422,7 @@ void PlayerHandler::processPlayerStatUpdate2(Net::MessageIn &msg)
             break;
         case 0x0014:
         {
-            const int oldMoney = PlayerInfo::getAttribute(PlayerInfo::MONEY);
+            const int oldMoney = PlayerInfo::getAttribute(Attributes::MONEY);
             const int newMoney = msg.readInt32();
             if (newMoney > oldMoney)
             {
@@ -434,11 +435,11 @@ void PlayerHandler::processPlayerStatUpdate2(Net::MessageIn &msg)
                     Units::formatCurrency(oldMoney - newMoney).c_str());
             }
 
-            PlayerInfo::setAttribute(PlayerInfo::MONEY, newMoney);
+            PlayerInfo::setAttribute(Attributes::MONEY, newMoney);
             break;
         }
         case 0x0016:
-            PlayerInfo::setAttribute(PlayerInfo::EXP_NEEDED, msg.readInt32());
+            PlayerInfo::setAttribute(Attributes::EXP_NEEDED, msg.readInt32());
             break;
         case 0x0017:
             PlayerInfo::setStatExperience(EA_JOB,
@@ -460,7 +461,7 @@ void PlayerHandler::processPlayerStatUpdate3(Net::MessageIn &msg)
 
     PlayerInfo::setStatBase(type, base, false);
     PlayerInfo::setStatMod(type, bonus);
-    if (type == EA_ATK || type == PlayerInfo::ATTACK_DELAY)
+    if (type == EA_ATK || type == Attributes::ATTACK_DELAY)
         PlayerInfo::updateAttrs();
     BLOCK_END("PlayerHandler::processPlayerStatUpdate3")
 }
@@ -475,9 +476,9 @@ void PlayerHandler::processPlayerStatUpdate4(Net::MessageIn &msg)
     if (ok != 1)
     {
         const int oldValue = PlayerInfo::getStatBase(type);
-        const int points = PlayerInfo::getAttribute(PlayerInfo::CHAR_POINTS)
+        const int points = PlayerInfo::getAttribute(Attributes::CHAR_POINTS)
             + oldValue - value;
-        PlayerInfo::setAttribute(PlayerInfo::CHAR_POINTS, points);
+        PlayerInfo::setAttribute(Attributes::CHAR_POINTS, points);
         NotifyManager::notify(NotifyManager::SKILL_RAISE_ERROR);
     }
 
@@ -488,7 +489,7 @@ void PlayerHandler::processPlayerStatUpdate4(Net::MessageIn &msg)
 void PlayerHandler::processPlayerStatUpdate5(Net::MessageIn &msg)
 {
     BLOCK_START("PlayerHandler::processPlayerStatUpdate5")
-    PlayerInfo::setAttribute(PlayerInfo::CHAR_POINTS, msg.readInt16());
+    PlayerInfo::setAttribute(Attributes::CHAR_POINTS, msg.readInt16());
 
     int val = msg.readInt8();
     PlayerInfo::setStatBase(EA_STR, val);