Core/Conditions: Refactor ConditionMgr internals to get rid of separate containers for some condition source types

(cherry picked from commit 0b5406dd882c6d96bc1be6fd0a78375c3b316415)

1 files changed, 3 insertions, 3 deletions

diff --git a/src/server/game/Loot/Loot.cpp b/src/server/game/Loot/Loot.cpp
index c19a2a1c36c..eb19f3eb42f 100644
--- a/src/server/game/Loot/Loot.cpp
+++ b/src/server/game/Loot/Loot.cpp
@@ -74,10 +74,10 @@ bool LootItem::AllowedForPlayer(Player const* player, Loot const* loot) const
 }
 
 bool LootItem::AllowedForPlayer(Player const* player, Loot const* loot, uint32 itemid, bool needs_quest, bool follow_loot_rules, bool strictUsabilityCheck,
-    ConditionContainer const& conditions)
+    ConditionsReference const& conditions)
 {
     // DB conditions check
-    if (!sConditionMgr->IsObjectMeetToConditions(player, conditions))
+    if (!conditions.Meets(player))
         return false;
 
     ItemTemplate const* pProto = sObjectMgr->GetItemTemplate(itemid);
@@ -954,7 +954,7 @@ bool Loot::hasItemForAll() const
         return true;
 
     for (LootItem const& item : items)
-        if (!item.is_looted && item.follow_loot_rules && !item.freeforall && item.conditions.empty())
+        if (!item.is_looted && item.follow_loot_rules && !item.freeforall && item.conditions.IsEmpty())
             return true;
     return false;
 }