aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/server/scripts/Spells/spell_dk.cpp74
-rw-r--r--src/server/scripts/Spells/spell_druid.cpp94
-rw-r--r--src/server/scripts/Spells/spell_generic.cpp226
-rw-r--r--src/server/scripts/Spells/spell_holiday.cpp103
-rw-r--r--src/server/scripts/Spells/spell_hunter.cpp80
-rw-r--r--src/server/scripts/Spells/spell_item.cpp375
-rw-r--r--src/server/scripts/Spells/spell_mage.cpp96
-rw-r--r--src/server/scripts/Spells/spell_paladin.cpp111
-rw-r--r--src/server/scripts/Spells/spell_pet.cpp2
-rw-r--r--src/server/scripts/Spells/spell_priest.cpp95
-rw-r--r--src/server/scripts/Spells/spell_quest.cpp138
-rw-r--r--src/server/scripts/Spells/spell_rogue.cpp70
-rw-r--r--src/server/scripts/Spells/spell_shaman.cpp148
-rw-r--r--src/server/scripts/Spells/spell_warlock.cpp118
-rw-r--r--src/server/scripts/Spells/spell_warrior.cpp77
15 files changed, 684 insertions, 1123 deletions
diff --git a/src/server/scripts/Spells/spell_dk.cpp b/src/server/scripts/Spells/spell_dk.cpp
index 0bed78d7b07..7fd84c261ac 100644
--- a/src/server/scripts/Spells/spell_dk.cpp
+++ b/src/server/scripts/Spells/spell_dk.cpp
@@ -27,6 +27,7 @@
#include "Player.h"
#include "SpellAuraEffects.h"
#include "SpellHistory.h"
+#include "SpellMgr.h"
#include "SpellScript.h"
enum DeathKnightSpells
@@ -144,13 +145,12 @@ class spell_dk_anti_magic_shell : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DK_RUNIC_POWER_ENERGIZE))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_DK_GLYPH_OF_ABSORB_MAGIC))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_DK_GLYPH_OF_REGENERATIVE_MAGIC))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_DK_RUNIC_POWER_ENERGIZE,
+ SPELL_DK_GLYPH_OF_ABSORB_MAGIC,
+ SPELL_DK_GLYPH_OF_REGENERATIVE_MAGIC
+ });
}
bool Load() override
@@ -329,11 +329,7 @@ class spell_dk_blood_boil : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DK_BLOOD_PLAGUE))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_DK_FROST_FEVER))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DK_BLOOD_PLAGUE, SPELL_DK_FROST_FEVER });
}
void FilterTargets(std::list<WorldObject*>& targets)
@@ -527,9 +523,12 @@ class spell_dk_death_coil : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DK_DEATH_COIL_DAMAGE) || !sSpellMgr->GetSpellInfo(SPELL_DK_DEATH_COIL_HEAL) || !sSpellMgr->GetSpellInfo(SPELL_DK_GLYPH_OF_DEATH_COIL))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_DK_DEATH_COIL_DAMAGE,
+ SPELL_DK_DEATH_COIL_HEAL,
+ SPELL_DK_GLYPH_OF_DEATH_COIL
+ });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -709,11 +708,12 @@ class spell_dk_death_strike : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DK_DEATH_STRIKE_HEAL) ||
- !sSpellMgr->GetSpellInfo(SPELL_DK_BLOOD_SHIELD_MASTERY) ||
- !sSpellMgr->GetSpellInfo(SPELL_DK_BLOOD_SHIELD_ABSORB))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_DK_DEATH_STRIKE_HEAL,
+ SPELL_DK_BLOOD_SHIELD_MASTERY,
+ SPELL_DK_BLOOD_SHIELD_ABSORB
+ });
}
void HandleHeal(SpellEffIndex /*effIndex*/)
@@ -755,9 +755,7 @@ class spell_dk_festering_strike : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DK_FROST_FEVER) || !sSpellMgr->GetSpellInfo(SPELL_DK_BLOOD_PLAGUE) || !sSpellMgr->GetSpellInfo(SPELL_DK_CHAINS_OF_ICE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DK_FROST_FEVER, SPELL_DK_BLOOD_PLAGUE, SPELL_DK_CHAINS_OF_ICE });
}
void HandleScriptEffect(SpellEffIndex /*effIndex*/)
@@ -812,9 +810,7 @@ class spell_dk_ghoul_explode : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DK_CORPSE_EXPLOSION_TRIGGERED))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DK_CORPSE_EXPLOSION_TRIGGERED });
}
void HandleDamage(SpellEffIndex /*effIndex*/)
@@ -886,9 +882,7 @@ class spell_dk_glyph_of_runic_power : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DK_GLYPH_OF_RUNIC_POWER_TRIGGERED))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DK_GLYPH_OF_RUNIC_POWER_TRIGGERED });
}
bool Load() override
@@ -962,9 +956,7 @@ class spell_dk_mark_of_blood : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DK_MARK_OF_BLOOD_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DK_MARK_OF_BLOOD_HEAL });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -999,9 +991,7 @@ class spell_dk_necrosis : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DK_NECROSIS_EFFECT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DK_NECROSIS_EFFECT });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -1104,9 +1094,7 @@ class spell_dk_pvp_4p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DK_RUNIC_RETURN))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DK_RUNIC_RETURN });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -1154,9 +1142,7 @@ class spell_dk_raise_dead : public SpellScriptLoader
private:
bool Validate(SpellInfo const* spellInfo) override
{
- if (!spellInfo->GetEffect(EFFECT_0) || !sSpellMgr->GetSpellInfo(spellInfo->GetEffect(EFFECT_0)->CalcValue()))
- return false;
- return true;
+ return spellInfo->GetEffect(EFFECT_0) && ValidateSpellInfo({ uint32(spellInfo->GetEffect(EFFECT_0)->CalcValue()) });
}
bool Load() override
@@ -1298,11 +1284,7 @@ class spell_dk_will_of_the_necropolis : public SpellScriptLoader
bool Validate(SpellInfo const* spellInfo) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DK_WILL_OF_THE_NECROPOLIS))
- return false;
- if (!spellInfo->GetEffect(EFFECT_0))
- return false;
- return true;
+ return spellInfo->GetEffect(EFFECT_0) && ValidateSpellInfo({ SPELL_DK_WILL_OF_THE_NECROPOLIS });
}
bool CheckProc(ProcEventInfo& eventInfo)
diff --git a/src/server/scripts/Spells/spell_druid.cpp b/src/server/scripts/Spells/spell_druid.cpp
index 0773a2c39f3..e28bd4a8236 100644
--- a/src/server/scripts/Spells/spell_druid.cpp
+++ b/src/server/scripts/Spells/spell_druid.cpp
@@ -21,12 +21,13 @@
* Scriptnames of files in this file should be prefixed with "spell_dru_".
*/
-#include "Player.h"
#include "ScriptMgr.h"
-#include "SpellScript.h"
+#include "Containers.h"
+#include "Player.h"
#include "SpellAuraEffects.h"
#include "SpellHistory.h"
-#include "Containers.h"
+#include "SpellMgr.h"
+#include "SpellScript.h"
enum DruidSpells
{
@@ -134,13 +135,14 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_FORMS_TRINKET_BEAR) ||
- !sSpellMgr->GetSpellInfo(SPELL_DRUID_FORMS_TRINKET_CAT) ||
- !sSpellMgr->GetSpellInfo(SPELL_DRUID_FORMS_TRINKET_MOONKIN) ||
- !sSpellMgr->GetSpellInfo(SPELL_DRUID_FORMS_TRINKET_NONE) ||
- !sSpellMgr->GetSpellInfo(SPELL_DRUID_FORMS_TRINKET_TREE))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_DRUID_FORMS_TRINKET_BEAR,
+ SPELL_DRUID_FORMS_TRINKET_CAT,
+ SPELL_DRUID_FORMS_TRINKET_MOONKIN,
+ SPELL_DRUID_FORMS_TRINKET_NONE,
+ SPELL_DRUID_FORMS_TRINKET_TREE
+ });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -285,11 +287,7 @@ public:
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_LIFEBLOOM_FINAL_HEAL))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_LIFEBLOOM_ENERGIZE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DRUID_LIFEBLOOM_FINAL_HEAL, SPELL_DRUID_LIFEBLOOM_ENERGIZE });
}
void AfterRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
@@ -377,9 +375,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_LIVING_SEED_PROC))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DRUID_LIVING_SEED_PROC });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -413,9 +409,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_LIVING_SEED_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DRUID_LIVING_SEED_HEAL });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& /*eventInfo*/)
@@ -552,10 +546,7 @@ class spell_dru_omen_of_clarity : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_BALANCE_T10_BONUS) ||
- !sSpellMgr->GetSpellInfo(SPELL_DRUID_BALANCE_T10_BONUS_PROC))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DRUID_BALANCE_T10_BONUS, SPELL_DRUID_BALANCE_T10_BONUS_PROC });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& /*eventInfo*/)
@@ -608,9 +599,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_SAVAGE_ROAR))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DRUID_SAVAGE_ROAR });
}
void AfterApply(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
@@ -654,13 +643,14 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_STAMPEDE_BAER_RANK_1) ||
- !sSpellMgr->GetSpellInfo(SPELL_DRUID_STAMPEDE_CAT_RANK_1) ||
- !sSpellMgr->GetSpellInfo(SPELL_DRUID_STAMPEDE_CAT_STATE) ||
- !sSpellMgr->GetSpellInfo(SPELL_DRUID_FERAL_CHARGE_CAT) ||
- !sSpellMgr->GetSpellInfo(SPELL_DRUID_FERAL_CHARGE_BEAR))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_DRUID_STAMPEDE_BAER_RANK_1,
+ SPELL_DRUID_STAMPEDE_CAT_RANK_1,
+ SPELL_DRUID_STAMPEDE_CAT_STATE,
+ SPELL_DRUID_FERAL_CHARGE_CAT,
+ SPELL_DRUID_FERAL_CHARGE_BEAR
+ });
}
void HandleEffectCatProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -800,9 +790,7 @@ public:
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_SURVIVAL_INSTINCTS))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DRUID_SURVIVAL_INSTINCTS });
}
void AfterApply(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
@@ -881,9 +869,7 @@ class spell_dru_t3_6p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_BLESSING_OF_THE_CLAW))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DRUID_BLESSING_OF_THE_CLAW });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -916,9 +902,7 @@ class spell_dru_t3_8p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_EXHILARATE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DRUID_EXHILARATE });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -963,9 +947,7 @@ class spell_dru_t4_2p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_INFUSION))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DRUID_INFUSION });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -998,11 +980,11 @@ class spell_dru_item_t6_trinket : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_BLESSING_OF_REMULOS) ||
- !sSpellMgr->GetSpellInfo(SPELL_DRUID_BLESSING_OF_ELUNE) ||
- !sSpellMgr->GetSpellInfo(SPELL_DRUID_BLESSING_OF_CENARIUS))
- return false;
- return true;
+ return ValidateSpellInfo({
+ SPELL_DRUID_BLESSING_OF_REMULOS,
+ SPELL_DRUID_BLESSING_OF_ELUNE,
+ SPELL_DRUID_BLESSING_OF_CENARIUS
+ });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -1064,9 +1046,7 @@ class spell_dru_t10_balance_4p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_LANGUISH))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DRUID_LANGUISH });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -1168,9 +1148,7 @@ class spell_dru_t10_restoration_4p_bonus_dummy : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DRUID_REJUVENATION_T10_PROC))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DRUID_REJUVENATION_T10_PROC });
}
bool CheckProc(ProcEventInfo& eventInfo)
diff --git a/src/server/scripts/Spells/spell_generic.cpp b/src/server/scripts/Spells/spell_generic.cpp
index fbd9a8363ee..25e1ce45c75 100644
--- a/src/server/scripts/Spells/spell_generic.cpp
+++ b/src/server/scripts/Spells/spell_generic.cpp
@@ -24,12 +24,12 @@
#include "ScriptMgr.h"
#include "Battleground.h"
-#include "Cell.h"
#include "CellImpl.h"
-#include "GridNotifiers.h"
+#include "DB2Stores.h"
#include "GridNotifiersImpl.h"
#include "Group.h"
#include "InstanceScript.h"
+#include "Item.h"
#include "LFGMgr.h"
#include "Log.h"
#include "NPCPackets.h"
@@ -38,6 +38,7 @@
#include "SkillDiscovery.h"
#include "SpellAuraEffects.h"
#include "SpellHistory.h"
+#include "SpellMgr.h"
#include "SpellScript.h"
#include "Vehicle.h"
@@ -104,13 +105,14 @@ class spell_gen_adaptive_warding : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_GEN_ADAPTIVE_WARDING_FIRE) ||
- !sSpellMgr->GetSpellInfo(SPELL_GEN_ADAPTIVE_WARDING_NATURE) ||
- !sSpellMgr->GetSpellInfo(SPELL_GEN_ADAPTIVE_WARDING_FROST) ||
- !sSpellMgr->GetSpellInfo(SPELL_GEN_ADAPTIVE_WARDING_SHADOW) ||
- !sSpellMgr->GetSpellInfo(SPELL_GEN_ADAPTIVE_WARDING_ARCANE))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_GEN_ADAPTIVE_WARDING_FIRE,
+ SPELL_GEN_ADAPTIVE_WARDING_NATURE,
+ SPELL_GEN_ADAPTIVE_WARDING_FROST,
+ SPELL_GEN_ADAPTIVE_WARDING_SHADOW,
+ SPELL_GEN_ADAPTIVE_WARDING_ARCANE
+ });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -219,9 +221,7 @@ class spell_gen_animal_blood : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SPAWN_BLOOD_POOL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SPAWN_BLOOD_POOL });
}
void OnApply(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -301,9 +301,7 @@ class spell_gen_aura_service_uniform : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SERVICE_UNIFORM))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SERVICE_UNIFORM });
}
void OnApply(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -398,9 +396,7 @@ class spell_gen_bandage : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_RECENTLY_BANDAGED))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_RECENTLY_BANDAGED });
}
SpellCastResult CheckCast()
@@ -450,9 +446,7 @@ class spell_gen_blood_reserve : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_GEN_BLOOD_RESERVE_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_GEN_BLOOD_RESERVE_HEAL });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -515,10 +509,8 @@ class spell_gen_bonked : public SpellScriptLoader
target->RemoveAurasDueToSpell(SPELL_BONKED);
if (Aura const* auraOnGuard = target->GetAura(SPELL_ON_GUARD))
- {
if (Item* item = target->GetItemByGuid(auraOnGuard->GetCastItemGUID()))
target->DestroyItemCount(item->GetEntry(), 1, true);
- }
}
}
@@ -687,9 +679,7 @@ class spell_gen_burning_depths_necrolyte_image : public SpellScriptLoader
bool Validate(SpellInfo const* spellInfo) override
{
- if (!sSpellMgr->GetSpellInfo(uint32(spellInfo->GetEffect(EFFECT_2)->CalcValue())))
- return false;
- return true;
+ return ValidateSpellInfo({ uint32(spellInfo->GetEffect(EFFECT_2)->CalcValue()) });
}
void HandleApply(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -732,9 +722,7 @@ class spell_gen_cannibalize : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_CANNIBALIZE_TRIGGERED))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_CANNIBALIZE_TRIGGERED });
}
SpellCastResult CheckIfCorpseNear()
@@ -787,10 +775,9 @@ class spell_gen_chaos_blast : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_CHAOS_BLAST))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_CHAOS_BLAST });
}
+
void HandleDummy(SpellEffIndex /* effIndex */)
{
int32 basepoints0 = 100;
@@ -909,14 +896,15 @@ class spell_gen_clone_weapon_aura : public SpellScriptLoader
private:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_COPY_WEAPON_AURA) ||
- !sSpellMgr->GetSpellInfo(SPELL_COPY_WEAPON_2_AURA) ||
- !sSpellMgr->GetSpellInfo(SPELL_COPY_WEAPON_3_AURA) ||
- !sSpellMgr->GetSpellInfo(SPELL_COPY_OFFHAND_AURA) ||
- !sSpellMgr->GetSpellInfo(SPELL_COPY_OFFHAND_2_AURA) ||
- !sSpellMgr->GetSpellInfo(SPELL_COPY_RANGED_AURA))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_COPY_WEAPON_AURA,
+ SPELL_COPY_WEAPON_2_AURA,
+ SPELL_COPY_WEAPON_3_AURA,
+ SPELL_COPY_OFFHAND_AURA,
+ SPELL_COPY_OFFHAND_2_AURA,
+ SPELL_COPY_RANGED_AURA
+ });
}
void OnApply(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -1070,10 +1058,7 @@ class spell_gen_create_lance : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_CREATE_LANCE_ALLIANCE) ||
- !sSpellMgr->GetSpellInfo(SPELL_CREATE_LANCE_HORDE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_CREATE_LANCE_ALLIANCE, SPELL_CREATE_LANCE_HORDE });
}
void HandleScript(SpellEffIndex effIndex)
@@ -1157,15 +1142,9 @@ class spell_gen_dalaran_disguise : public SpellScriptLoader
switch (spellInfo->Id)
{
case SPELL_SUNREAVER_DISGUISE_TRIGGER:
- if (!sSpellMgr->GetSpellInfo(SPELL_SUNREAVER_DISGUISE_FEMALE) ||
- !sSpellMgr->GetSpellInfo(SPELL_SUNREAVER_DISGUISE_MALE))
- return false;
- break;
+ return ValidateSpellInfo({ SPELL_SUNREAVER_DISGUISE_FEMALE, SPELL_SUNREAVER_DISGUISE_MALE });
case SPELL_SILVER_COVENANT_DISGUISE_TRIGGER:
- if (!sSpellMgr->GetSpellInfo(SPELL_SILVER_COVENANT_DISGUISE_FEMALE) ||
- !sSpellMgr->GetSpellInfo(SPELL_SILVER_COVENANT_DISGUISE_MALE))
- return false;
- break;
+ return ValidateSpellInfo({ SPELL_SILVER_COVENANT_DISGUISE_FEMALE, SPELL_SILVER_COVENANT_DISGUISE_MALE });
}
return true;
}
@@ -1224,13 +1203,7 @@ class spell_gen_defend : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_VISUAL_SHIELD_1))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_VISUAL_SHIELD_2))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_VISUAL_SHIELD_3))
- return false;
- return false;
+ return ValidateSpellInfo({ SPELL_VISUAL_SHIELD_1, SPELL_VISUAL_SHIELD_2, SPELL_VISUAL_SHIELD_3 });
}
void RefreshVisualShields(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
@@ -1350,9 +1323,7 @@ class spell_gen_divine_storm_cd_reset : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DIVINE_STORM))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DIVINE_STORM });
}
void HandleScript(SpellEffIndex /*effIndex*/)
@@ -1481,13 +1452,14 @@ class spell_gen_elune_candle : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ELUNE_CANDLE_OMEN_HEAD) ||
- !sSpellMgr->GetSpellInfo(SPELL_ELUNE_CANDLE_OMEN_CHEST) ||
- !sSpellMgr->GetSpellInfo(SPELL_ELUNE_CANDLE_OMEN_HAND_R) ||
- !sSpellMgr->GetSpellInfo(SPELL_ELUNE_CANDLE_OMEN_HAND_L) ||
- !sSpellMgr->GetSpellInfo(SPELL_ELUNE_CANDLE_NORMAL))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_ELUNE_CANDLE_OMEN_HEAD,
+ SPELL_ELUNE_CANDLE_OMEN_CHEST,
+ SPELL_ELUNE_CANDLE_OMEN_HAND_R,
+ SPELL_ELUNE_CANDLE_OMEN_HAND_L,
+ SPELL_ELUNE_CANDLE_NORMAL
+ });
}
void HandleScript(SpellEffIndex /*effIndex*/)
@@ -1599,11 +1571,12 @@ class spell_gen_gadgetzan_transporter_backfire : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_TRANSPORTER_MALFUNCTION_POLYMORPH) ||
- !sSpellMgr->GetSpellInfo(SPELL_TRANSPORTER_EVIL_TWIN) ||
- !sSpellMgr->GetSpellInfo(SPELL_TRANSPORTER_MALFUNCTION_MISS))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_TRANSPORTER_MALFUNCTION_POLYMORPH,
+ SPELL_TRANSPORTER_EVIL_TWIN,
+ SPELL_TRANSPORTER_MALFUNCTION_MISS
+ });
}
void HandleDummy(SpellEffIndex /* effIndex */)
@@ -1700,10 +1673,7 @@ class spell_gen_gnomish_transporter : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_TRANSPORTER_SUCCESS) ||
- !sSpellMgr->GetSpellInfo(SPELL_TRANSPORTER_FAILURE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_TRANSPORTER_SUCCESS, SPELL_TRANSPORTER_FAILURE });
}
void HandleDummy(SpellEffIndex /* effIndex */)
@@ -1742,9 +1712,7 @@ class spell_gen_interrupt : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_GEN_THROW_INTERRUPT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_GEN_THROW_INTERRUPT });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -1817,9 +1785,7 @@ class spell_gen_lifebloom : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(_spellId))
- return false;
- return true;
+ return ValidateSpellInfo({ _spellId });
}
void AfterRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
@@ -2017,7 +1983,7 @@ class spell_gen_mounted_charge: public SpellScriptLoader
}
};
-enum Netherbloom
+enum Netherbloom : uint32
{
SPELL_NETHERBLOOM_POLLEN_1 = 28703
};
@@ -2035,7 +2001,7 @@ class spell_gen_netherbloom : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
for (uint8 i = 0; i < 5; ++i)
- if (!sSpellMgr->GetSpellInfo(SPELL_NETHERBLOOM_POLLEN_1 + i))
+ if (!ValidateSpellInfo({ SPELL_NETHERBLOOM_POLLEN_1 + i }))
return false;
return true;
}
@@ -2092,9 +2058,7 @@ class spell_gen_nightmare_vine : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_NIGHTMARE_POLLEN))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_NIGHTMARE_POLLEN });
}
void HandleScript(SpellEffIndex effIndex)
@@ -2143,14 +2107,15 @@ class spell_gen_obsidian_armor : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_GEN_OBSIDIAN_ARMOR_HOLY) ||
- !sSpellMgr->GetSpellInfo(SPELL_GEN_OBSIDIAN_ARMOR_FIRE) ||
- !sSpellMgr->GetSpellInfo(SPELL_GEN_OBSIDIAN_ARMOR_NATURE) ||
- !sSpellMgr->GetSpellInfo(SPELL_GEN_OBSIDIAN_ARMOR_FROST) ||
- !sSpellMgr->GetSpellInfo(SPELL_GEN_OBSIDIAN_ARMOR_SHADOW) ||
- !sSpellMgr->GetSpellInfo(SPELL_GEN_OBSIDIAN_ARMOR_ARCANE))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_GEN_OBSIDIAN_ARMOR_HOLY,
+ SPELL_GEN_OBSIDIAN_ARMOR_FIRE,
+ SPELL_GEN_OBSIDIAN_ARMOR_NATURE,
+ SPELL_GEN_OBSIDIAN_ARMOR_FROST,
+ SPELL_GEN_OBSIDIAN_ARMOR_SHADOW,
+ SPELL_GEN_OBSIDIAN_ARMOR_ARCANE
+ });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -2529,11 +2494,7 @@ class spell_gen_orc_disguise : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ORC_DISGUISE_TRIGGER) ||
- !sSpellMgr->GetSpellInfo(SPELL_ORC_DISGUISE_MALE) ||
- !sSpellMgr->GetSpellInfo(SPELL_ORC_DISGUISE_FEMALE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ORC_DISGUISE_TRIGGER, SPELL_ORC_DISGUISE_MALE, SPELL_ORC_DISGUISE_FEMALE });
}
void HandleScript(SpellEffIndex /*effIndex*/)
@@ -2614,10 +2575,7 @@ class spell_gen_parachute : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PARACHUTE) ||
- !sSpellMgr->GetSpellInfo(SPELL_PARACHUTE_BUFF))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PARACHUTE, SPELL_PARACHUTE_BUFF });
}
void HandleEffectPeriodic(AuraEffect const* /*aurEff*/)
@@ -2973,9 +2931,7 @@ class spell_gen_running_wild : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ALTERED_FORM))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ALTERED_FORM });
}
bool Load() override
@@ -3093,9 +3049,7 @@ class spell_gen_seaforium_blast : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PLANT_CHARGES_CREDIT_ACHIEVEMENT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PLANT_CHARGES_CREDIT_ACHIEVEMENT });
}
bool Load() override
@@ -3181,7 +3135,7 @@ class spell_gen_spirit_healer_res : public SpellScriptLoader
{
WorldPackets::NPC::SpiritHealerConfirm spiritHealerConfirm;
spiritHealerConfirm.Unit = target->GetGUID();
- originalCaster->GetSession()->SendPacket(spiritHealerConfirm.Write());
+ originalCaster->SendDirectMessage(spiritHealerConfirm.Write());
}
}
@@ -3217,9 +3171,7 @@ class spell_gen_summon_elemental : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(_spellId))
- return false;
- return true;
+ return ValidateSpellInfo({ _spellId });
}
void AfterApply(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -3272,9 +3224,7 @@ class spell_gen_summon_tournament_mount : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_LANCE_EQUIPPED))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_LANCE_EQUIPPED });
}
SpellCastResult CheckIfLanceEquiped()
@@ -3348,10 +3298,7 @@ class spell_gen_tournament_duel : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ON_TOURNAMENT_MOUNT) ||
- !sSpellMgr->GetSpellInfo(SPELL_MOUNTED_DUEL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ON_TOURNAMENT_MOUNT, SPELL_MOUNTED_DUEL });
}
void HandleScriptEffect(SpellEffIndex /*effIndex*/)
@@ -3440,10 +3387,7 @@ class spell_pvp_trinket_wotf_shared_cd : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WILL_OF_THE_FORSAKEN_COOLDOWN_TRIGGER) ||
- !sSpellMgr->GetSpellInfo(SPELL_WILL_OF_THE_FORSAKEN_COOLDOWN_TRIGGER_WOTF))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WILL_OF_THE_FORSAKEN_COOLDOWN_TRIGGER, SPELL_WILL_OF_THE_FORSAKEN_COOLDOWN_TRIGGER_WOTF });
}
void HandleScript()
@@ -3579,9 +3523,7 @@ class spell_gen_vampiric_touch : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_VAMPIRIC_TOUCH_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_VAMPIRIC_TOUCH_HEAL });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -3746,9 +3688,7 @@ class spell_gen_whisper_gulch_yogg_saron_whisper : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_YOGG_SARON_WHISPER_DUMMY))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_YOGG_SARON_WHISPER_DUMMY });
}
void HandleEffectPeriodic(AuraEffect const* /*aurEff*/)
@@ -3850,9 +3790,7 @@ class spell_gen_gm_freeze : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_GM_FREEZE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_GM_FREEZE });
}
void OnApply(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -4044,9 +3982,7 @@ public:
private:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MIXOLOGY))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MIXOLOGY });
}
bool Load() override
@@ -4464,9 +4400,7 @@ class spell_gen_mark_of_kazrogal_hellfire : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MARK_OF_KAZROGAL_DAMAGE_HELLFIRE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MARK_OF_KAZROGAL_DAMAGE_HELLFIRE });
}
void OnPeriodic(AuraEffect const* aurEff)
@@ -4542,9 +4476,7 @@ class spell_gen_face_rage : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_FACE_RAGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_FACE_RAGE });
}
void OnRemove(AuraEffect const* /*effect*/, AuraEffectHandleModes /*mode*/)
@@ -4576,9 +4508,7 @@ class spell_gen_impatient_mind : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_IMPATIENT_MIND))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_IMPATIENT_MIND });
}
void OnRemove(AuraEffect const* effect, AuraEffectHandleModes /*mode*/)
diff --git a/src/server/scripts/Spells/spell_holiday.cpp b/src/server/scripts/Spells/spell_holiday.cpp
index 1f60cd7fb93..23c4aa759c9 100644
--- a/src/server/scripts/Spells/spell_holiday.cpp
+++ b/src/server/scripts/Spells/spell_holiday.cpp
@@ -136,10 +136,7 @@ class spell_hallow_end_candy : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- for (uint32 spellId : spells)
- if (!sSpellMgr->GetSpellInfo(spellId))
- return false;
- return true;
+ return ValidateSpellInfo(spells);
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -182,10 +179,7 @@ class spell_hallow_end_candy_pirate : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HALLOWS_END_CANDY_FEMALE_DEFIAS_PIRATE)
- || !sSpellMgr->GetSpellInfo(SPELL_HALLOWS_END_CANDY_MALE_DEFIAS_PIRATE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HALLOWS_END_CANDY_FEMALE_DEFIAS_PIRATE, SPELL_HALLOWS_END_CANDY_MALE_DEFIAS_PIRATE });
}
void HandleApply(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -239,11 +233,19 @@ class spell_hallow_end_trick : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PIRATE_COSTUME_MALE) || !sSpellMgr->GetSpellInfo(SPELL_PIRATE_COSTUME_FEMALE) || !sSpellMgr->GetSpellInfo(SPELL_NINJA_COSTUME_MALE)
- || !sSpellMgr->GetSpellInfo(SPELL_NINJA_COSTUME_FEMALE) || !sSpellMgr->GetSpellInfo(SPELL_LEPER_GNOME_COSTUME_MALE) || !sSpellMgr->GetSpellInfo(SPELL_LEPER_GNOME_COSTUME_FEMALE)
- || !sSpellMgr->GetSpellInfo(SPELL_SKELETON_COSTUME) || !sSpellMgr->GetSpellInfo(SPELL_GHOST_COSTUME_MALE) || !sSpellMgr->GetSpellInfo(SPELL_GHOST_COSTUME_FEMALE) || !sSpellMgr->GetSpellInfo(SPELL_TRICK_BUFF))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_PIRATE_COSTUME_MALE,
+ SPELL_PIRATE_COSTUME_FEMALE,
+ SPELL_NINJA_COSTUME_MALE,
+ SPELL_NINJA_COSTUME_FEMALE,
+ SPELL_LEPER_GNOME_COSTUME_MALE,
+ SPELL_LEPER_GNOME_COSTUME_FEMALE,
+ SPELL_SKELETON_COSTUME,
+ SPELL_GHOST_COSTUME_MALE,
+ SPELL_GHOST_COSTUME_FEMALE,
+ SPELL_TRICK_BUFF
+ });
}
void HandleScript(SpellEffIndex /*effIndex*/)
@@ -312,9 +314,7 @@ class spell_hallow_end_trick_or_treat : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_TRICK) || !sSpellMgr->GetSpellInfo(SPELL_TREAT) || !sSpellMgr->GetSpellInfo(SPELL_TRICKED_OR_TREATED))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_TRICK, SPELL_TREAT, SPELL_TRICKED_OR_TREATED });
}
void HandleScript(SpellEffIndex /*effIndex*/)
@@ -350,13 +350,7 @@ class spell_hallow_end_tricky_treat : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_TRICKY_TREAT_SPEED))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_TRICKY_TREAT_TRIGGER))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_UPSET_TUMMY))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_TRICKY_TREAT_SPEED, SPELL_TRICKY_TREAT_TRIGGER, SPELL_UPSET_TUMMY });
}
void HandleScript(SpellEffIndex /*effIndex*/)
@@ -403,16 +397,17 @@ public:
bool Validate(SpellInfo const* /*spellEntry*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PIRATE_COSTUME_MALE) ||
- !sSpellMgr->GetSpellInfo(SPELL_PIRATE_COSTUME_FEMALE) ||
- !sSpellMgr->GetSpellInfo(SPELL_NINJA_COSTUME_MALE) ||
- !sSpellMgr->GetSpellInfo(SPELL_NINJA_COSTUME_FEMALE) ||
- !sSpellMgr->GetSpellInfo(SPELL_LEPER_GNOME_COSTUME_MALE) ||
- !sSpellMgr->GetSpellInfo(SPELL_LEPER_GNOME_COSTUME_FEMALE) ||
- !sSpellMgr->GetSpellInfo(SPELL_GHOST_COSTUME_MALE) ||
- !sSpellMgr->GetSpellInfo(SPELL_GHOST_COSTUME_FEMALE))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_PIRATE_COSTUME_MALE,
+ SPELL_PIRATE_COSTUME_FEMALE,
+ SPELL_NINJA_COSTUME_MALE,
+ SPELL_NINJA_COSTUME_FEMALE,
+ SPELL_LEPER_GNOME_COSTUME_MALE,
+ SPELL_LEPER_GNOME_COSTUME_FEMALE,
+ SPELL_GHOST_COSTUME_MALE,
+ SPELL_GHOST_COSTUME_FEMALE
+ });
}
void HandleScriptEffect()
@@ -611,9 +606,7 @@ class spell_pilgrims_bounty_turkey_tracker : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_KILL_COUNTER_VISUAL) || !sSpellMgr->GetSpellInfo(SPELL_KILL_COUNTER_VISUAL_MAX))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_KILL_COUNTER_VISUAL, SPELL_KILL_COUNTER_VISUAL_MAX });
}
void HandleScript(SpellEffIndex /*effIndex*/)
@@ -688,9 +681,7 @@ class spell_pilgrims_bounty_well_fed : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(_triggeredSpellId))
- return false;
- return true;
+ return ValidateSpellInfo({ _triggeredSpellId });
}
void HandleScript(SpellEffIndex effIndex)
@@ -754,11 +745,12 @@ class spell_winter_veil_mistletoe : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_CREATE_MISTLETOE) ||
- !sSpellMgr->GetSpellInfo(SPELL_CREATE_HOLLY) ||
- !sSpellMgr->GetSpellInfo(SPELL_CREATE_SNOWFLAKES))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_CREATE_MISTLETOE,
+ SPELL_CREATE_HOLLY,
+ SPELL_CREATE_SNOWFLAKES
+ });
}
void HandleScript(SpellEffIndex /*effIndex*/)
@@ -802,12 +794,13 @@ class spell_winter_veil_px_238_winter_wondervolt : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PX_238_WINTER_WONDERVOLT_TRANSFORM_1) ||
- !sSpellMgr->GetSpellInfo(SPELL_PX_238_WINTER_WONDERVOLT_TRANSFORM_2) ||
- !sSpellMgr->GetSpellInfo(SPELL_PX_238_WINTER_WONDERVOLT_TRANSFORM_3) ||
- !sSpellMgr->GetSpellInfo(SPELL_PX_238_WINTER_WONDERVOLT_TRANSFORM_4))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_PX_238_WINTER_WONDERVOLT_TRANSFORM_1,
+ SPELL_PX_238_WINTER_WONDERVOLT_TRANSFORM_2,
+ SPELL_PX_238_WINTER_WONDERVOLT_TRANSFORM_3,
+ SPELL_PX_238_WINTER_WONDERVOLT_TRANSFORM_4
+ });
}
void HandleScript(SpellEffIndex effIndex)
@@ -1285,9 +1278,7 @@ class spell_midsummer_braziers_hit : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_TORCH_TOSSING_TRAINING) || !sSpellMgr->GetSpellInfo(SPELL_TORCH_TOSSING_PRACTICE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_TORCH_TOSSING_TRAINING, SPELL_TORCH_TOSSING_PRACTICE });
}
void HandleEffectApply(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -1338,11 +1329,7 @@ class spell_gen_ribbon_pole_dancer_check : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HAS_FULL_MIDSUMMER_SET)
- || !sSpellMgr->GetSpellInfo(SPELL_RIBBON_DANCE)
- || !sSpellMgr->GetSpellInfo(SPELL_BURNING_HOT_POLE_DANCE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HAS_FULL_MIDSUMMER_SET, SPELL_RIBBON_DANCE, SPELL_BURNING_HOT_POLE_DANCE });
}
void PeriodicTick(AuraEffect const* /*aurEff*/)
diff --git a/src/server/scripts/Spells/spell_hunter.cpp b/src/server/scripts/Spells/spell_hunter.cpp
index 033f1f06dc3..e37f65fa559 100644
--- a/src/server/scripts/Spells/spell_hunter.cpp
+++ b/src/server/scripts/Spells/spell_hunter.cpp
@@ -21,15 +21,14 @@
* Scriptnames of files in this file should be prefixed with "spell_hun_".
*/
-#include "Pet.h"
#include "ScriptMgr.h"
-#include "Cell.h"
#include "CellImpl.h"
-#include "GridNotifiers.h"
#include "GridNotifiersImpl.h"
-#include "SpellScript.h"
+#include "Pet.h"
#include "SpellAuraEffects.h"
#include "SpellHistory.h"
+#include "SpellMgr.h"
+#include "SpellScript.h"
enum HunterSpells
{
@@ -79,12 +78,13 @@ class spell_hun_ancient_hysteria : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HUNTER_INSANITY)
- || !sSpellMgr->GetSpellInfo(SPELL_MAGE_TEMPORAL_DISPLACEMENT)
- || !sSpellMgr->GetSpellInfo(SPELL_SHAMAN_EXHAUSTION)
- || !sSpellMgr->GetSpellInfo(SPELL_SHAMAN_SATED))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_HUNTER_INSANITY,
+ SPELL_MAGE_TEMPORAL_DISPLACEMENT,
+ SPELL_SHAMAN_EXHAUSTION,
+ SPELL_SHAMAN_SATED
+ });
}
void RemoveInvalidTargets(std::list<WorldObject*>& targets)
@@ -162,10 +162,7 @@ class spell_hun_chimera_shot : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HUNTER_CHIMERA_SHOT_HEAL) ||
- !sSpellMgr->GetSpellInfo(SPELL_HUNTER_SERPENT_STING))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HUNTER_CHIMERA_SHOT_HEAL, SPELL_HUNTER_SERPENT_STING });
}
bool Load() override
@@ -205,10 +202,7 @@ class spell_hun_cobra_shot : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HUNTER_GENERIC_ENERGIZE_FOCUS) ||
- !sSpellMgr->GetSpellInfo(SPELL_HUNTER_SERPENT_STING))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HUNTER_GENERIC_ENERGIZE_FOCUS, SPELL_HUNTER_SERPENT_STING });
}
bool Load() override
@@ -320,9 +314,7 @@ class spell_hun_improved_mend_pet : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HUNTER_IMPROVED_MEND_PET))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HUNTER_IMPROVED_MEND_PET });
}
bool CheckProc(ProcEventInfo& /*eventInfo*/)
@@ -361,9 +353,7 @@ class spell_hun_last_stand_pet : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HUNTER_PET_LAST_STAND_TRIGGERED))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HUNTER_PET_LAST_STAND_TRIGGERED });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -397,10 +387,7 @@ class spell_hun_masters_call : public SpellScriptLoader
bool Validate(SpellInfo const* spellInfo) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HUNTER_MASTERS_CALL_TRIGGERED) ||
- !sSpellMgr->GetSpellInfo(spellInfo->GetEffect(EFFECT_0)->CalcValue()))
- return false;
- return true;
+ return spellInfo->GetEffect(EFFECT_0) && ValidateSpellInfo({ SPELL_HUNTER_MASTERS_CALL_TRIGGERED, uint32(spellInfo->GetEffect(EFFECT_0)->CalcValue()) });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -449,9 +436,7 @@ class spell_hun_misdirection : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HUNTER_MISDIRECTION_PROC))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HUNTER_MISDIRECTION_PROC });
}
void OnRemove(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -531,9 +516,7 @@ class spell_hun_pet_carrion_feeder : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HUNTER_PET_CARRION_FEEDER_TRIGGERED))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HUNTER_PET_CARRION_FEEDER_TRIGGERED });
}
SpellCastResult CheckIfCorpseNear()
@@ -590,9 +573,7 @@ class spell_hun_pet_heart_of_the_phoenix : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HUNTER_PET_HEART_OF_THE_PHOENIX_TRIGGERED) || !sSpellMgr->GetSpellInfo(SPELL_HUNTER_PET_HEART_OF_THE_PHOENIX_DEBUFF))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HUNTER_PET_HEART_OF_THE_PHOENIX_TRIGGERED, SPELL_HUNTER_PET_HEART_OF_THE_PHOENIX_DEBUFF });
}
void HandleScript(SpellEffIndex /*effIndex*/)
@@ -677,9 +658,7 @@ class spell_hun_ready_set_aim : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HUNTER_FIRE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HUNTER_FIRE });
}
void OnApply(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
@@ -715,9 +694,7 @@ class spell_hun_roar_of_sacrifice : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ROAR_OF_SACRIFICE_TRIGGERED))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ROAR_OF_SACRIFICE_TRIGGERED });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -794,10 +771,7 @@ class spell_hun_sniper_training : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HUNTER_SNIPER_TRAINING_R1) ||
- !sSpellMgr->GetSpellInfo(SPELL_HUNTER_SNIPER_TRAINING_BUFF_R1))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HUNTER_SNIPER_TRAINING_R1, SPELL_HUNTER_SNIPER_TRAINING_BUFF_R1 });
}
void HandlePeriodic(AuraEffect const* aurEff)
@@ -855,9 +829,7 @@ class spell_hun_steady_shot : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HUNTER_STEADY_SHOT_FOCUS))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HUNTER_STEADY_SHOT_FOCUS });
}
bool Load() override
@@ -977,9 +949,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HUNTER_T9_4P_GREATNESS))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HUNTER_T9_4P_GREATNESS });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -1022,9 +992,7 @@ class spell_hun_tnt : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HUNTER_LOCK_AND_LOAD))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HUNTER_LOCK_AND_LOAD });
}
bool CheckProc(ProcEventInfo& /*eventInfo*/)
diff --git a/src/server/scripts/Spells/spell_item.cpp b/src/server/scripts/Spells/spell_item.cpp
index 64cc2d34a4e..1a721521ae0 100644
--- a/src/server/scripts/Spells/spell_item.cpp
+++ b/src/server/scripts/Spells/spell_item.cpp
@@ -24,13 +24,17 @@
#include "ScriptMgr.h"
#include "Battleground.h"
#include "CreatureAIImpl.h"
+#include "DB2Stores.h"
+#include "Item.h"
#include "Log.h"
+#include "Map.h"
#include "ObjectMgr.h"
#include "Player.h"
#include "ScriptedCreature.h"
#include "SkillDiscovery.h"
#include "SpellAuraEffects.h"
#include "SpellHistory.h"
+#include "SpellMgr.h"
#include "SpellScript.h"
// Generic script for handling item dummy effects which trigger another spell.
@@ -53,9 +57,7 @@ class spell_item_trigger_spell : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(_triggeredSpellId))
- return false;
- return true;
+ return ValidateSpellInfo({ _triggeredSpellId });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -94,9 +96,7 @@ class spell_item_aegis_of_preservation : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_AEGIS_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_AEGIS_HEAL });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& /*eventInfo*/)
@@ -144,10 +144,7 @@ class spell_item_alchemist_stone : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ALCHEMIST_STONE_EXTRA_HEAL) ||
- !sSpellMgr->GetSpellInfo(SPELL_ALCHEMIST_STONE_EXTRA_MANA))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ALCHEMIST_STONE_EXTRA_HEAL, SPELL_ALCHEMIST_STONE_EXTRA_MANA });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -310,18 +307,18 @@ class spell_item_aura_of_madness : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SOCIOPATH) ||
- !sSpellMgr->GetSpellInfo(SPELL_DELUSIONAL) ||
- !sSpellMgr->GetSpellInfo(SPELL_KLEPTOMANIA) ||
- !sSpellMgr->GetSpellInfo(SPELL_MEGALOMANIA) ||
- !sSpellMgr->GetSpellInfo(SPELL_PARANOIA) ||
- !sSpellMgr->GetSpellInfo(SPELL_MANIC) ||
- !sSpellMgr->GetSpellInfo(SPELL_NARCISSISM) ||
- !sSpellMgr->GetSpellInfo(SPELL_MARTYR_COMPLEX) ||
- !sSpellMgr->GetSpellInfo(SPELL_DEMENTIA) ||
- !sBroadcastTextStore.LookupEntry(SAY_MADNESS))
- return false;
- return true;
+ return sBroadcastTextStore.LookupEntry(SAY_MADNESS) && ValidateSpellInfo(
+ {
+ SPELL_SOCIOPATH,
+ SPELL_DELUSIONAL,
+ SPELL_KLEPTOMANIA,
+ SPELL_MEGALOMANIA,
+ SPELL_PARANOIA,
+ SPELL_MANIC,
+ SPELL_NARCISSISM,
+ SPELL_MARTYR_COMPLEX,
+ SPELL_DEMENTIA
+ });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -387,10 +384,7 @@ class spell_item_dementia : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DEMENTIA_POS) ||
- !sSpellMgr->GetSpellInfo(SPELL_DEMENTIA_NEG))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DEMENTIA_POS, SPELL_DEMENTIA_NEG });
}
void HandlePeriodicDummy(AuraEffect const* /*aurEff*/)
@@ -428,9 +422,7 @@ class spell_item_blessing_of_ancient_kings : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PROTECTION_OF_ANCIENT_KINGS))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PROTECTION_OF_ANCIENT_KINGS });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -517,9 +509,7 @@ class spell_item_deadly_precision_dummy : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DEADLY_PRECISION))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DEADLY_PRECISION });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -706,9 +696,7 @@ class spell_item_defibrillate : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (_failSpell && !sSpellMgr->GetSpellInfo(_failSpell))
- return false;
- return true;
+ return !_failSpell || ValidateSpellInfo({ _failSpell });
}
void HandleScript(SpellEffIndex effIndex)
@@ -758,9 +746,7 @@ class spell_item_desperate_defense : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DESPERATE_RAGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DESPERATE_RAGE });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& /*eventInfo*/)
@@ -808,10 +794,7 @@ class spell_item_deviate_fish : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- for (uint32 spellId = SPELL_SLEEPY; spellId <= SPELL_HEALTHY_SPIRIT; ++spellId)
- if (!sSpellMgr->GetSpellInfo(spellId))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SLEEPY, SPELL_INVIGORATE, SPELL_SHRINK, SPELL_PARTY_TIME, SPELL_HEALTHY_SPIRIT });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -850,9 +833,7 @@ class spell_item_discerning_eye_beast_dummy : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DISCERNING_EYE_BEAST))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_DISCERNING_EYE_BEAST });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -924,9 +905,7 @@ class spell_item_fate_rune_of_unsurpassed_vigor : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_UNSURPASSED_VIGOR))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_UNSURPASSED_VIGOR });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& /*eventInfo*/)
@@ -967,9 +946,12 @@ class spell_item_flask_of_the_north : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_FLASK_OF_THE_NORTH_SP) || !sSpellMgr->GetSpellInfo(SPELL_FLASK_OF_THE_NORTH_AP) || !sSpellMgr->GetSpellInfo(SPELL_FLASK_OF_THE_NORTH_STR))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_FLASK_OF_THE_NORTH_SP,
+ SPELL_FLASK_OF_THE_NORTH_AP,
+ SPELL_FLASK_OF_THE_NORTH_STR
+ });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -1041,9 +1023,7 @@ class spell_item_frozen_shadoweave : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHADOWMEND))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHADOWMEND });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -1089,9 +1069,7 @@ class spell_item_gnomish_death_ray : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_GNOMISH_DEATH_RAY_SELF) || !sSpellMgr->GetSpellInfo(SPELL_GNOMISH_DEATH_RAY_TARGET))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_GNOMISH_DEATH_RAY_SELF, SPELL_GNOMISH_DEATH_RAY_TARGET });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -1261,9 +1239,14 @@ class spell_item_make_a_wish : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MR_PINCHYS_BLESSING) || !sSpellMgr->GetSpellInfo(SPELL_SUMMON_MIGHTY_MR_PINCHY) || !sSpellMgr->GetSpellInfo(SPELL_SUMMON_FURIOUS_MR_PINCHY) || !sSpellMgr->GetSpellInfo(SPELL_TINY_MAGICAL_CRAWDAD) || !sSpellMgr->GetSpellInfo(SPELL_MR_PINCHYS_GIFT))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_MR_PINCHYS_BLESSING,
+ SPELL_SUMMON_MIGHTY_MR_PINCHY,
+ SPELL_SUMMON_FURIOUS_MR_PINCHY,
+ SPELL_TINY_MAGICAL_CRAWDAD,
+ SPELL_MR_PINCHYS_GIFT
+ });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -1311,9 +1294,7 @@ class spell_item_mark_of_conquest : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MARK_OF_CONQUEST_ENERGIZE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MARK_OF_CONQUEST_ENERGIZE });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -1412,9 +1393,7 @@ class spell_item_necrotic_touch : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ITEM_NECROTIC_TOUCH_PROC))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ITEM_NECROTIC_TOUCH_PROC });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -1462,9 +1441,12 @@ class spell_item_net_o_matic : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_NET_O_MATIC_TRIGGERED1) || !sSpellMgr->GetSpellInfo(SPELL_NET_O_MATIC_TRIGGERED2) || !sSpellMgr->GetSpellInfo(SPELL_NET_O_MATIC_TRIGGERED3))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_NET_O_MATIC_TRIGGERED1,
+ SPELL_NET_O_MATIC_TRIGGERED2,
+ SPELL_NET_O_MATIC_TRIGGERED3
+ });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -1519,9 +1501,12 @@ class spell_item_noggenfogger_elixir : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_NOGGENFOGGER_ELIXIR_TRIGGERED1) || !sSpellMgr->GetSpellInfo(SPELL_NOGGENFOGGER_ELIXIR_TRIGGERED2) || !sSpellMgr->GetSpellInfo(SPELL_NOGGENFOGGER_ELIXIR_TRIGGERED3))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_NOGGENFOGGER_ELIXIR_TRIGGERED1,
+ SPELL_NOGGENFOGGER_ELIXIR_TRIGGERED2,
+ SPELL_NOGGENFOGGER_ELIXIR_TRIGGERED3
+ });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -1601,9 +1586,7 @@ class spell_item_persistent_shield : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PERSISTENT_SHIELD_TRIGGERED))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PERSISTENT_SHIELD_TRIGGERED });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -1657,9 +1640,7 @@ class spell_item_pet_healing : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_HEALTH_LINK))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_HEALTH_LINK });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -1741,10 +1722,7 @@ class spell_item_savory_deviate_delight : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- for (uint32 spellId = SPELL_FLIP_OUT_MALE; spellId <= SPELL_YAAARRRR_FEMALE; ++spellId)
- if (!sSpellMgr->GetSpellInfo(spellId))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_FLIP_OUT_MALE, SPELL_FLIP_OUT_FEMALE, SPELL_YAAARRRR_MALE, SPELL_YAAARRRR_FEMALE });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -1939,13 +1917,12 @@ class spell_item_shadowmourne : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHADOWMOURNE_CHAOS_BANE_DAMAGE))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_SHADOWMOURNE_SOUL_FRAGMENT))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_SHADOWMOURNE_CHAOS_BANE_BUFF))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_SHADOWMOURNE_CHAOS_BANE_DAMAGE,
+ SPELL_SHADOWMOURNE_SOUL_FRAGMENT,
+ SPELL_SHADOWMOURNE_CHAOS_BANE_BUFF
+ });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -1996,9 +1973,12 @@ class spell_item_shadowmourne_soul_fragment : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHADOWMOURNE_VISUAL_LOW) || !sSpellMgr->GetSpellInfo(SPELL_SHADOWMOURNE_VISUAL_HIGH) || !sSpellMgr->GetSpellInfo(SPELL_SHADOWMOURNE_CHAOS_BANE_BUFF))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_SHADOWMOURNE_VISUAL_LOW,
+ SPELL_SHADOWMOURNE_VISUAL_HIGH,
+ SPELL_SHADOWMOURNE_CHAOS_BANE_BUFF
+ });
}
void OnStackChange(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -2065,9 +2045,15 @@ class spell_item_six_demon_bag : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_FROSTBOLT) || !sSpellMgr->GetSpellInfo(SPELL_POLYMORPH) || !sSpellMgr->GetSpellInfo(SPELL_SUMMON_FELHOUND_MINION) || !sSpellMgr->GetSpellInfo(SPELL_FIREBALL) || !sSpellMgr->GetSpellInfo(SPELL_CHAIN_LIGHTNING) || !sSpellMgr->GetSpellInfo(SPELL_ENVELOPING_WINDS))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_FROSTBOLT,
+ SPELL_POLYMORPH,
+ SPELL_SUMMON_FELHOUND_MINION,
+ SPELL_FIREBALL,
+ SPELL_CHAIN_LIGHTNING,
+ SPELL_ENVELOPING_WINDS
+ });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -2130,9 +2116,7 @@ class spell_item_swift_hand_justice_dummy : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SWIFT_HAND_OF_JUSTICE_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SWIFT_HAND_OF_JUSTICE_HEAL });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -2209,9 +2193,12 @@ class spell_item_underbelly_elixir : public SpellScriptLoader
}
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_UNDERBELLY_ELIXIR_TRIGGERED1) || !sSpellMgr->GetSpellInfo(SPELL_UNDERBELLY_ELIXIR_TRIGGERED2) || !sSpellMgr->GetSpellInfo(SPELL_UNDERBELLY_ELIXIR_TRIGGERED3))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_UNDERBELLY_ELIXIR_TRIGGERED1,
+ SPELL_UNDERBELLY_ELIXIR_TRIGGERED2,
+ SPELL_UNDERBELLY_ELIXIR_TRIGGERED3
+ });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -2251,7 +2238,7 @@ enum WormholeGeneratorPandariaSpell
SPELL_WORMHOLE_PANDARIA_WHITEPETAL_LAKE = 126764,
};
-std::vector<uint32> const WormholeTargetLocations =
+uint32 const WormholeTargetLocations[] =
{
SPELL_WORMHOLE_PANDARIA_ISLE_OF_RECKONING,
SPELL_WORMHOLE_PANDARIA_KUNLAI_UNDERWATER,
@@ -2276,10 +2263,7 @@ class spell_item_wormhole_pandaria : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- for (uint32 spellId : WormholeTargetLocations)
- if (!sSpellMgr->GetSpellInfo(spellId))
- return false;
- return true;
+ return ValidateSpellInfo(WormholeTargetLocations);
}
void HandleTeleport(SpellEffIndex effIndex)
@@ -2370,9 +2354,7 @@ class spell_item_red_rider_air_rifle : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_AIR_RIFLE_HOLD_VISUAL) || !sSpellMgr->GetSpellInfo(SPELL_AIR_RIFLE_SHOOT) || !sSpellMgr->GetSpellInfo(SPELL_AIR_RIFLE_SHOOT_SELF))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_AIR_RIFLE_HOLD_VISUAL, SPELL_AIR_RIFLE_SHOOT, SPELL_AIR_RIFLE_SHOOT_SELF });
}
void HandleScript(SpellEffIndex effIndex)
@@ -2601,9 +2583,7 @@ class spell_item_vanquished_clutches : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_CRUSHER) || !sSpellMgr->GetSpellInfo(SPELL_CONSTRICTOR) || !sSpellMgr->GetSpellInfo(SPELL_CORRUPTOR))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_CRUSHER, SPELL_CONSTRICTOR, SPELL_CORRUPTOR });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -2787,9 +2767,7 @@ class spell_item_purify_helboar_meat : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SUMMON_PURIFIED_HELBOAR_MEAT) || !sSpellMgr->GetSpellInfo(SPELL_SUMMON_TOXIC_HELBOAR_MEAT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SUMMON_PURIFIED_HELBOAR_MEAT, SPELL_SUMMON_TOXIC_HELBOAR_MEAT });
}
void HandleDummy(SpellEffIndex /* effIndex */)
@@ -2873,11 +2851,14 @@ class spell_item_reindeer_transformation : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_FLYING_REINDEER_310) || !sSpellMgr->GetSpellInfo(SPELL_FLYING_REINDEER_280)
- || !sSpellMgr->GetSpellInfo(SPELL_FLYING_REINDEER_60) || !sSpellMgr->GetSpellInfo(SPELL_REINDEER_100)
- || !sSpellMgr->GetSpellInfo(SPELL_REINDEER_60))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_FLYING_REINDEER_310,
+ SPELL_FLYING_REINDEER_280,
+ SPELL_FLYING_REINDEER_60,
+ SPELL_REINDEER_100,
+ SPELL_REINDEER_60
+ });
}
void HandleDummy(SpellEffIndex /* effIndex */)
@@ -2938,9 +2919,7 @@ class spell_item_nigh_invulnerability : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_NIGH_INVULNERABILITY) || !sSpellMgr->GetSpellInfo(SPELL_COMPLETE_VULNERABILITY))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_NIGH_INVULNERABILITY, SPELL_COMPLETE_VULNERABILITY });
}
void HandleDummy(SpellEffIndex /* effIndex */)
@@ -2984,9 +2963,7 @@ class spell_item_poultryizer : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_POULTRYIZER_SUCCESS) || !sSpellMgr->GetSpellInfo(SPELL_POULTRYIZER_BACKFIRE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_POULTRYIZER_SUCCESS, SPELL_POULTRYIZER_BACKFIRE });
}
void HandleDummy(SpellEffIndex /* effIndex */)
@@ -3028,9 +3005,7 @@ class spell_item_socrethars_stone : public SpellScriptLoader
}
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SOCRETHAR_TO_SEAT) || !sSpellMgr->GetSpellInfo(SPELL_SOCRETHAR_FROM_SEAT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SOCRETHAR_TO_SEAT, SPELL_SOCRETHAR_FROM_SEAT });
}
void HandleDummy(SpellEffIndex /* effIndex */)
@@ -3079,9 +3054,9 @@ class spell_item_demon_broiled_surprise : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_CREATE_DEMON_BROILED_SURPRISE) || !sObjectMgr->GetCreatureTemplate(NPC_ABYSSAL_FLAMEBRINGER) || !sObjectMgr->GetQuestTemplate(QUEST_SUPER_HOT_STEW))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_CREATE_DEMON_BROILED_SURPRISE }) &&
+ sObjectMgr->GetCreatureTemplate(NPC_ABYSSAL_FLAMEBRINGER) &&
+ sObjectMgr->GetQuestTemplate(QUEST_SUPER_HOT_STEW);
}
bool Load() override
@@ -3136,9 +3111,7 @@ class spell_item_complete_raptor_capture : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_RAPTOR_CAPTURE_CREDIT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_RAPTOR_CAPTURE_CREDIT });
}
void HandleDummy(SpellEffIndex /* effIndex */)
@@ -3230,9 +3203,13 @@ class spell_item_brewfest_mount_transformation : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MOUNT_RAM_100) || !sSpellMgr->GetSpellInfo(SPELL_MOUNT_RAM_60) || !sSpellMgr->GetSpellInfo(SPELL_MOUNT_KODO_100) || !sSpellMgr->GetSpellInfo(SPELL_MOUNT_KODO_60))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_MOUNT_RAM_100,
+ SPELL_MOUNT_RAM_60,
+ SPELL_MOUNT_KODO_100,
+ SPELL_MOUNT_KODO_60
+ });
}
void HandleDummy(SpellEffIndex /* effIndex */)
@@ -3300,9 +3277,7 @@ class spell_item_nitro_boots : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_NITRO_BOOTS_SUCCESS) || !sSpellMgr->GetSpellInfo(SPELL_NITRO_BOOTS_BACKFIRE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_NITRO_BOOTS_SUCCESS, SPELL_NITRO_BOOTS_BACKFIRE });
}
void HandleDummy(SpellEffIndex /* effIndex */)
@@ -3346,9 +3321,7 @@ class spell_item_teach_language : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_LEARN_GNOMISH_BINARY) || !sSpellMgr->GetSpellInfo(SPELL_LEARN_GOBLIN_BINARY))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_LEARN_GNOMISH_BINARY, SPELL_LEARN_GOBLIN_BINARY });
}
void HandleDummy(SpellEffIndex /* effIndex */)
@@ -3392,9 +3365,7 @@ class spell_item_rocket_boots : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ROCKET_BOOTS_PROC))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ROCKET_BOOTS_PROC });
}
void HandleDummy(SpellEffIndex /* effIndex */)
@@ -3444,9 +3415,7 @@ class spell_item_pygmy_oil : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PYGMY_OIL_PYGMY_AURA) || !sSpellMgr->GetSpellInfo(SPELL_PYGMY_OIL_SMALLER_AURA))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PYGMY_OIL_PYGMY_AURA, SPELL_PYGMY_OIL_SMALLER_AURA });
}
void HandleDummy(SpellEffIndex /* effIndex */)
@@ -3530,9 +3499,9 @@ class spell_item_chicken_cover : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_CHICKEN_NET) || !sSpellMgr->GetSpellInfo(SPELL_CAPTURE_CHICKEN_ESCAPE) || !sObjectMgr->GetQuestTemplate(QUEST_CHICKEN_PARTY) || !sObjectMgr->GetQuestTemplate(QUEST_FLOWN_THE_COOP))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_CHICKEN_NET, SPELL_CAPTURE_CHICKEN_ESCAPE }) &&
+ sObjectMgr->GetQuestTemplate(QUEST_CHICKEN_PARTY) &&
+ sObjectMgr->GetQuestTemplate(QUEST_FLOWN_THE_COOP);
}
void HandleDummy(SpellEffIndex /* effIndex */)
@@ -3696,12 +3665,13 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SOUL_PRESERVER_DRUID) ||
- !sSpellMgr->GetSpellInfo(SPELL_SOUL_PRESERVER_PALADIN) ||
- !sSpellMgr->GetSpellInfo(SPELL_SOUL_PRESERVER_PRIEST) ||
- !sSpellMgr->GetSpellInfo(SPELL_SOUL_PRESERVER_SHAMAN))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_SOUL_PRESERVER_DRUID,
+ SPELL_SOUL_PRESERVER_PALADIN,
+ SPELL_SOUL_PRESERVER_PRIEST,
+ SPELL_SOUL_PRESERVER_SHAMAN
+ });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -3883,12 +3853,13 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DEATH_CHOICE_NORMAL_STRENGTH) ||
- !sSpellMgr->GetSpellInfo(SPELL_DEATH_CHOICE_NORMAL_AGILITY) ||
- !sSpellMgr->GetSpellInfo(SPELL_DEATH_CHOICE_HEROIC_STRENGTH) ||
- !sSpellMgr->GetSpellInfo(SPELL_DEATH_CHOICE_HEROIC_AGILITY))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_DEATH_CHOICE_NORMAL_STRENGTH,
+ SPELL_DEATH_CHOICE_NORMAL_AGILITY,
+ SPELL_DEATH_CHOICE_HEROIC_STRENGTH,
+ SPELL_DEATH_CHOICE_HEROIC_AGILITY
+ });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -3969,9 +3940,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(_stackSpell) || !sSpellMgr->GetSpellInfo(_triggerSpell))
- return false;
- return true;
+ return ValidateSpellInfo({ _stackSpell, _triggerSpell });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -4034,12 +4003,13 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_DARKMOON_CARD_STRENGHT) ||
- !sSpellMgr->GetSpellInfo(SPELL_DARKMOON_CARD_AGILITY) ||
- !sSpellMgr->GetSpellInfo(SPELL_DARKMOON_CARD_INTELLECT) ||
- !sSpellMgr->GetSpellInfo(SPELL_DARKMOON_CARD_VERSATILITY))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_DARKMOON_CARD_STRENGHT,
+ SPELL_DARKMOON_CARD_AGILITY,
+ SPELL_DARKMOON_CARD_INTELLECT,
+ SPELL_DARKMOON_CARD_VERSATILITY
+ });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -4112,10 +4082,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MANA_DRAIN_ENERGIZE)
- || !sSpellMgr->GetSpellInfo(SPELL_MANA_DRAIN_LEECH))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MANA_DRAIN_ENERGIZE, SPELL_MANA_DRAIN_LEECH });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -4163,11 +4130,8 @@ class spell_item_taunt_flag_targeting : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_TAUNT_FLAG))
- return false;
- if (!sBroadcastTextStore.LookupEntry(EMOTE_PLANTS_FLAG))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_TAUNT_FLAG }) &&
+ sBroadcastTextStore.LookupEntry(EMOTE_PLANTS_FLAG);
}
void FilterTargets(std::list<WorldObject*>& targets)
@@ -4233,9 +4197,7 @@ class spell_item_zandalarian_charm : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(_spellId))
- return false;
- return true;
+ return ValidateSpellInfo({ _spellId });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -4372,9 +4334,7 @@ class spell_item_talisman_of_ascendance : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_TALISMAN_OF_ASCENDANCE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_TALISMAN_OF_ASCENDANCE });
}
void OnRemove(AuraEffect const* effect, AuraEffectHandleModes /*mode*/)
@@ -4406,9 +4366,7 @@ class spell_item_jom_gabbar : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_JOM_GABBAR))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_JOM_GABBAR });
}
void OnRemove(AuraEffect const* effect, AuraEffectHandleModes /*mode*/)
@@ -4440,9 +4398,7 @@ class spell_item_battle_trance : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_BATTLE_TRANCE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_BATTLE_TRANCE });
}
void OnRemove(AuraEffect const* effect, AuraEffectHandleModes /*mode*/)
@@ -4474,9 +4430,7 @@ class spell_item_world_queller_focus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WORLD_QUELLER_FOCUS))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WORLD_QUELLER_FOCUS });
}
void OnRemove(AuraEffect const* effect, AuraEffectHandleModes /*mode*/)
@@ -4512,17 +4466,14 @@ class spell_item_water_strider : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_AZURE_WATER_STRIDER))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_CRIMSON_WATER_STRIDER))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_ORANGE_WATER_STRIDER))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_JADE_WATER_STRIDER))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_GOLDEN_WATER_STRIDER))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_AZURE_WATER_STRIDER,
+ SPELL_CRIMSON_WATER_STRIDER,
+ SPELL_ORANGE_WATER_STRIDER,
+ SPELL_JADE_WATER_STRIDER,
+ SPELL_GOLDEN_WATER_STRIDER
+ });
}
void OnRemove(AuraEffect const* /*effect*/, AuraEffectHandleModes /*mode*/)
@@ -4555,11 +4506,7 @@ class spell_item_brutal_kinship : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_BRUTAL_KINSHIP_1))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_BRUTAL_KINSHIP_2))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_BRUTAL_KINSHIP_1, SPELL_BRUTAL_KINSHIP_2 });
}
void OnRemove(AuraEffect const* effect, AuraEffectHandleModes /*mode*/)
diff --git a/src/server/scripts/Spells/spell_mage.cpp b/src/server/scripts/Spells/spell_mage.cpp
index 9b75fb508eb..ec456e2c42f 100644
--- a/src/server/scripts/Spells/spell_mage.cpp
+++ b/src/server/scripts/Spells/spell_mage.cpp
@@ -28,6 +28,7 @@
#include "Player.h"
#include "SpellAuraEffects.h"
#include "SpellHistory.h"
+#include "SpellMgr.h"
#include "SpellScript.h"
enum MageSpells
@@ -286,9 +287,7 @@ class spell_mage_focus_magic : public SpellScriptLoader
private:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_FOCUS_MAGIC_PROC))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MAGE_FOCUS_MAGIC_PROC });
}
bool CheckProc(ProcEventInfo& /*eventInfo*/)
@@ -392,9 +391,7 @@ class spell_mage_glyph_of_polymorph : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_SHADOW_WORD_DEATH))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_SHADOW_WORD_DEATH });
}
void HandleEffectProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -432,9 +429,7 @@ class spell_mage_imp_mana_gems : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_MANA_SURGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MAGE_MANA_SURGE });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -469,9 +464,7 @@ class spell_mage_living_bomb : public SpellScriptLoader
{
if (!spellInfo->GetEffect(EFFECT_1))
return false;
- if (!sSpellMgr->GetSpellInfo(uint32(spellInfo->GetEffect(EFFECT_1)->CalcValue())))
- return false;
- return true;
+ return ValidateSpellInfo({ uint32(spellInfo->GetEffect(EFFECT_1)->CalcValue()) });
}
void AfterRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
@@ -555,9 +548,7 @@ class spell_mage_ignite : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_IGNITE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MAGE_IGNITE });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -602,9 +593,7 @@ class spell_mage_master_of_elements : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_MASTER_OF_ELEMENTS_ENERGIZE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MAGE_MASTER_OF_ELEMENTS_ENERGIZE });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -651,9 +640,7 @@ class spell_mage_nether_vortex : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_SLOW))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MAGE_SLOW });
}
bool DoCheck(ProcEventInfo& eventInfo)
@@ -696,9 +683,7 @@ class spell_mage_permafrost : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_PERMAFROST))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MAGE_PERMAFROST });
}
bool DoCheck(ProcEventInfo& eventInfo)
@@ -739,11 +724,12 @@ class spell_mage_polymorph : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_IMPROVED_POLYMORPH_RANK_1) ||
- !sSpellMgr->GetSpellInfo(SPELL_MAGE_IMPROVED_POLYMORPH_STUN_RANK_1) ||
- !sSpellMgr->GetSpellInfo(SPELL_MAGE_IMPROVED_POLYMORPH_MARKER))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_MAGE_IMPROVED_POLYMORPH_RANK_1,
+ SPELL_MAGE_IMPROVED_POLYMORPH_STUN_RANK_1,
+ SPELL_MAGE_IMPROVED_POLYMORPH_MARKER
+ });
}
bool DoCheck(ProcEventInfo& eventInfo)
@@ -802,11 +788,7 @@ class spell_mage_polymorph_cast_visual : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- // check if spell ids exist in dbc
- for (uint32 i = 0; i < 6; i++)
- if (!sSpellMgr->GetSpellInfo(PolymorhForms[i]))
- return false;
- return true;
+ return ValidateSpellInfo(PolymorhForms);
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -881,13 +863,12 @@ class spell_mage_ring_of_frost : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_RING_OF_FROST_SUMMON))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_RING_OF_FROST_FREEZE))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_RING_OF_FROST_DUMMY))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_MAGE_RING_OF_FROST_SUMMON,
+ SPELL_MAGE_RING_OF_FROST_FREEZE,
+ SPELL_MAGE_RING_OF_FROST_DUMMY
+ });
}
void HandleEffectPeriodic(AuraEffect const* /*aurEff*/)
@@ -958,11 +939,7 @@ class spell_mage_ring_of_frost_freeze : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_RING_OF_FROST_SUMMON))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_RING_OF_FROST_FREEZE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MAGE_RING_OF_FROST_SUMMON, SPELL_MAGE_RING_OF_FROST_FREEZE });
}
void FilterTargets(std::list<WorldObject*>& targets)
@@ -997,9 +974,7 @@ class spell_mage_ring_of_frost_freeze : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_RING_OF_FROST_DUMMY))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MAGE_RING_OF_FROST_DUMMY });
}
void OnRemove(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -1033,13 +1008,14 @@ class spell_mage_time_warp : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_TEMPORAL_DISPLACEMENT)
- || !sSpellMgr->GetSpellInfo(SPELL_HUNTER_INSANITY)
- || !sSpellMgr->GetSpellInfo(SPELL_SHAMAN_EXHAUSTION)
- || !sSpellMgr->GetSpellInfo(SPELL_SHAMAN_SATED)
- || !sSpellMgr->GetSpellInfo(SPELL_PET_NETHERWINDS_FATIGUED))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_MAGE_TEMPORAL_DISPLACEMENT,
+ SPELL_HUNTER_INSANITY,
+ SPELL_SHAMAN_EXHAUSTION,
+ SPELL_SHAMAN_SATED,
+ SPELL_PET_NETHERWINDS_FATIGUED
+ });
}
void RemoveInvalidTargets(std::list<WorldObject*>& targets)
@@ -1083,9 +1059,7 @@ class spell_mage_trigger_chilled : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_CHILLED))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MAGE_CHILLED });
}
void HandleChilled()
@@ -1118,9 +1092,7 @@ class spell_mage_water_elemental_freeze : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MAGE_FINGERS_OF_FROST))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MAGE_FINGERS_OF_FROST });
}
void HandleImprovedFreeze()
diff --git a/src/server/scripts/Spells/spell_paladin.cpp b/src/server/scripts/Spells/spell_paladin.cpp
index ba8d6eaa17b..af62ef1de7a 100644
--- a/src/server/scripts/Spells/spell_paladin.cpp
+++ b/src/server/scripts/Spells/spell_paladin.cpp
@@ -21,13 +21,14 @@
* Scriptnames of files in this file should be prefixed with "spell_pal_".
*/
-#include "Player.h"
#include "ScriptMgr.h"
-#include "SpellScript.h"
-#include "SpellAuraEffects.h"
-#include "SpellHistory.h"
#include "Group.h"
+#include "Player.h"
#include "Random.h"
+#include "SpellAuraEffects.h"
+#include "SpellHistory.h"
+#include "SpellMgr.h"
+#include "SpellScript.h"
enum PaladinSpells
{
@@ -172,9 +173,7 @@ class spell_pal_aura_mastery : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_AURA_MASTERY_IMMUNE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_AURA_MASTERY_IMMUNE });
}
void HandleEffectApply(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -212,9 +211,7 @@ class spell_pal_aura_mastery_immune : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_CONCENTRACTION_AURA))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_CONCENTRACTION_AURA });
}
bool CheckAreaTarget(Unit* target)
@@ -246,10 +243,7 @@ class spell_pal_avenging_wrath : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_SANCTIFIED_WRATH)
- || !sSpellMgr->GetSpellInfo(SPELL_PALADIN_SANCTIFIED_WRATH_TALENT_R1))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_SANCTIFIED_WRATH, SPELL_PALADIN_SANCTIFIED_WRATH_TALENT_R1 });
}
void HandleApply(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -292,9 +286,13 @@ class spell_pal_blessing_of_faith : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_BLESSING_OF_LOWER_CITY_DRUID) || !sSpellMgr->GetSpellInfo(SPELL_PALADIN_BLESSING_OF_LOWER_CITY_PALADIN) || !sSpellMgr->GetSpellInfo(SPELL_PALADIN_BLESSING_OF_LOWER_CITY_PRIEST) || !sSpellMgr->GetSpellInfo(SPELL_PALADIN_BLESSING_OF_LOWER_CITY_SHAMAN))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_PALADIN_BLESSING_OF_LOWER_CITY_DRUID,
+ SPELL_PALADIN_BLESSING_OF_LOWER_CITY_PALADIN,
+ SPELL_PALADIN_BLESSING_OF_LOWER_CITY_PRIEST,
+ SPELL_PALADIN_BLESSING_OF_LOWER_CITY_SHAMAN
+ });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -348,12 +346,13 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_DIVINE_STEED_HUMAN) ||
- !sSpellMgr->GetSpellInfo(SPELL_PALADIN_DIVINE_STEED_DRAENEI) ||
- !sSpellMgr->GetSpellInfo(SPELL_PALADIN_DIVINE_STEED_BLOODELF) ||
- !sSpellMgr->GetSpellInfo(SPELL_PALADIN_DIVINE_STEED_TAUREN))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_PALADIN_DIVINE_STEED_HUMAN,
+ SPELL_PALADIN_DIVINE_STEED_DRAENEI,
+ SPELL_PALADIN_DIVINE_STEED_BLOODELF,
+ SPELL_PALADIN_DIVINE_STEED_TAUREN
+ });
}
void HandleOnCast()
@@ -407,9 +406,7 @@ class spell_pal_divine_storm : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_DIVINE_STORM_DAMAGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_DIVINE_STORM_DAMAGE });
}
void HandleOnCast()
@@ -489,9 +486,7 @@ class spell_pal_eye_for_an_eye : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_EYE_FOR_AN_EYE_DAMAGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_EYE_FOR_AN_EYE_DAMAGE });
}
void HandleEffectProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -525,9 +520,7 @@ class spell_pal_grand_crusader : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_AVENGERS_SHIELD))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_AVENGERS_SHIELD });
}
bool CheckProc(ProcEventInfo& /*eventInfo*/)
@@ -721,9 +714,7 @@ class spell_pal_item_healing_discount : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_ITEM_HEALING_TRANCE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_ITEM_HEALING_TRANCE });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& /*eventInfo*/)
@@ -756,10 +747,7 @@ class spell_pal_item_t6_trinket : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_ENDURING_LIGHT) ||
- !sSpellMgr->GetSpellInfo(SPELL_PALADIN_ENDURING_JUDGEMENT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_ENDURING_LIGHT, SPELL_PALADIN_ENDURING_JUDGEMENT });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -816,9 +804,7 @@ class spell_pal_judgement : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_JUDGEMENT_DAMAGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_JUDGEMENT_DAMAGE });
}
void HandleScriptEffect(SpellEffIndex /*effIndex*/)
@@ -866,11 +852,7 @@ class spell_pal_lay_on_hands : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_FORBEARANCE))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_IMMUNE_SHIELD_MARKER))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_FORBEARANCE, SPELL_PALADIN_IMMUNE_SHIELD_MARKER });
}
SpellCastResult CheckCast()
@@ -919,10 +901,7 @@ class spell_pal_light_s_beacon : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_BEACON_OF_LIGHT)
- || !sSpellMgr->GetSpellInfo(SPELL_PALADIN_BEACON_OF_LIGHT_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_BEACON_OF_LIGHT, SPELL_PALADIN_BEACON_OF_LIGHT_HEAL });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -983,9 +962,7 @@ class spell_pal_righteous_defense : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_RIGHTEOUS_DEFENSE_TAUNT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_RIGHTEOUS_DEFENSE_TAUNT });
}
SpellCastResult CheckCast()
@@ -1131,10 +1108,7 @@ class spell_pal_templar_s_verdict : public SpellScriptLoader
bool Validate (SpellInfo const* /*spellEntry*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_DIVINE_PURPOSE_PROC))
- return false;
-
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_DIVINE_PURPOSE_PROC });
}
bool Load() override
@@ -1198,9 +1172,7 @@ class spell_pal_seal_of_righteousness : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_SEAL_OF_RIGHTEOUSNESS))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_SEAL_OF_RIGHTEOUSNESS });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -1244,12 +1216,13 @@ class spell_pal_t3_6p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_HOLY_POWER_ARMOR) ||
- !sSpellMgr->GetSpellInfo(SPELL_PALADIN_HOLY_POWER_ATTACK_POWER) ||
- !sSpellMgr->GetSpellInfo(SPELL_PALADIN_HOLY_POWER_SPELL_POWER) ||
- !sSpellMgr->GetSpellInfo(SPELL_PALADIN_HOLY_POWER_MP5))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_PALADIN_HOLY_POWER_ARMOR,
+ SPELL_PALADIN_HOLY_POWER_ATTACK_POWER,
+ SPELL_PALADIN_HOLY_POWER_SPELL_POWER,
+ SPELL_PALADIN_HOLY_POWER_MP5
+ });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -1310,9 +1283,7 @@ class spell_pal_t8_2p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PALADIN_HOLY_MENDING))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PALADIN_HOLY_MENDING });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
diff --git a/src/server/scripts/Spells/spell_pet.cpp b/src/server/scripts/Spells/spell_pet.cpp
index 32d5a81f238..28a9e5178de 100644
--- a/src/server/scripts/Spells/spell_pet.cpp
+++ b/src/server/scripts/Spells/spell_pet.cpp
@@ -26,8 +26,8 @@
#include "Pet.h"
#include "Player.h"
#include "SpellAuraEffects.h"
+#include "SpellMgr.h"
#include "SpellScript.h"
-#include "Unit.h"
enum HunterPetCalculate
{
diff --git a/src/server/scripts/Spells/spell_priest.cpp b/src/server/scripts/Spells/spell_priest.cpp
index 1fb0d3fd0a2..67cd65dcdee 100644
--- a/src/server/scripts/Spells/spell_priest.cpp
+++ b/src/server/scripts/Spells/spell_priest.cpp
@@ -21,13 +21,13 @@
* Scriptnames of files in this file should be prefixed with "spell_pri_".
*/
-#include "Player.h"
#include "ScriptMgr.h"
-#include "SpellScript.h"
-#include "SpellAuraEffects.h"
-#include "GridNotifiers.h"
-#include "AreaTriggerTemplate.h"
#include "AreaTriggerAI.h"
+#include "GridNotifiers.h"
+#include "Player.h"
+#include "SpellAuraEffects.h"
+#include "SpellMgr.h"
+#include "SpellScript.h"
enum PriestSpells
{
@@ -134,9 +134,7 @@ class spell_pri_aq_3p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_ORACULAR_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_ORACULAR_HEAL });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -178,10 +176,7 @@ class spell_pri_body_and_soul : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_CURE_DISEASE) ||
- !sSpellMgr->GetSpellInfo(SPELL_PRIEST_BODY_AND_SOUL_DISPEL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_CURE_DISEASE, SPELL_PRIEST_BODY_AND_SOUL_DISPEL });
}
void HandleEffectSpeedProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -230,9 +225,7 @@ class spell_pri_circle_of_healing : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_GLYPH_OF_CIRCLE_OF_HEALING))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_GLYPH_OF_CIRCLE_OF_HEALING });
}
void FilterTargets(std::list<WorldObject*>& targets)
@@ -272,13 +265,12 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_ABSOLUTION))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_GLYPH_OF_DISPEL_MAGIC_HEAL))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_GLYPH_OF_DISPEL_MAGIC))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_PRIEST_ABSOLUTION,
+ SPELL_PRIEST_GLYPH_OF_DISPEL_MAGIC_HEAL,
+ SPELL_PRIEST_GLYPH_OF_DISPEL_MAGIC
+ });
}
SpellCastResult CheckCast()
@@ -331,9 +323,7 @@ class spell_pri_divine_aegis : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_DIVINE_AEGIS))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_DIVINE_AEGIS });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -420,9 +410,7 @@ class spell_pri_glyph_of_prayer_of_healing : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_GLYPH_OF_PRAYER_OF_HEALING_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_GLYPH_OF_PRAYER_OF_HEALING_HEAL });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -507,9 +495,7 @@ class spell_pri_guardian_spirit : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_GUARDIAN_SPIRIT_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_GUARDIAN_SPIRIT_HEAL });
}
bool Load() override
@@ -598,10 +584,7 @@ class spell_pri_item_t6_trinket : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_DIVINE_BLESSING) ||
- !sSpellMgr->GetSpellInfo(SPELL_PRIEST_DIVINE_WRATH))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_DIVINE_BLESSING, SPELL_PRIEST_DIVINE_WRATH });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -639,9 +622,7 @@ class spell_pri_leap_of_faith_effect_trigger : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_LEAP_OF_FAITH_EFFECT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_LEAP_OF_FAITH_EFFECT });
}
void HandleEffectDummy(SpellEffIndex /*effIndex*/)
@@ -678,9 +659,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_LEVITATE_EFFECT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_LEVITATE_EFFECT });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -779,9 +758,7 @@ class spell_pri_mana_leech : public SpellScriptLoader
private:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_MANA_LEECH_PROC))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_MANA_LEECH_PROC });
}
bool CheckProc(ProcEventInfo& /*eventInfo*/)
@@ -1085,10 +1062,7 @@ class spell_pri_shadowform : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_SHADOWFORM_VISUAL_WITHOUT_GLYPH) ||
- !sSpellMgr->GetSpellInfo(SPELL_PRIEST_SHADOWFORM_VISUAL_WITH_GLYPH))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_SHADOWFORM_VISUAL_WITHOUT_GLYPH, SPELL_PRIEST_SHADOWFORM_VISUAL_WITH_GLYPH });
}
void HandleEffectApply(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -1126,9 +1100,7 @@ class spell_pri_t3_4p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_ARMOR_OF_FAITH))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_ARMOR_OF_FAITH });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -1161,9 +1133,7 @@ class spell_pri_t5_heal_2p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_ITEM_EFFICIENCY))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_ITEM_EFFICIENCY });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -1208,9 +1178,7 @@ class spell_pri_t10_heal_2p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_BLESSED_HEALING))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_BLESSED_HEALING });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -1257,9 +1225,7 @@ class spell_pri_vampiric_embrace : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_VAMPIRIC_EMBRACE_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_VAMPIRIC_EMBRACE_HEAL });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -1333,10 +1299,7 @@ class spell_pri_vampiric_touch : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_VAMPIRIC_TOUCH_DISPEL) ||
- !sSpellMgr->GetSpellInfo(SPELL_GEN_REPLENISHMENT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_VAMPIRIC_TOUCH_DISPEL, SPELL_GEN_REPLENISHMENT });
}
void HandleDispel(DispelInfo* /*dispelInfo*/)
@@ -1392,9 +1355,7 @@ class spell_pri_angelic_feather_trigger : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_PRIEST_ANGELIC_FEATHER_AREATRIGGER))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_PRIEST_ANGELIC_FEATHER_AREATRIGGER });
}
void HandleEffectDummy(SpellEffIndex /*effIndex*/)
diff --git a/src/server/scripts/Spells/spell_quest.cpp b/src/server/scripts/Spells/spell_quest.cpp
index 72eec36d0e5..a6364960102 100644
--- a/src/server/scripts/Spells/spell_quest.cpp
+++ b/src/server/scripts/Spells/spell_quest.cpp
@@ -104,9 +104,7 @@ class spell_q2203_thaumaturgy_channel : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_THAUMATURGY_CHANNEL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_THAUMATURGY_CHANNEL });
}
void HandleEffectPeriodic(AuraEffect const* /*aurEff*/)
@@ -152,9 +150,7 @@ class spell_q5206_test_fetid_skull : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellEntry*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_CREATE_RESONATING_SKULL) || !sSpellMgr->GetSpellInfo(SPELL_CREATE_BONE_DUST))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_CREATE_RESONATING_SKULL, SPELL_CREATE_BONE_DUST });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -316,9 +312,7 @@ class spell_q11396_11399_scourging_crystal_controller : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellEntry*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_FORCE_SHIELD_ARCANE_PURPLE_X3) || !sSpellMgr->GetSpellInfo(SPELL_SCOURGING_CRYSTAL_CONTROLLER))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_FORCE_SHIELD_ARCANE_PURPLE_X3, SPELL_SCOURGING_CRYSTAL_CONTROLLER });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -354,9 +348,7 @@ class spell_q11396_11399_scourging_crystal_controller_dummy : public SpellScript
bool Validate(SpellInfo const* /*spellEntry*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_FORCE_SHIELD_ARCANE_PURPLE_X3))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_FORCE_SHIELD_ARCANE_PURPLE_X3 });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -417,9 +409,7 @@ class spell_q11587_arcane_prisoner_rescue : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellEntry*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SUMMON_ARCANE_PRISONER_MALE) || !sSpellMgr->GetSpellInfo(SPELL_SUMMON_ARCANE_PRISONER_FEMALE) || !sSpellMgr->GetSpellInfo(SPELL_ARCANE_PRISONER_KILL_CREDIT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SUMMON_ARCANE_PRISONER_MALE, SPELL_SUMMON_ARCANE_PRISONER_FEMALE, SPELL_ARCANE_PRISONER_KILL_CREDIT });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -480,9 +470,15 @@ class spell_q11730_ultrasonic_screwdriver : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellEntry*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SUMMON_SCAVENGEBOT_004A8) || !sSpellMgr->GetSpellInfo(SPELL_SUMMON_SENTRYBOT_57K) || !sSpellMgr->GetSpellInfo(SPELL_SUMMON_DEFENDOTANK_66D) || !sSpellMgr->GetSpellInfo(SPELL_SUMMON_SCAVENGEBOT_005B6) || !sSpellMgr->GetSpellInfo(SPELL_SUMMON_55D_COLLECTATRON) || !sSpellMgr->GetSpellInfo(SPELL_ROBOT_KILL_CREDIT))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_SUMMON_SCAVENGEBOT_004A8,
+ SPELL_SUMMON_SENTRYBOT_57K,
+ SPELL_SUMMON_DEFENDOTANK_66D,
+ SPELL_SUMMON_SCAVENGEBOT_005B6,
+ SPELL_SUMMON_55D_COLLECTATRON,
+ SPELL_ROBOT_KILL_CREDIT
+ });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -592,9 +588,13 @@ class spell_q12634_despawn_fruit_tosser : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellEntry*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_BANANAS_FALL_TO_GROUND) || !sSpellMgr->GetSpellInfo(SPELL_ORANGE_FALLS_TO_GROUND) || !sSpellMgr->GetSpellInfo(SPELL_PAPAYA_FALLS_TO_GROUND) || !sSpellMgr->GetSpellInfo(SPELL_SUMMON_ADVENTUROUS_DWARF))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_BANANAS_FALL_TO_GROUND,
+ SPELL_ORANGE_FALLS_TO_GROUND,
+ SPELL_PAPAYA_FALLS_TO_GROUND,
+ SPELL_SUMMON_ADVENTUROUS_DWARF
+ });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -744,9 +744,7 @@ class spell_q12937_relief_for_the_fallen : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellEntry*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_TRIGGER_AID_OF_THE_EARTHEN))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_TRIGGER_AID_OF_THE_EARTHEN });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -790,9 +788,7 @@ class spell_q10041_q10040_who_are_they : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellEntry*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_MALE_DISGUISE) || !sSpellMgr->GetSpellInfo(SPELL_FEMALE_DISGUISE) || !sSpellMgr->GetSpellInfo(SPELL_GENERIC_DISGUISE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_MALE_DISGUISE, SPELL_FEMALE_DISGUISE, SPELL_GENERIC_DISGUISE });
}
void HandleScript(SpellEffIndex effIndex)
@@ -1051,9 +1047,13 @@ class spell_q14112_14145_chum_the_water: public SpellScriptLoader
bool Validate(SpellInfo const* /*spellEntry*/) override
{
- if (!sSpellMgr->GetSpellInfo(SUMMON_ANGRY_KVALDIR) || !sSpellMgr->GetSpellInfo(SUMMON_NORTH_SEA_MAKO) || !sSpellMgr->GetSpellInfo(SUMMON_NORTH_SEA_THRESHER) || !sSpellMgr->GetSpellInfo(SUMMON_NORTH_SEA_BLUE_SHARK))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SUMMON_ANGRY_KVALDIR,
+ SUMMON_NORTH_SEA_MAKO,
+ SUMMON_NORTH_SEA_THRESHER,
+ SUMMON_NORTH_SEA_BLUE_SHARK
+ });
}
void HandleScriptEffect(SpellEffIndex /*effIndex*/)
@@ -1662,9 +1662,7 @@ class spell_q12527_zuldrak_rat : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SUMMON_GORGED_LURKING_BASILISK))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SUMMON_GORGED_LURKING_BASILISK });
}
void HandleScriptEffect(SpellEffIndex /* effIndex */)
@@ -1734,9 +1732,7 @@ class spell_q12730_quenching_mist : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_FLICKERING_FLAMES))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_FLICKERING_FLAMES });
}
void HandleEffectPeriodic(AuraEffect const* /*aurEff*/)
@@ -1777,10 +1773,7 @@ class spell_q13291_q13292_q13239_q13261_frostbrood_skytalon_grab_decoy : public
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_RIDE))
- return false;
-
- return true;
+ return ValidateSpellInfo({ SPELL_RIDE });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -1880,10 +1873,7 @@ class spell_q13011_bear_flank_master : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_BEAR_FLANK_MASTER) ||
- !sSpellMgr->GetSpellInfo(SPELL_CREATE_BEAR_FLANK))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_BEAR_FLANK_MASTER, SPELL_CREATE_BEAR_FLANK });
}
bool Load() override
@@ -1929,9 +1919,7 @@ class spell_q13086_cannons_target : public SpellScriptLoader
bool Validate(SpellInfo const* spellInfo) override
{
- if (!sSpellMgr->GetSpellInfo(spellInfo->GetEffect(EFFECT_0)->CalcValue()))
- return false;
- return true;
+ return ValidateSpellInfo({ uint32(spellInfo->GetEffect(EFFECT_0)->CalcValue()) });
}
void HandleEffectDummy(SpellEffIndex /*effIndex*/)
@@ -1980,14 +1968,15 @@ class spell_q12690_burst_at_the_seams : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_BURST_AT_THE_SEAMS)
- || !sSpellMgr->GetSpellInfo(SPELL_BURST_AT_THE_SEAMS_DMG)
- || !sSpellMgr->GetSpellInfo(SPELL_BURST_AT_THE_SEAMS_DMG_2)
- || !sSpellMgr->GetSpellInfo(SPELL_BURST_AT_THE_SEAMS_BONE)
- || !sSpellMgr->GetSpellInfo(SPELL_BURST_AT_THE_SEAMS_MEAT)
- || !sSpellMgr->GetSpellInfo(SPELL_BURST_AT_THE_SEAMS_BMEAT))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_BURST_AT_THE_SEAMS,
+ SPELL_BURST_AT_THE_SEAMS_DMG,
+ SPELL_BURST_AT_THE_SEAMS_DMG_2,
+ SPELL_BURST_AT_THE_SEAMS_BONE,
+ SPELL_BURST_AT_THE_SEAMS_MEAT,
+ SPELL_BURST_AT_THE_SEAMS_BMEAT
+ });
}
bool Load() override
@@ -2056,9 +2045,7 @@ class spell_q12308_escape_from_silverbrook : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SUMMON_WORGEN))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SUMMON_WORGEN });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -2134,12 +2121,13 @@ class spell_q12641_death_comes_from_on_high : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_FORGE_CREDIT) ||
- !sSpellMgr->GetSpellInfo(SPELL_TOWN_HALL_CREDIT) ||
- !sSpellMgr->GetSpellInfo(SPELL_SCARLET_HOLD_CREDIT) ||
- !sSpellMgr->GetSpellInfo(SPELL_CHAPEL_CREDIT))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_FORGE_CREDIT,
+ SPELL_TOWN_HALL_CREDIT,
+ SPELL_SCARLET_HOLD_CREDIT,
+ SPELL_CHAPEL_CREDIT
+ });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -2289,9 +2277,7 @@ class spell_q12919_gymers_grab : public SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_RIDE_GYMER))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_RIDE_GYMER });
}
void HandleScript(SpellEffIndex /*effIndex*/)
@@ -2368,9 +2354,7 @@ class spell_q13400_illidan_kill_master : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ILLIDAN_KILL_CREDIT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ILLIDAN_KILL_CREDIT });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -2410,9 +2394,7 @@ class spell_q14100_q14111_make_player_destroy_totems : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_TOTEM_OF_THE_EARTHEN_RING))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_TOTEM_OF_THE_EARTHEN_RING });
}
void HandleScriptEffect(SpellEffIndex /*effIndex*/)
@@ -2451,11 +2433,7 @@ class spell_q10929_fumping : SpellScriptLoader
bool Validate(SpellInfo const* /*spell*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SUMMON_SAND_GNOME))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_SUMMON_BONE_SLICER))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SUMMON_SAND_GNOME, SPELL_SUMMON_BONE_SLICER });
}
void HandleEffectRemove(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -2497,9 +2475,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_RENEWED_LIFE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_RENEWED_LIFE });
}
void HandleDummyEffect()
diff --git a/src/server/scripts/Spells/spell_rogue.cpp b/src/server/scripts/Spells/spell_rogue.cpp
index 894f17e6a05..731af46ec9b 100644
--- a/src/server/scripts/Spells/spell_rogue.cpp
+++ b/src/server/scripts/Spells/spell_rogue.cpp
@@ -23,11 +23,14 @@
#include "ScriptMgr.h"
#include "Containers.h"
+#include "DB2Stores.h"
+#include "Item.h"
#include "Log.h"
#include "ObjectAccessor.h"
#include "Player.h"
#include "SpellAuraEffects.h"
#include "SpellHistory.h"
+#include "SpellMgr.h"
#include "SpellScript.h"
enum RogueSpells
@@ -78,9 +81,7 @@ class spell_rog_blade_flurry : public SpellScriptLoader
private:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ROGUE_BLADE_FLURRY_EXTRA_ATTACK))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ROGUE_BLADE_FLURRY_EXTRA_ATTACK });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -139,9 +140,7 @@ class spell_rog_cheat_death : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ROGUE_CHEAT_DEATH_COOLDOWN))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ROGUE_CHEAT_DEATH_COOLDOWN });
}
bool Load() override
@@ -201,9 +200,7 @@ class spell_rog_crippling_poison : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ROGUE_CRIPPLING_POISON))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ROGUE_CRIPPLING_POISON });
}
void OnProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -371,11 +368,12 @@ class spell_rog_killing_spree : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ROGUE_KILLING_SPREE_TELEPORT)
- || !sSpellMgr->GetSpellInfo(SPELL_ROGUE_KILLING_SPREE_WEAPON_DMG)
- || !sSpellMgr->GetSpellInfo(SPELL_ROGUE_KILLING_SPREE_DMG_BUFF))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_ROGUE_KILLING_SPREE_TELEPORT,
+ SPELL_ROGUE_KILLING_SPREE_WEAPON_DMG,
+ SPELL_ROGUE_KILLING_SPREE_DMG_BUFF
+ });
}
void HandleApply(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -439,9 +437,7 @@ class spell_rog_master_of_subtlety : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ROGUE_MASTER_OF_SUBTLETY_DAMAGE_PERCENT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ROGUE_MASTER_OF_SUBTLETY_DAMAGE_PERCENT });
}
void HandleEffectPeriodic(AuraEffect const* /*aurEff*/)
@@ -481,9 +477,7 @@ class spell_rog_preparation : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ROGUE_GLYPH_OF_PREPARATION))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ROGUE_GLYPH_OF_PREPARATION });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -586,9 +580,7 @@ class spell_rog_shiv : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ROGUE_SHIV_TRIGGERED))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ROGUE_SHIV_TRIGGERED });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -622,14 +614,15 @@ class spell_rog_stealth : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ROGUE_MASTER_OF_SUBTLETY_PASSIVE) ||
- !sSpellMgr->GetSpellInfo(SPELL_ROGUE_MASTER_OF_SUBTLETY_DAMAGE_PERCENT) ||
- !sSpellMgr->GetSpellInfo(SPELL_ROGUE_MASTER_OF_SUBTLETY_PERIODIC) ||
- !sSpellMgr->GetSpellInfo(SPELL_ROGUE_SANCTUARY) ||
- !sSpellMgr->GetSpellInfo(SPELL_ROGUE_STEALTH_STEALTH_AURA) ||
- !sSpellMgr->GetSpellInfo(SPELL_ROGUE_STEALTH_SHAPESHIFT_AURA))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_ROGUE_MASTER_OF_SUBTLETY_PASSIVE,
+ SPELL_ROGUE_MASTER_OF_SUBTLETY_DAMAGE_PERCENT,
+ SPELL_ROGUE_MASTER_OF_SUBTLETY_PERIODIC,
+ SPELL_ROGUE_SANCTUARY,
+ SPELL_ROGUE_STEALTH_STEALTH_AURA,
+ SPELL_ROGUE_STEALTH_SHAPESHIFT_AURA
+ });
}
void HandleEffectApply(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -682,10 +675,7 @@ class spell_rog_vanish : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ROGUE_VANISH_AURA) ||
- !sSpellMgr->GetSpellInfo(SPELL_ROGUE_STEALTH_SHAPESHIFT_AURA))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ROGUE_VANISH_AURA, SPELL_ROGUE_STEALTH_SHAPESHIFT_AURA });
}
void OnLaunchTarget(SpellEffIndex effIndex)
@@ -730,9 +720,7 @@ class spell_rog_vanish_aura : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ROGUE_STEALTH))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ROGUE_STEALTH });
}
void HandleEffectRemove(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -771,11 +759,7 @@ class spell_rog_tricks_of_the_trade : public SpellScriptLoader
private:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_ROGUE_TRICKS_OF_THE_TRADE_DMG_BOOST))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_ROGUE_TRICKS_OF_THE_TRADE_PROC))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_ROGUE_TRICKS_OF_THE_TRADE_DMG_BOOST, SPELL_ROGUE_TRICKS_OF_THE_TRADE_PROC });
}
void OnRemove(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
diff --git a/src/server/scripts/Spells/spell_shaman.cpp b/src/server/scripts/Spells/spell_shaman.cpp
index 2667f83b42d..86c88b23970 100644
--- a/src/server/scripts/Spells/spell_shaman.cpp
+++ b/src/server/scripts/Spells/spell_shaman.cpp
@@ -27,9 +27,10 @@
#include "GridNotifiersImpl.h"
#include "ObjectAccessor.h"
#include "Player.h"
-#include "SpellScript.h"
-#include "SpellHistory.h"
#include "SpellAuraEffects.h"
+#include "SpellHistory.h"
+#include "SpellMgr.h"
+#include "SpellScript.h"
enum ShamanSpells
{
@@ -93,9 +94,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_ANCESTRAL_GUIDANCE_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_ANCESTRAL_GUIDANCE_HEAL });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -138,9 +137,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_ANCESTRAL_GUIDANCE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_ANCESTRAL_GUIDANCE });
}
void ResizeTargets(std::list<WorldObject*>& targets)
@@ -172,12 +169,13 @@ class spell_sha_bloodlust : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_SATED)
- || !sSpellMgr->GetSpellInfo(SPELL_HUNTER_INSANITY)
- || !sSpellMgr->GetSpellInfo(SPELL_MAGE_TEMPORAL_DISPLACEMENT)
- || !sSpellMgr->GetSpellInfo(SPELL_PET_NETHERWINDS_FATIGUED))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_SHAMAN_SATED,
+ SPELL_HUNTER_INSANITY,
+ SPELL_MAGE_TEMPORAL_DISPLACEMENT,
+ SPELL_PET_NETHERWINDS_FATIGUED
+ });
}
void RemoveInvalidTargets(std::list<WorldObject*>& targets)
@@ -219,9 +217,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_CRASH_LIGHTNING_CLEAVE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_CRASH_LIGHTNING_CLEAVE });
}
void CountTargets(std::list<WorldObject*>& targets)
@@ -264,9 +260,7 @@ class spell_sha_earth_shield : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_EARTH_SHIELD_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_EARTH_SHIELD_HEAL });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -308,11 +302,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_EARTHEN_RAGE_PERIODIC))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_EARTHEN_RAGE_DAMAGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_EARTHEN_RAGE_PERIODIC, SPELL_SHAMAN_EARTHEN_RAGE_DAMAGE });
}
void HandleEffectProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -354,11 +344,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_EARTHEN_RAGE_PASSIVE))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_EARTHEN_RAGE_DAMAGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_EARTHEN_RAGE_PASSIVE, SPELL_SHAMAN_EARTHEN_RAGE_DAMAGE });
}
void HandleEffectPeriodic(AuraEffect const* /*aurEff*/)
@@ -396,13 +382,12 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_ELEMENTAL_BLAST_CRIT))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_ELEMENTAL_BLAST_HASTE))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_ELEMENTAL_BLAST_MASTERY))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_SHAMAN_ELEMENTAL_BLAST_CRIT,
+ SPELL_SHAMAN_ELEMENTAL_BLAST_HASTE,
+ SPELL_SHAMAN_ELEMENTAL_BLAST_MASTERY
+ });
}
bool Load() override
@@ -471,9 +456,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_FLAMETONGUE_ATTACK))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_FLAMETONGUE_ATTACK });
}
void HandleEffectProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -544,12 +527,13 @@ class spell_sha_heroism : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_EXHAUSTION)
- || !sSpellMgr->GetSpellInfo(SPELL_HUNTER_INSANITY)
- || !sSpellMgr->GetSpellInfo(SPELL_MAGE_TEMPORAL_DISPLACEMENT)
- || !sSpellMgr->GetSpellInfo(SPELL_PET_NETHERWINDS_FATIGUED))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_SHAMAN_EXHAUSTION,
+ SPELL_HUNTER_INSANITY,
+ SPELL_MAGE_TEMPORAL_DISPLACEMENT,
+ SPELL_PET_NETHERWINDS_FATIGUED
+ });
}
void RemoveInvalidTargets(std::list<WorldObject*>& targets)
@@ -591,9 +575,7 @@ class spell_sha_item_lightning_shield : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_ITEM_LIGHTNING_SHIELD))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_ITEM_LIGHTNING_SHIELD });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -626,9 +608,7 @@ class spell_sha_item_lightning_shield_trigger : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_ITEM_LIGHTNING_SHIELD_DAMAGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_ITEM_LIGHTNING_SHIELD_DAMAGE });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& /*eventInfo*/)
@@ -661,9 +641,7 @@ class spell_sha_item_mana_surge : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_ITEM_MANA_SURGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_ITEM_MANA_SURGE });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -713,10 +691,7 @@ class spell_sha_item_t6_trinket : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_ENERGY_SURGE) ||
- !sSpellMgr->GetSpellInfo(SPELL_SHAMAN_POWER_SURGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_ENERGY_SURGE, SPELL_SHAMAN_POWER_SURGE });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -778,9 +753,7 @@ class spell_sha_item_t10_elemental_2p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_ELEMENTAL_MASTERY))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_ELEMENTAL_MASTERY });
}
void HandleEffectProc(AuraEffect const* aurEff, ProcEventInfo& /*eventInfo*/)
@@ -843,11 +816,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_PATH_OF_FLAMES_TALENT))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_PATH_OF_FLAMES_SPREAD))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_PATH_OF_FLAMES_TALENT, SPELL_SHAMAN_PATH_OF_FLAMES_SPREAD });
}
void HandleScript(SpellEffIndex /*effIndex*/)
@@ -882,9 +851,7 @@ class spell_sha_lava_surge : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_LAVA_SURGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_LAVA_SURGE });
}
void HandleEffectProc(AuraEffect const* /*aurEff*/, ProcEventInfo& /*eventInfo*/)
@@ -917,9 +884,7 @@ class spell_sha_lava_surge_proc : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_LAVA_BURST))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_LAVA_BURST });
}
bool Load() override
@@ -956,9 +921,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_FLAME_SHOCK_MAELSTROM))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_FLAME_SHOCK_MAELSTROM });
}
void FilterTargets(std::list<WorldObject*>& targets)
@@ -1010,9 +973,7 @@ class spell_sha_tidal_waves : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_TIDAL_WAVES))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_TIDAL_WAVES });
}
void HandleEffectProc(AuraEffect const* aurEff, ProcEventInfo& /*eventInfo*/)
@@ -1048,12 +1009,13 @@ class spell_sha_t3_6p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_TOTEMIC_POWER_ARMOR) ||
- !sSpellMgr->GetSpellInfo(SPELL_SHAMAN_TOTEMIC_POWER_ATTACK_POWER) ||
- !sSpellMgr->GetSpellInfo(SPELL_SHAMAN_TOTEMIC_POWER_SPELL_POWER) ||
- !sSpellMgr->GetSpellInfo(SPELL_SHAMAN_TOTEMIC_POWER_MP5))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_SHAMAN_TOTEMIC_POWER_ARMOR,
+ SPELL_SHAMAN_TOTEMIC_POWER_ATTACK_POWER,
+ SPELL_SHAMAN_TOTEMIC_POWER_SPELL_POWER,
+ SPELL_SHAMAN_TOTEMIC_POWER_MP5
+ });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -1114,9 +1076,7 @@ class spell_sha_t8_elemental_4p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_ELECTRIFIED))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_ELECTRIFIED });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -1163,9 +1123,7 @@ class spell_sha_t9_elemental_4p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_LAVA_BURST_BONUS_DAMAGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_LAVA_BURST_BONUS_DAMAGE });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -1257,9 +1215,7 @@ class spell_sha_t10_restoration_4p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_CHAINED_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_CHAINED_HEAL });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -1306,9 +1262,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_SHAMAN_WINDFURY_ATTACK))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_SHAMAN_WINDFURY_ATTACK });
}
void HandleEffectProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
diff --git a/src/server/scripts/Spells/spell_warlock.cpp b/src/server/scripts/Spells/spell_warlock.cpp
index 8ab6dd5be3e..a788f04ad25 100644
--- a/src/server/scripts/Spells/spell_warlock.cpp
+++ b/src/server/scripts/Spells/spell_warlock.cpp
@@ -30,6 +30,7 @@
#include "Player.h"
#include "SpellAuraEffects.h"
#include "SpellAuras.h"
+#include "SpellMgr.h"
#include "SpellScript.h"
enum WarlockSpells
@@ -139,9 +140,7 @@ class spell_warl_conflagrate : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_IMMOLATE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARLOCK_IMMOLATE });
}
// 6.x dmg formula in tooltip
@@ -175,9 +174,7 @@ class spell_warl_create_healthstone : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_CREATE_HEALTHSTONE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARLOCK_CREATE_HEALTHSTONE });
}
bool Load() override
@@ -215,9 +212,7 @@ class spell_warl_bane_of_doom : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_BANE_OF_DOOM_EFFECT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARLOCK_BANE_OF_DOOM_EFFECT });
}
bool Load() override
@@ -350,17 +345,14 @@ class spell_warl_demon_soul : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_DEMON_SOUL_IMP))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_DEMON_SOUL_FELHUNTER))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_DEMON_SOUL_FELGUARD))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_DEMON_SOUL_SUCCUBUS))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_DEMON_SOUL_VOIDWALKER))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_WARLOCK_DEMON_SOUL_IMP,
+ SPELL_WARLOCK_DEMON_SOUL_FELHUNTER,
+ SPELL_WARLOCK_DEMON_SOUL_FELGUARD,
+ SPELL_WARLOCK_DEMON_SOUL_SUCCUBUS,
+ SPELL_WARLOCK_DEMON_SOUL_VOIDWALKER
+ });
}
void OnHitTarget(SpellEffIndex /*effIndex*/)
@@ -420,9 +412,14 @@ class spell_warl_demonic_empowerment : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_DEMONIC_EMPOWERMENT_SUCCUBUS) || !sSpellMgr->GetSpellInfo(SPELL_WARLOCK_DEMONIC_EMPOWERMENT_VOIDWALKER) || !sSpellMgr->GetSpellInfo(SPELL_WARLOCK_DEMONIC_EMPOWERMENT_FELGUARD) || !sSpellMgr->GetSpellInfo(SPELL_WARLOCK_DEMONIC_EMPOWERMENT_FELHUNTER) || !sSpellMgr->GetSpellInfo(SPELL_WARLOCK_DEMONIC_EMPOWERMENT_IMP))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_WARLOCK_DEMONIC_EMPOWERMENT_SUCCUBUS,
+ SPELL_WARLOCK_DEMONIC_EMPOWERMENT_VOIDWALKER,
+ SPELL_WARLOCK_DEMONIC_EMPOWERMENT_FELGUARD,
+ SPELL_WARLOCK_DEMONIC_EMPOWERMENT_FELHUNTER,
+ SPELL_WARLOCK_DEMONIC_EMPOWERMENT_IMP
+ });
}
void HandleScriptEffect(SpellEffIndex /*effIndex*/)
@@ -484,11 +481,7 @@ class spell_warl_devour_magic : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_GLYPH_OF_DEMON_TRAINING))
- return false;
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_DEVOUR_MAGIC_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARLOCK_GLYPH_OF_DEMON_TRAINING, SPELL_WARLOCK_DEVOUR_MAGIC_HEAL });
}
void OnSuccessfulDispel(SpellEffIndex /*effIndex*/)
@@ -569,9 +562,7 @@ class spell_warl_fel_synergy : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_FEL_SYNERGY_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARLOCK_FEL_SYNERGY_HEAL });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -612,9 +603,7 @@ class spell_warl_glyph_of_shadowflame : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_GLYPH_OF_SHADOWFLAME))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARLOCK_GLYPH_OF_SHADOWFLAME });
}
void OnProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -665,9 +654,7 @@ class spell_warl_haunt : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_HAUNT_HEAL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARLOCK_HAUNT_HEAL });
}
void HandleRemove(AuraEffect const* aurEff, AuraEffectHandleModes /*mode*/)
@@ -799,10 +786,7 @@ class spell_warl_improved_soul_fire : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_IMPROVED_SOUL_FIRE_PCT) ||
- !sSpellMgr->GetSpellInfo(SPELL_WARLOCK_IMPROVED_SOUL_FIRE_STATE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARLOCK_IMPROVED_SOUL_FIRE_PCT, SPELL_WARLOCK_IMPROVED_SOUL_FIRE_STATE });
}
void OnProc(AuraEffect const* aurEff, ProcEventInfo& /*eventInfo*/)
@@ -837,11 +821,12 @@ class spell_warl_nether_ward_overrride : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_NETHER_TALENT) ||
- !sSpellMgr->GetSpellInfo(SPELL_WARLOCK_NETHER_WARD) ||
- !sSpellMgr->GetSpellInfo(SPELL_WARLOCK_SHADOW_WARD))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_WARLOCK_NETHER_TALENT,
+ SPELL_WARLOCK_NETHER_WARD,
+ SPELL_WARLOCK_SHADOW_WARD
+ });
}
void CalculateAmount(AuraEffect const* /*aurEff*/, int32& amount, bool& /*canBeRecalculated*/)
@@ -876,10 +861,7 @@ class spell_warl_seduction : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_GLYPH_OF_SUCCUBUS) ||
- !sSpellMgr->GetSpellInfo(SPELL_PRIEST_SHADOW_WORD_DEATH))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARLOCK_GLYPH_OF_SUCCUBUS, SPELL_PRIEST_SHADOW_WORD_DEATH });
}
void HandleScriptEffect(SpellEffIndex /*effIndex*/)
@@ -949,9 +931,7 @@ class spell_warl_seed_of_corruption_dummy : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_SEED_OF_CORRUPTION_DAMAGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARLOCK_SEED_OF_CORRUPTION_DAMAGE });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -1008,9 +988,7 @@ class spell_warl_seed_of_corruption_generic : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_SEED_OF_CORRUPTION_GENERIC))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARLOCK_SEED_OF_CORRUPTION_GENERIC });
}
void HandleProc(AuraEffect const* aurEff, ProcEventInfo& eventInfo)
@@ -1097,9 +1075,7 @@ class spell_warl_soul_leech : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_GEN_REPLENISHMENT))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_GEN_REPLENISHMENT });
}
void OnProc(AuraEffect const* aurEff, ProcEventInfo& /*eventInfo*/)
@@ -1131,11 +1107,12 @@ class spell_warl_soul_swap : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_GLYPH_OF_SOUL_SWAP) ||
- !sSpellMgr->GetSpellInfo(SPELL_WARLOCK_SOUL_SWAP_CD_MARKER) ||
- !sSpellMgr->GetSpellInfo(SPELL_WARLOCK_SOUL_SWAP_OVERRIDE))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_WARLOCK_GLYPH_OF_SOUL_SWAP,
+ SPELL_WARLOCK_SOUL_SWAP_CD_MARKER,
+ SPELL_WARLOCK_SOUL_SWAP_OVERRIDE
+ });
}
void HandleHit(SpellEffIndex /*effIndex*/)
@@ -1252,10 +1229,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_SOUL_SWAP_MOD_COST) ||
- !sSpellMgr->GetSpellInfo(SPELL_WARLOCK_SOUL_SWAP_OVERRIDE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARLOCK_SOUL_SWAP_MOD_COST, SPELL_WARLOCK_SOUL_SWAP_OVERRIDE });
}
SpellCastResult CheckCast()
@@ -1332,9 +1306,7 @@ class spell_warl_soulshatter : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_SOULSHATTER))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARLOCK_SOULSHATTER });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -1409,9 +1381,7 @@ class spell_warl_unstable_affliction : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARLOCK_UNSTABLE_AFFLICTION_DISPEL))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARLOCK_UNSTABLE_AFFLICTION_DISPEL });
}
void HandleDispel(DispelInfo* dispelInfo)
diff --git a/src/server/scripts/Spells/spell_warrior.cpp b/src/server/scripts/Spells/spell_warrior.cpp
index af7e805c2a5..6b626d19146 100644
--- a/src/server/scripts/Spells/spell_warrior.cpp
+++ b/src/server/scripts/Spells/spell_warrior.cpp
@@ -21,12 +21,14 @@
* Scriptnames of files in this file should be prefixed with "spell_warr_".
*/
-#include "Player.h"
#include "ScriptMgr.h"
+#include "Map.h"
#include "MoveSpline.h"
+#include "PathGenerator.h"
+#include "Player.h"
+#include "SpellAuraEffects.h"
#include "SpellHistory.h"
#include "SpellScript.h"
-#include "SpellAuraEffects.h"
enum WarriorSpells
{
@@ -322,9 +324,7 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARRIOR_HEROIC_LEAP_JUMP))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARRIOR_HEROIC_LEAP_JUMP });
}
SpellCastResult CheckElevation()
@@ -393,12 +393,13 @@ public:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARRIOR_GLYPH_OF_HEROIC_LEAP) ||
- !sSpellMgr->GetSpellInfo(SPELL_WARRIOR_GLYPH_OF_HEROIC_LEAP_BUFF) ||
- !sSpellMgr->GetSpellInfo(SPELL_WARRIOR_IMPROVED_HEROIC_LEAP) ||
- !sSpellMgr->GetSpellInfo(SPELL_WARRIOR_TAUNT))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_WARRIOR_GLYPH_OF_HEROIC_LEAP,
+ SPELL_WARRIOR_GLYPH_OF_HEROIC_LEAP_BUFF,
+ SPELL_WARRIOR_IMPROVED_HEROIC_LEAP,
+ SPELL_WARRIOR_TAUNT
+ });
}
void AfterJump(SpellEffIndex /*effIndex*/)
@@ -496,9 +497,7 @@ class spell_warr_item_t10_prot_4p_bonus : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARRIOR_STOICISM))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARRIOR_STOICISM });
}
void HandleProc(ProcEventInfo& eventInfo)
@@ -534,10 +533,7 @@ class spell_warr_lambs_to_the_slaughter : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARRIOR_MORTAL_STRIKE) ||
- !sSpellMgr->GetSpellInfo(SPELL_WARRIOR_REND))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARRIOR_MORTAL_STRIKE, SPELL_WARRIOR_REND });
}
void OnProc(AuraEffect const* /*aurEff*/, ProcEventInfo& eventInfo)
@@ -572,9 +568,7 @@ class spell_warr_last_stand : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARRIOR_LAST_STAND_TRIGGERED))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARRIOR_LAST_STAND_TRIGGERED });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -647,9 +641,7 @@ class spell_warr_rallying_cry : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARRIOR_RALLYING_CRY))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARRIOR_RALLYING_CRY });
}
bool Load() override
@@ -726,9 +718,7 @@ class spell_warr_retaliation : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARRIOR_RETALIATION_DAMAGE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARRIOR_RETALIATION_DAMAGE });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -799,9 +789,7 @@ class spell_warr_slam : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARRIOR_SLAM))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARRIOR_SLAM });
}
void HandleDummy(SpellEffIndex /*effIndex*/)
@@ -833,12 +821,13 @@ class spell_warr_second_wind_proc : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARRIOR_SECOUND_WIND_PROC_RANK_1) ||
- !sSpellMgr->GetSpellInfo(SPELL_WARRIOR_SECOUND_WIND_PROC_RANK_2) ||
- !sSpellMgr->GetSpellInfo(SPELL_WARRIOR_SECOUND_WIND_TRIGGER_RANK_1) ||
- !sSpellMgr->GetSpellInfo(SPELL_WARRIOR_SECOUND_WIND_TRIGGER_RANK_2))
- return false;
- return true;
+ return ValidateSpellInfo(
+ {
+ SPELL_WARRIOR_SECOUND_WIND_PROC_RANK_1,
+ SPELL_WARRIOR_SECOUND_WIND_PROC_RANK_2,
+ SPELL_WARRIOR_SECOUND_WIND_TRIGGER_RANK_1,
+ SPELL_WARRIOR_SECOUND_WIND_TRIGGER_RANK_2
+ });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -1004,9 +993,7 @@ class spell_warr_sudden_death : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARRIOR_COLOSSUS_SMASH))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARRIOR_COLOSSUS_SMASH });
}
void HandleApply(AuraEffect const* /*aurEff*/, AuraEffectHandleModes /*mode*/)
@@ -1047,9 +1034,7 @@ class spell_warr_sweeping_strikes : public SpellScriptLoader
private:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARRIOR_SWEEPING_STRIKES_EXTRA_ATTACK_1) || !sSpellMgr->GetSpellInfo(SPELL_WARRIOR_SWEEPING_STRIKES_EXTRA_ATTACK_2))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARRIOR_SWEEPING_STRIKES_EXTRA_ATTACK_1, SPELL_WARRIOR_SWEEPING_STRIKES_EXTRA_ATTACK_2 });
}
bool CheckProc(ProcEventInfo& eventInfo)
@@ -1105,9 +1090,7 @@ class spell_warr_sword_and_board : public SpellScriptLoader
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARRIOR_SHIELD_SLAM))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARRIOR_SHIELD_SLAM });
}
void HandleProc(AuraEffect const* /*aurEff*/, ProcEventInfo& /*eventInfo*/)
@@ -1256,9 +1239,7 @@ class spell_warr_vigilance : public SpellScriptLoader
private:
bool Validate(SpellInfo const* /*spellInfo*/) override
{
- if (!sSpellMgr->GetSpellInfo(SPELL_WARRIOR_VENGEANCE))
- return false;
- return true;
+ return ValidateSpellInfo({ SPELL_WARRIOR_VENGEANCE });
}
bool Load() override