From 009adc43242c1a33eb59ea4a89213c915d857c38 Mon Sep 17 00:00:00 2001 From: ariel- Date: Sun, 20 Nov 2016 03:34:20 -0300 Subject: [PATCH] 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. --- src/server/game/Spells/Spell.cpp | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/server/game/Spells/Spell.cpp b/src/server/game/Spells/Spell.cpp index def833e1a0..835b11096c 100644 --- a/src/server/game/Spells/Spell.cpp +++ b/src/server/game/Spells/Spell.cpp @@ -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