From d8ad0c520060280fe48fb9d73e2284cad241794d Mon Sep 17 00:00:00 2001 From: Shard <30301841+Shard-MW@users.noreply.github.com> Date: Wed, 26 Aug 2020 11:44:41 +0200 Subject: [PATCH] Allow mages to refresh Living Bomb (#3228) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Currently, mages can't refresh Living Bomb if the target already have the dot. ###How to reproduce the issue : 1. Use Living Bomb on a target 2. Try to re-use Living Bomb on the same target 3. Observe the error message ###Expected Blizzlike behaviour Living Bomb should be reapplyable ###Why this PR : This PR allow to refresh the duration of Living Bomb on the same target I've didn't found a bad mage's video that refreshes Living Bomb on his target, but found some Fire Mage 3.3.5 guides that mentions : "Never refresh your LB (Living Bomb) before the explosion (except if I clearly tell you to)." https://sites.google.com/site/pvemage335/fire/rotation-gameplay "Living Bomb – every 12 seconds **after the explosion**" https://www.gnarlyguides.com/wotlk/guides/pve-fire-mage/#plink-rotation Co-authored-by: Stefano Borzì Co-authored-by: Francesco Borzì --- src/server/game/Spells/Spell.cpp | 19 ------------------- 1 file changed, 19 deletions(-) diff --git a/src/server/game/Spells/Spell.cpp b/src/server/game/Spells/Spell.cpp index 7e570404e..f02098936 100644 --- a/src/server/game/Spells/Spell.cpp +++ b/src/server/game/Spells/Spell.cpp @@ -6166,26 +6166,7 @@ SpellCastResult Spell::CheckCast(bool strict) switch (m_spellInfo->Effects[i].ApplyAuraName) { case SPELL_AURA_DUMMY: - { - //custom check - switch (m_spellInfo->Id) - { - // Living Bomb - case 44457: - case 55359: - case 55360: - { - if( Unit *target = m_targets.GetUnitTarget() ) - if( target->HasAura(m_spellInfo->Id, m_caster->GetGUID()) ) - return SPELL_FAILED_AURA_BOUNCED; - - break; - } - default: - break; - } break; - } case SPELL_AURA_MOD_POSSESS_PET: { if (m_caster->GetTypeId() != TYPEID_PLAYER)