diff options
| author | Shocker <none@none> | 2010-12-22 20:18:34 +0200 |
|---|---|---|
| committer | Shocker <none@none> | 2010-12-22 20:18:34 +0200 |
| commit | 446c30050583bf1605ca6b402020fb68fcdc85d4 (patch) | |
| tree | e34c867443d16776400fe2096dc2d65aad97818c /src | |
| parent | 42304c0b257ca8a6e5a374d8c8719fea84cb8105 (diff) | |
Core/Spells: Small cleanup in SpellEffects, spells always have caster
--HG--
branch : trunk
Diffstat (limited to 'src')
| -rwxr-xr-x | src/server/game/Spells/SpellEffects.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/server/game/Spells/SpellEffects.cpp b/src/server/game/Spells/SpellEffects.cpp index de4c91af9b4..d2e079e9ef1 100755 --- a/src/server/game/Spells/SpellEffects.cpp +++ b/src/server/game/Spells/SpellEffects.cpp @@ -2690,7 +2690,7 @@ void Spell::SendLoot(uint64 guid, LootType loottype) void Spell::EffectOpenLock(SpellEffIndex effIndex) { - if (!m_caster || m_caster->GetTypeId() != TYPEID_PLAYER) + if (m_caster->GetTypeId() != TYPEID_PLAYER) { sLog.outDebug("WORLD: Open Lock - No Player Caster!"); return; @@ -4713,7 +4713,7 @@ void Spell::EffectScriptEffect(SpellEffIndex effIndex) // Retribution by Sevenfold Retribution case 51854: { - if (!m_caster || !unitTarget) + if (!unitTarget) return; if (unitTarget->HasAura(51845)) unitTarget->CastSpell(m_caster, 51856, true); @@ -5386,7 +5386,7 @@ void Spell::EffectAddComboPoints(SpellEffIndex /*effIndex*/) void Spell::EffectDuel(SpellEffIndex effIndex) { - if (!m_caster || !unitTarget || m_caster->GetTypeId() != TYPEID_PLAYER || unitTarget->GetTypeId() != TYPEID_PLAYER) + if (!unitTarget || m_caster->GetTypeId() != TYPEID_PLAYER || unitTarget->GetTypeId() != TYPEID_PLAYER) return; Player *caster = (Player*)m_caster; @@ -5965,7 +5965,7 @@ void Spell::EffectSkinning(SpellEffIndex /*effIndex*/) { if (unitTarget->GetTypeId() != TYPEID_UNIT) return; - if (!m_caster || m_caster->GetTypeId() != TYPEID_PLAYER) + if (m_caster->GetTypeId() != TYPEID_PLAYER) return; Creature* creature = unitTarget->ToCreature(); |