[patch] Will of the forsaken update

[Auntie Mangos]

i tried the AddTriggeredSpell method before and didn't work, spells don't trigger item cds and viceversa.

http://getmangos.eu/community/showthread.php?13504-Will-of-the-Forsaken&highlight=will+of+the+forsaken

[Auntie Mangos]

i tried the AddTriggeredSpell method before and didn't work, spells don't trigger item cds and viceversa.

http://getmangos.eu/community/showthread.php?13504-Will-of-the-Forsaken&highlight=will+of+the+forsaken

i can't reproduce your problem with YTDB r547 and mangos 9953.

I'm trying to add .additem 40476

equipping it

.cast 72757

and see...

cooldown is applied as should

Maybe your bug is just because of Precast/Trigger spells system? I didn't tried my patch yet

[The_Game_Master]

Revision: 9968

Author: Me

Descripton: Since patch 3.3.0 Will of the Forsaken now adds a global cooldown to other movement impairing effects (Eg.: Insignia of the Horde)

Source: http://www.wowwiki.com/Patch_3.3.0 (scrool down to Races:General)

diff --git a/src/game/SpellAuras.cpp b/src/game/SpellAuras.cpp
index f13857f..0b345c9 100644
--- a/src/game/SpellAuras.cpp
+++ b/src/game/SpellAuras.cpp
@@ -4502,7 +4502,7 @@ void Aura::HandleModMechanicImmunity(bool apply, bool /*Real*/)
        uint32 mechanic = 1 << (misc-1);

        //immune movement impairment and loss of control
-        if(GetId()==42292 || GetId()==59752)
+        if(GetId()==7744 || GetId()==42292 || GetId()==59752)
            mechanic=IMMUNE_TO_MOVEMENT_IMPAIRMENT_AND_LOSS_CONTROL_MASK;

        m_target->RemoveAurasAtMechanicImmunity(mechanic,GetId());

[KAPATEJIb]

Revision: 9968

Author: Me

Descripton: Since patch 3.3.0 Will of the Forsaken now adds a global cooldown to other movement impairing effects (Eg.: Insignia of the Horde)

Source: http://www.wowwiki.com/Patch_3.3.0 (scrool down to Races:General)

diff --git a/src/game/SpellAuras.cpp b/src/game/SpellAuras.cpp
index f13857f..0b345c9 100644
--- a/src/game/SpellAuras.cpp
+++ b/src/game/SpellAuras.cpp
@@ -4502,7 +4502,7 @@ void Aura::HandleModMechanicImmunity(bool apply, bool /*Real*/)
        uint32 mechanic = 1 << (misc-1);

        //immune movement impairment and loss of control
-        if(GetId()==42292 || GetId()==59752)
+        if(GetId()==7744 || GetId()==42292 || GetId()==59752)
            mechanic=IMMUNE_TO_MOVEMENT_IMPAIRMENT_AND_LOSS_CONTROL_MASK;

        m_target->RemoveAurasAtMechanicImmunity(mechanic,GetId());

did you test your patch? It think this can make Will of the Forsaken to remove another CC effects that shouldn't be removed by this spell - it will start to work like insignia

because you set

mechanic=IMMUNE_TO_MOVEMENT_IMPAIRMENT_AND_LOSS_CONTROL_MASK;

then remove auras with this mechanic

m_target->RemoveAurasAtMechanicImmunity(mechanic,GetId());

Here MY version

diff --git a/src/game/Spell.cpp b/src/game/Spell.cpp
index e8b4529..b205b12 100644
--- a/src/game/Spell.cpp
+++ b/src/game/Spell.cpp
@@ -2736,10 +2736,14 @@ void Spell::cast(bool skipCheck)
        {
            if (m_spellInfo->Mechanic == MECHANIC_BANDAGE)  // Bandages
                AddPrecastSpell(11196);                     // Recently Bandaged
+            else if(m_spellInfo->Id == 7744)
+                AddTriggeredSpell(72757);
            else if(m_spellInfo->Id == 20594)               // Stoneskin
               AddTriggeredSpell(65116);                   // Stoneskin - armor 10% for 8 sec
            else if(m_spellInfo->Id == 71904)               // Chaos Bane strength buff
                AddTriggeredSpell(73422);
+            else if(m_spellInfo->Id == 42292)
+                AddTriggeredSpell(72752);
            break;
        }
        case SPELLFAMILY_MAGE:

[The_Game_Master]

Yes, you're right. But must be a better way to add this part.

           else if(m_spellInfo->Id == 42292)                
               AddTriggeredSpell(72752);