[patch][7317] AURA_STATE_FROZEN for frozen(stunned) units

[Tassader2]

What bug does the patch fix? What features does the patch add?

Makes units under frost stun aura (like freezing trap, deep freeze) "frozen" so isFrozen() returns true for them (needed for shatter etc.). Same way already used for frost root auras.

For which repository revision was the patch created?

7295

Is there a thread in the bug report section or at lighthouse? If yes, please add a link to the thread.

No

Who has been writing this patch? Please include either forum user names or email addresses.

Me, Tassader

diff --git a/src/game/SpellAuras.cpp b/src/game/SpellAuras.cpp
index 5db2926..f3fb701 100644
--- a/src/game/SpellAuras.cpp
+++ b/src/game/SpellAuras.cpp
@@ -3312,6 +3312,10 @@ void Aura::HandleAuraModStun(bool apply, bool Real)
    if(!Real)
        return;

+    // Frost stun aura -> freeze/unfreeze target
+    if (GetSpellSchoolMask(m_spellProto) & SPELL_SCHOOL_MASK_FROST)
+        m_target->ModifyAuraState(AURA_STATE_FROZEN, apply);
+
    if (apply)
    {
        m_target->addUnitState(UNIT_STAT_STUNNED);

[Tassader2]

updated for [7295]

[Sephiroth1983]

Thanks, this patch is needed, I was almost going to report about Frost trap targets not considered as Frozen, but you beat me to it

[ApoC]

Applied in [7317]

Thank You.

[VladimirMangos]

What about stacked auars at target and remove only one from? For example root and stun...

remove one from its will remove frozen state.

[Tassader2]

What about stacked auars at target and remove only one from? For example root and stun...

remove one from its will remove frozen state.

Hm then the implementation was incorrect even before I copied it from root handle ... because it is possible to have two different frost root aras (at least on mangos, maybe not on official)

[VladimirMangos]

In [7321] frozen aura state will remove only if no other frozen auuras at raget.

Is or not some auras stacked nother question.

[Tassader2]

thanks! ......