* NULL to nullptr
* NULL to nullptr
* NULL to nullptr
* NULL to nullptr
* NULL to nullptr
Co-authored-by: Francesco Borzì <borzifrancesco@gmail.com>
Co-authored-by: Stefano Borzì <stefanoborzi32@gmail.com>
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>
* Implement SPELL_ATTR0_CU_POSITIVE_EFF0
* Implement SPELL_ATTR0_CU_POSITIVE_EFF1
* Implement SPELL_ATTR0_CU_POSITIVE_EFF2
* Move SpellFamilyName spells into DB to define if they are POSITIVE or NEGATIVE spells
* Rename column from entry to spell_id to make more sense
* fix(Core/Spell): Flame Breath (Catapult)
- Will fix issue with Wintergrasp Catapult Siege spell (Flame Breath).
- The SQL is added just in case, if someone already tried temp fix applying another spell.
https://github.com/azerothcore/azerothcore-wotlk/issues/1978#issuecomment-507880360
- Done, updated. Wintergrasp Demolisher - Spell Ram is fixed also
* Fix treating numeric values as string
* Reworked Archimonde
* missing spell scripts on db
* Travis?
* Thanks Nefertum
* Update rev_1554524766481187700.sql
* compact sql
* Update data/sql/updates/pending_db_world/rev_1554524766481187700.sql
Co-Authored-By: Pondaveia <43385840+Pondaveia@users.noreply.github.com>
* Update rev_1554524766481187700.sql
* Update boss_archimonde.cpp
* Resets
* Update boss_archimonde.cpp
* Update boss_archimonde.cpp
* Finger of death check improvement
Deku suggested adding this check instead:
- me->IsWithinMeleeRange(unit)
* fix: remove unneeded if statement
* Possible targets list bug
Added a list for both Finger of death and Spell protections specifically. Also cleared the lists when needed to avoid possible bugs.
* comment typo
* Last touches
* Copy paste fail?
Removed waypoint reached function. No idea how it got there!
* Update boss_archimonde.cpp
* The meele check was too small
This check allows the tank to move the boss without getting snipped by finger of death
* Update boss_archimonde.cpp
* Initializing summonlist on constructor
* Update hyjal.h
* Update hyjal.h
* Changed Archimonde AI
Hyjal_trash AI to BossAI
* Removing hyjal_trashAI function
* Somehow the commit above didn't remove
* unused variable
* bool initialization on the constructor
bool Enraged;
bool BelowTenPercent;
bool HasProtected;
bool IsChanneling;
* HOLY SHIT TRAVIS
