nchebrov/azerothcore-wotlk
1
0
Fork 0
mirror of https://github.com/mod-playerbots/azerothcore-wotlk.git synced 2026-01-17 19:05:42 +00:00
Code Issues Packages Projects Releases Wiki Activity

Allow mages to refresh Living Bomb (#3228)

Browse Source 
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ì <stefanoborzi32@gmail.com>
Co-authored-by: Francesco Borzì <borzifrancesco@gmail.com>
This commit is contained in:
Shard
2020-08-26 11:44:41 +02:00
committed by GitHub
parent 4db1164dcd
commit d8ad0c5200
1 changed files with 0 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
src/server/game/Spells/Spell.cpp
View File

@@ -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)