Revert "Core/Spell: add EquipedItemClass/EquippedItemSubClass check to passive spell casts"

This reverts commit f188ff1ce47905d7a81dff987a66b3d72152ae7c.

Was breaking Titan's Grip.
Spells are loaded before items, and they CAST the spell on the player.
When items are then loaded, Player::ApplyItemDependentAuras only has AddAura on it. Titan's Grip is an effect, no auras involved.

In turn reverting this commit will now allow Two-handed weapon specialization and similar from spells load. Which is the lesser of two evils.
This commit is contained in:
ariel- authored and Aokromes committed 2016-11-27 03:11:27 +01:00
1 parent 6283cce52f
commit 009adc4324
1 file changed
-4
-4
View File
@@ -5159,10 +5159,6 @@ SpellCastResult Spell::CheckCast(bool strict, uint32* param1 /*= nullptr*/, uint
if (castResult != SPELL_CAST_OK)
return castResult;
}
// passive spells must check equipped items class (55108 for example)
else if (m_caster->GetTypeId() == TYPEID_PLAYER && !m_spellInfo->HasAttribute(SPELL_ATTR2_IGNORE_ITEM_CHECK) && (!(_triggeredCastFlags & TRIGGERED_IGNORE_EQUIPPED_ITEM_REQUIREMENT)))
if (!m_caster->ToPlayer()->HasItemFitToSpellRequirements(m_spellInfo))
return SPELL_FAILED_EQUIPPED_ITEM_CLASS;
// Triggered spells also have range check
/// @todo determine if there is some flag to enable/disable the check