1 files changed, 22 insertions, 29 deletions

diff --git a/src/net/equipmenthandler.cpp b/src/net/equipmenthandler.cpp
index e5bbf6fe..9a3c396a 100644
--- a/src/net/equipmenthandler.cpp
+++ b/src/net/equipmenthandler.cpp
@@ -20,11 +20,9 @@
  */
 
 #include "equipmenthandler.h"
-
 #include "messagein.h"
 #include "protocol.h"
 
-#include "../beingmanager.h"
 #include "../equipment.h"
 #include "../inventory.h"
 #include "../item.h"
@@ -33,6 +31,8 @@
 
 #include "../gui/chat.h"
 
+#include "../utils/gettext.h"
+
 EquipmentHandler::EquipmentHandler()
 {
     static const Uint16 _messages[] = {
@@ -98,7 +98,7 @@ void EquipmentHandler::handleMessage(MessageIn *msg)
             logger->log("Equipping: %i %i %i", index, equipPoint, type);
 
             if (!type) {
-                chatWindow->chatLog("Unable to equip.", BY_SERVER);
+                chatWindow->chatLog(_("Unable to equip."), BY_SERVER);
                 break;
             }
 
@@ -107,7 +107,10 @@ void EquipmentHandler::handleMessage(MessageIn *msg)
                 break;
             }
 
-            // Unequip any existing equipped item in this position
+            /*
+             * An item may occupy more than 1 slot.  If so, it's
+             * only shown as equipped on the *first* slot.
+             */
             mask = 1;
             position = 0;
             while (!(equipPoint & mask)) {
@@ -115,7 +118,10 @@ void EquipmentHandler::handleMessage(MessageIn *msg)
                 position++;
             }
             logger->log("Position %i", position);
-            item =  player_node->getInventory()->getItem(player_node->mEquipment->getEquipment(position));
+
+            item = player_node->getInventory()->getItem(player_node->mEquipment->getEquipment(position));
+
+            // Unequip any existing equipped item in this position
             if (item) {
                 item->setEquipped(false);
             }
@@ -130,7 +136,7 @@ void EquipmentHandler::handleMessage(MessageIn *msg)
             type = msg->readInt8();
 
             if (!type) {
-                chatWindow->chatLog("Unable to unequip.", BY_SERVER);
+                chatWindow->chatLog(_("Unable to unequip."), BY_SERVER);
                 break;
             }
 
@@ -152,24 +158,11 @@ void EquipmentHandler::handleMessage(MessageIn *msg)
 
             item->setEquipped(false);
 
-            switch (item->getId()) {
-                case 529:
-                case 1199:
-                    player_node->mEquipment->setArrows(0);
-                    break;
-                case 521:
-                case 522:
-                case 530:
-                case 536:
-                case 1200:
-                case 1201:
-                    player_node->setSprite(Being::WEAPON_SPRITE, 0);
-                    // TODO: Why this break? Shouldn't a weapon be
-                    //       unequipped in inventory too?
-                    break;
-                default:
-                    player_node->mEquipment->removeEquipment(position);
-                    break;
+            if (equipPoint & 0x8000) {    // Arrows
+                player_node->mEquipment->setArrows(0);
+            }
+            else {
+                player_node->mEquipment->removeEquipment(position);
             }
             logger->log("Unequipping: %i %i(%i) %i",
                     index, equipPoint, type, position);
@@ -186,12 +179,12 @@ void EquipmentHandler::handleMessage(MessageIn *msg)
                 break;
 
             item = inventory->getItem(index);
-            if (!item)
-                break;
 
-            item->setEquipped(true);
-            player_node->mEquipment->setArrows(index);
-            logger->log("Arrows equipped: %i", index);
+            if (item) {
+                item->setEquipped(true);
+                player_node->mEquipment->setArrows(index);
+                logger->log("Arrows equipped: %i", index);
+            }
             break;
     }
 }