Split adjustBoundItem()

apps/openmw/mwmechanics/actors.cpp

@@ -235,70 +235,71 @@ namespace MWMechanics
         }
     };
 
-    void Actors::adjustBoundItem (const std::string& itemId, bool bound, const MWWorld::Ptr& actor)
+    void Actors::addBoundItem (const std::string& itemId, const MWWorld::Ptr& actor)
     {
         MWWorld::InventoryStore& store = actor.getClass().getInventoryStore(actor);
         int slot = getBoundItemSlot(itemId);
 
-        if (bound)
-        {
-            if (actor.getClass().getContainerStore(actor).count(itemId) != 0)
-                return;
+        if (actor.getClass().getContainerStore(actor).count(itemId) != 0)
+            return;
 
-            MWWorld::ContainerStoreIterator prevItem = store.getSlot(slot);
+        MWWorld::ContainerStoreIterator prevItem = store.getSlot(slot);
 
-            MWWorld::Ptr boundPtr = *store.MWWorld::ContainerStore::add(itemId, 1, actor);
-            MWWorld::ActionEquip action(boundPtr);
-            action.execute(actor);
+        MWWorld::Ptr boundPtr = *store.MWWorld::ContainerStore::add(itemId, 1, actor);
+        MWWorld::ActionEquip action(boundPtr);
+        action.execute(actor);
 
-            if (actor != MWMechanics::getPlayer())
-                return;
+        if (actor != MWMechanics::getPlayer())
+            return;
 
-            MWWorld::Ptr newItem = *store.getSlot(slot);
+        MWWorld::Ptr newItem = *store.getSlot(slot);
 
-            if (newItem.isEmpty() || boundPtr != newItem)
-                return;
+        if (newItem.isEmpty() || boundPtr != newItem)
+            return;
 
-            // change draw state only if the item is in player's right hand
-            if (slot == MWWorld::InventoryStore::Slot_CarriedRight)
-                MWBase::Environment::get().getWorld()->getPlayer().setDrawState(MWMechanics::DrawState_Weapon);
+        // change draw state only if the item is in player's right hand
+        if (slot == MWWorld::InventoryStore::Slot_CarriedRight)
+            MWBase::Environment::get().getWorld()->getPlayer().setDrawState(MWMechanics::DrawState_Weapon);
 
-            if (prevItem != store.end())
-                mPreviousItems[itemId] = *prevItem;
-        }
-        else
-        {
-            MWWorld::ContainerStoreIterator currentItem = store.getSlot(slot);
+        if (prevItem != store.end())
+            mPreviousItems[itemId] = *prevItem;
+    }
 
-            bool wasEquipped = currentItem != store.end() && Misc::StringUtils::ciEqual((*currentItem).getCellRef().getRefId(), itemId);
+    void Actors::removeBoundItem (const std::string& itemId, const MWWorld::Ptr& actor)
+    {
+        MWWorld::InventoryStore& store = actor.getClass().getInventoryStore(actor);
+        int slot = getBoundItemSlot(itemId);
 
-            store.remove(itemId, 1, actor, true);
+        MWWorld::ContainerStoreIterator currentItem = store.getSlot(slot);
 
-            if (actor != MWMechanics::getPlayer())
-                return;
+        bool wasEquipped = currentItem != store.end() && Misc::StringUtils::ciEqual((*currentItem).getCellRef().getRefId(), itemId);
 
-            MWWorld::Ptr prevItem = mPreviousItems[itemId];
+        store.remove(itemId, 1, actor, true);
 
-            mPreviousItems.erase(itemId);
+        if (actor != MWMechanics::getPlayer())
+            return;
 
-            if (prevItem.isEmpty())
-                return;
+        MWWorld::Ptr prevItem = mPreviousItems[itemId];
 
-            // check if the item is still in the player's inventory
-            MWWorld::ContainerStoreIterator it = store.begin();
-            for (; it != store.end(); ++it)
-            {
-                if (*it == prevItem)
-                    break;
-            }
+        mPreviousItems.erase(itemId);
 
-            // we should equip previous item only if expired bound item was equipped.
-            if (it == store.end() || !wasEquipped)
-                return;
+        if (prevItem.isEmpty())
+            return;
 
-            MWWorld::ActionEquip action(prevItem);
-            action.execute(actor);
+        // check if the item is still in the player's inventory
+        MWWorld::ContainerStoreIterator it = store.begin();
+        for (; it != store.end(); ++it)
+        {
+            if (*it == prevItem)
+                break;
         }
+
+        // we should equip previous item only if expired bound item was equipped.
+        if (it == store.end() || !wasEquipped)
+            return;
+
+        MWWorld::ActionEquip action(prevItem);
+        action.execute(actor);
     }
 
     void Actors::updateActor (const MWWorld::Ptr& ptr, float duration)
@@ -839,17 +840,14 @@ namespace MWMechanics
                 std::string item = MWBase::Environment::get().getWorld()->getStore().get<ESM::GameSetting>().find(
                             itemGmst)->getString();
 
+                magnitude > 0 ? addBoundItem(item, ptr) : removeBoundItem(item, ptr);
+
                 if (it->first == ESM::MagicEffect::BoundGloves)
                 {
-                    item = MWBase::Environment::get().getWorld()->getStore().get<ESM::GameSetting>().find(
-                                "sMagicBoundLeftGauntletID")->getString();
-                    adjustBoundItem(item, magnitude > 0, ptr);
                     item = MWBase::Environment::get().getWorld()->getStore().get<ESM::GameSetting>().find(
                                 "sMagicBoundRightGauntletID")->getString();
-                    adjustBoundItem(item, magnitude > 0, ptr);
+                    magnitude > 0 ? addBoundItem(item, ptr) : removeBoundItem(item, ptr);
                 }
-                else
-                    adjustBoundItem(item, magnitude > 0, ptr);
             }
         }
 

apps/openmw/mwmechanics/actors.hpp

@@ -28,7 +28,8 @@ namespace MWMechanics
             typedef std::map<std::string, MWWorld::Ptr> PreviousItems;
             PreviousItems mPreviousItems;
 
-            void adjustBoundItem (const std::string& itemId, bool bound, const MWWorld::Ptr& actor);
+            void addBoundItem (const std::string& itemId, const MWWorld::Ptr& actor);
+            void removeBoundItem (const std::string& itemId, const MWWorld::Ptr& actor);
 
             void updateNpc(const MWWorld::Ptr &ptr, float duration);