| Age | Commit message (Collapse) | Author |
|
|
|
passive spell casts"
This reverts commit f188ff1ce47905d7a81dff987a66b3d72152ae7c.
Was breaking Titan's Grip.
Spells are loaded before items, and they CAST the spell on the player.
When items are then loaded, Player::ApplyItemDependentAuras only has AddAura on it. Titan's Grip is an effect, no auras involved.
In turn reverting this commit will now allow Two-handed weapon specialization and similar from spells load. Which is the lesser of two evils.
|
|
|
|
|
|
casts
- Was adding auras on login that didn't satisfy requirements (like Two-Handed Weapon Specialization)
|
|
SPELL_AURA_MOD_WEAPON_CRIT_PERCENT, SPELL_AURA_MOD_DAMAGE_DONE and SPELL_AURA_MOD_DAMAGE_PERCENT_DONE auras
* Now the entire aura is removed when changing equipment
* All aura types can now depend on equipped items
(cherry picked from commit b9f7b500a362736235fc917ce4f34aa8520eb651)
|
|
EquippedItemInventoryTypeMask should only be used for enchantments
(cherry picked from commit 28fd8a424d828f36a2b05ed016b12c4b8cf2607c)
|
|
in a transaction
|
|
|
|
3f19eea5e41fd47b571c17529e08d9f8caa91705 follow-up. Without the #16658 regression
|
|
Closes #12830
|
|
- Fixes Leader of the Pack stuck when switching specs while shapeshifted
|
|
reapply
|
|
- Traps should select ATTACKABLE not UNFRIENDLY targets (it shouldn't attack unfriendly targets on sanctuary zones for example)
- Made more readable the nasty oneliner in Unit::_IsValidAttackTarget
Closes #6464
Closes #18271
|
|
2e457b3be418ce75b0e0ca901e84ebcdaf02ad18 followup
Closes #18275
|
|
- Use Spell::m_triggeredByAuraSpell and compare against trigger aura, it requires scripts to set triggeredByAura parameter.
- Fixed existing scripts lacking it
DB/Spell: Anger Capacitor (Tiny Abomination in a Jar) proc
Closes #18269
|
|
for quest credit
- Also sends proper SMSG_ITEM_PUSH_RESULT packet
Closes #3505
|
|
lower level enemies
|
|
using attributes
- Renamed IsProcTriggeredOnEvent to GetProcEffectMask, since long ago this function returned bool, new name should better reflect its use
|
|
|
|
Closes #18256
|
|
- Applied twice for some dots
- Missing for Health Leech Periodic
- Missing in a few scripts
Closes #17463
|
|
more than once
No more overpowered dps for you
Closes #16755
|
|
Ctrl-S failure
|
|
(9f5df023b746d324588f175e264c62205b69e165 follow up)
- Handling checked in sniffs: Spell 63710 Void Barrier vs 49143 Frost Strike
* Send spell miss immune only if spell consists of damage effects
- Checked with 348 Immolate:
* No packets sent if damage immune, aura is applied normally
Also... who the fuck uses 0 to compare against pointers
|
|
|
|
spell. Should be used only for damage
|
|
spellcasts while charmed
|
|
Closes #18248
|
|
- SPELL_ATTR1_UNAFFECTED_BY_SCHOOL_IMMUNE
- SPELL_ATTR2_UNAFFECTED_BY_AURA_SCHOOL_IMMUNE
- SPELL_ATTR3_IGNORE_HIT_RESULT
|
|
|
|
|
|
Fixes an issue with Skadi Harpoons.
Note to self: should actually implement GameObject casting already :/
|
|
Closes #8403
|
|
Fixes Master's call when pet is rooted
|
|
Defensive Stance
|
|
Was breaking functionality (eg: stacking scroll auras)
|
|
wasn't set for the current effect.
Closes #15388
|
|
rewrite too)
- Warlord's Presence, Uncontrollable Frenzy, Free Friend
- Overload, Lightning Tendrils
- Fervor, Berserk
- Determination
|
|
for the purposes of spell/aura targets
Ref #18233
|
|
followup of 50a3ce5703bdaa4fe16e2e773eabafa9a4818083
|
|
and hooks
|
|
|
|
- Removed dead code, outdated comments
- Fixed indentation
- Explicitly delete copy ctors
- const'd the operator() definitions
|
|
* Core/Scripts: Cleanup in instance_black_temple and added missing datas
* Fixed door timing issue
* Remove redundant call
|
|
Closes #17868
(cherry picked from commit f94e6cee38241e10daed711247a18b8706cb8924)
|
|
use typedef for constant reference
|
|
|
|
- Spell attributes research
- Container optimization
- Do not skip damage immunity checks on SPELL_ATTR0_UNAFFECTED_BY_INVULNERABILITY present
- Added additonal arguments support to SendCastResult / CheckCast, spells can now send proper error messages
- Loads spell immune masks/states/effects at startup.
- This cached info is used whenever a spell cast requires immunities to be known beforehand (to see if an aura could be applied for example)
Closes #16658
Closes #16901
Closes #18114
|
|
- Renamed CheckEffectExecuteData to AssertEffectExecuteData and made const, better reflects its purpose
- Added missing attribute check to IsNextMeleeSwingSpell
- Reworked SPELL_ATTR4_CAST_ONLY_IN_OUTLAND attr check (researched behavior)
|