diff options
| author | Shauren <shauren.trinity@gmail.com> | 2014-10-21 19:23:32 +0200 |
|---|---|---|
| committer | Shauren <shauren.trinity@gmail.com> | 2025-09-28 21:38:32 +0200 |
| commit | d144724e2fd8a74a3e896751dc9c866b3932f815 (patch) | |
| tree | e1f3c155560705bbd0f7dea15351b14602354454 /src/server/game/Spells/Spell.cpp | |
| parent | 3059fcd70a01bccdf8ea6dd1ae7bedefd737d558 (diff) | |
Core/Entities: First batch of removing implicit conversions of ObjectGuid to uint64
(cherry picked from commit 9cc7044546eaaaf4fd7a999c5e074ad0ea3d47ef)
Diffstat (limited to 'src/server/game/Spells/Spell.cpp')
| -rw-r--r-- | src/server/game/Spells/Spell.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/server/game/Spells/Spell.cpp b/src/server/game/Spells/Spell.cpp index b2660e1449d..a76bc21d886 100644 --- a/src/server/game/Spells/Spell.cpp +++ b/src/server/game/Spells/Spell.cpp @@ -458,7 +458,7 @@ void SpellCastTargets::Update(WorldObject* caster) } // update positions by transport move - if (HasSrc() && m_src._transportGUID) + if (HasSrc() && !m_src._transportGUID.IsEmpty()) { if (WorldObject* transport = ObjectAccessor::GetWorldObject(*caster, m_src._transportGUID)) { @@ -467,7 +467,7 @@ void SpellCastTargets::Update(WorldObject* caster) } } - if (HasDst() && m_dst._transportGUID) + if (HasDst() && !m_dst._transportGUID.IsEmpty()) { if (WorldObject* transport = ObjectAccessor::GetWorldObject(*caster, m_dst._transportGUID)) { @@ -1732,7 +1732,7 @@ void Spell::SelectEffectTypeImplicitTargets(SpellEffectInfo const& spellEffectIn { case SPELL_EFFECT_SUMMON_RAF_FRIEND: case SPELL_EFFECT_SUMMON_PLAYER: - if (m_caster->GetTypeId() == TYPEID_PLAYER && m_caster->ToPlayer()->GetTarget()) + if (m_caster->GetTypeId() == TYPEID_PLAYER && !m_caster->ToPlayer()->GetTarget().IsEmpty()) { WorldObject* target = ObjectAccessor::FindPlayer(m_caster->ToPlayer()->GetTarget()); CallScriptObjectTargetSelectHandlers(target, spellEffectInfo.EffectIndex, SpellImplicitTargetInfo()); @@ -3325,7 +3325,7 @@ void Spell::_cast(bool skipCheck) } // cancel at lost explicit target during cast - if (m_targets.GetObjectTargetGUID() && !m_targets.GetObjectTarget()) + if (!m_targets.GetObjectTargetGUID().IsEmpty() && !m_targets.GetObjectTarget()) { cancel(); return; @@ -3816,7 +3816,7 @@ void Spell::update(uint32 difftime) return; } - if (m_targets.GetUnitTargetGUID() && !m_targets.GetUnitTarget()) + if (!m_targets.GetUnitTargetGUID().IsEmpty() && !m_targets.GetUnitTarget()) { TC_LOG_DEBUG("spells", "Spell {} is cancelled due to removal of target.", m_spellInfo->Id); cancel(); @@ -7332,10 +7332,10 @@ bool Spell::UpdatePointers() m_originalCaster = nullptr; } - if (m_focusObjectGUID) + if (!m_focusObjectGUID.IsEmpty()) focusObject = ObjectAccessor::GetGameObject(*m_caster, m_focusObjectGUID); - if (m_castItemGUID && m_caster->GetTypeId() == TYPEID_PLAYER) + if (!m_castItemGUID.IsEmpty() && m_caster->GetTypeId() == TYPEID_PLAYER) { m_CastItem = m_caster->ToPlayer()->GetItemByGuid(m_castItemGUID); // cast item not found, somehow the item is no longer where we expected |