Consolidate item consumption code

apps/openmw/mwclass/npc.cpp

@@ -1100,12 +1100,14 @@ namespace MWClass
         return getNpcStats(ptr).isWerewolf() ? 0.0f : Actor::getEncumbrance(ptr);
     }
 
-    bool Npc::apply (const MWWorld::Ptr& ptr, const std::string& id,
+    bool Npc::consume(const MWWorld::Ptr& consumable, const MWWorld::Ptr& actor) const
-        const MWWorld::Ptr& actor) const
     {
-        MWBase::Environment::get().getLuaManager()->appliedToObject(ptr, id, actor);
+        MWBase::Environment::get().getWorld()->breakInvisibility(actor);
-        MWMechanics::CastSpell cast(ptr, ptr);
+        MWMechanics::CastSpell cast(actor, actor);
-        return cast.cast(id);
+        std::string recordId = consumable.getCellRef().getRefId();
+        MWBase::Environment::get().getLuaManager()->appliedToObject(actor, recordId, actor);
+        actor.getClass().getContainerStore(actor).remove(consumable, 1, actor);
+        return cast.cast(recordId);
     }
 
     void Npc::skillUsageSucceeded (const MWWorld::Ptr& ptr, int skill, int usageType, float extraFactor) const

apps/openmw/mwclass/npc.hpp

@@ -110,11 +110,7 @@ namespace MWClass
             float getArmorRating (const MWWorld::Ptr& ptr) const override;
             ///< @return combined armor rating of this actor
 
-            bool apply (const MWWorld::Ptr& ptr, const std::string& id,
+            bool consume(const MWWorld::Ptr& consumable, const MWWorld::Ptr& actor) const override;
-                const MWWorld::Ptr& actor) const override;
-            ///< Apply \a id on \a ptr.
-            /// \param actor Actor that is resposible for the ID being applied to \a ptr.
-            /// \return Any effect?
 
             void adjustScale (const MWWorld::ConstPtr &ptr, osg::Vec3f &scale, bool rendering) const override;
             /// @param rendering Indicates if the scale to adjust is for the rendering mesh, or for the collision mesh

apps/openmw/mwmechanics/aicombataction.cpp

@@ -85,8 +85,7 @@ namespace MWMechanics
 
     void ActionPotion::prepare(const MWWorld::Ptr &actor)
     {
-        actor.getClass().apply(actor, mPotion.getCellRef().getRefId(), actor);
+        actor.getClass().consume(mPotion, actor);
-        actor.getClass().getContainerStore(actor).remove(mPotion, 1, actor);
     }
 
     void ActionWeapon::prepare(const MWWorld::Ptr &actor)

apps/openmw/mwworld/actionapply.cpp

@@ -17,11 +17,7 @@ namespace MWWorld
 
     void ActionApply::executeImp (const Ptr& actor)
     {
-        MWBase::Environment::get().getWorld()->breakInvisibility(actor);
+        actor.getClass().consume(getTarget(), actor);
-
-        actor.getClass().apply (actor, mId, actor);
-
-        actor.getClass().getContainerStore(actor).remove(getTarget(), 1, actor);
     }
 
 
@@ -32,11 +28,8 @@ namespace MWWorld
 
     void ActionApplyWithSkill::executeImp (const Ptr& actor)
     {
-        MWBase::Environment::get().getWorld()->breakInvisibility(actor);
+        bool consumed = actor.getClass().consume(getTarget(), actor);
-
+        if (consumed && mUsageType != -1 && actor == MWMechanics::getPlayer())
-        if (actor.getClass().apply (actor, mId, actor) && mUsageType!=-1 && actor == MWMechanics::getPlayer())
             actor.getClass().skillUsageSucceeded (actor, mSkillIndex, mUsageType);
-
-        actor.getClass().getContainerStore(actor).remove(getTarget(), 1, actor);
     }
 }

apps/openmw/mwworld/actioneat.cpp

@@ -12,13 +12,7 @@ namespace MWWorld
 {
     void ActionEat::executeImp (const Ptr& actor)
     {
-        // remove used item (assume the item is present in inventory)
+        if (actor.getClass().consume(getTarget(), actor) && actor == MWMechanics::getPlayer())
-        getTarget().getContainerStore()->remove(getTarget(), 1, actor);
-
-        // apply to actor
-        std::string id = getTarget().getCellRef().getRefId();
-
-        if (actor.getClass().apply (actor, id, actor) && actor == MWMechanics::getPlayer())
             actor.getClass().skillUsageSucceeded (actor, ESM::Skill::Alchemy, 1);
     }
 

apps/openmw/mwworld/class.cpp

@@ -40,7 +40,7 @@ namespace MWWorld
     void Class::insertObjectPhysics(const Ptr& ptr, const std::string& mesh, const osg::Quat& rotation, MWPhysics::PhysicsSystem& physics) const
     {}
 
-    bool Class::apply (const MWWorld::Ptr& ptr, const std::string& id,  const MWWorld::Ptr& actor) const
+    bool Class::consume(const MWWorld::Ptr& consumable, const MWWorld::Ptr& actor) const
     {
         return false;
     }

apps/openmw/mwworld/class.hpp

@@ -220,13 +220,8 @@ namespace MWWorld
             virtual float getNormalizedEncumbrance (const MWWorld::Ptr& ptr) const;
             ///< Returns encumbrance re-scaled to capacity
 
-            virtual bool apply (const MWWorld::Ptr& ptr, const std::string& id,
+            virtual bool consume(const MWWorld::Ptr& consumable, const MWWorld::Ptr& actor) const;
-                const MWWorld::Ptr& actor) const;
+            /// Consume an item, e. g. a potion.
-            ///< Apply \a id on \a ptr.
-            /// \param actor Actor that is resposible for the ID being applied to \a ptr.
-            /// \return Any effect?
-            ///
-            /// (default implementation: ignore and return false)
 
             virtual void skillUsageSucceeded (const MWWorld::Ptr& ptr, int skill, int usageType, float extraFactor=1.f) const;
             ///< Inform actor \a ptr that a skill use has succeeded.