From 0176bde314753d2b90e5930a840ae580723078d5 Mon Sep 17 00:00:00 2001
From: maximius <none@none>
Date: Tue, 17 Nov 2009 16:36:10 -0800
Subject: [PATCH] *Improved Barkskin - apply/remove armor bonus due to
 shapeshift, original by Insider, thanks to bodompelle. Closes #141

--HG--
branch : trunk
---
 src/game/SpellAuras.cpp | 24 +++++++++++++++++-------
 1 file changed, 17 insertions(+), 7 deletions(-)

diff --git a/src/game/SpellAuras.cpp b/src/game/SpellAuras.cpp
index caea0fa287..7a00ca5673 100644
--- a/src/game/SpellAuras.cpp
+++ b/src/game/SpellAuras.cpp
@@ -2099,14 +2099,21 @@ void AuraEffect::HandleShapeshiftBoosts(bool apply)
                 if (spellInfo->Stances & (1<<(form)))
                     m_target->CastSpell(m_target, itr->first, true, NULL, this);
             }
-            //LotP
+            // Leader of the Pack
             if (((Player*)m_target)->HasSpell(17007))
             {
                 SpellEntry const *spellInfo = sSpellStore.LookupEntry(24932);
                 if (spellInfo && spellInfo->Stances & (1<<(form)))
                     m_target->CastSpell(m_target, 24932, true, NULL, this);
             }
-            // HotW
+            // Improved Barkskin - apply/remove armor bonus due to shapeshift
+            if (((Player*)m_target)->HasSpell(63410) || ((Player*)m_target)->HasSpell(63411))
+            {
+                m_target->RemoveAurasDueToSpell(66530);
+                if (form == FORM_TRAVEL || form == FORM_NONE) // "while in Travel Form or while not shapeshifted"
+                    m_target->CastSpell(m_target, 66530, true);
+            }
+            // Heart of the Wild
             if (HotWSpellId)
             {
                 Unit::AuraEffectList const& mModTotalStatPct = m_target->GetAurasByType(SPELL_AURA_MOD_TOTAL_STAT_PERCENTAGE);
@@ -2189,17 +2196,20 @@ void AuraEffect::HandleShapeshiftBoosts(bool apply)
         if (spellId2)
             m_target->RemoveAurasDueToSpell(spellId2);
 
+        // Improved Barkskin - apply/remove armor bonus due to shapeshift
+        if (((Player*)m_target)->HasSpell(63410) || ((Player*)m_target)->HasSpell(63411))
+        {
+            m_target->RemoveAurasDueToSpell(66530);
+            m_target->CastSpell(m_target,66530,true);
+        }
+
         Unit::AuraMap& tAuras = m_target->GetAuras();
-        for (Unit::AuraMap::iterator itr = tAuras.begin(); itr != tAuras.end(); )
+        for (Unit::AuraMap::iterator itr = tAuras.begin(); itr != tAuras.end();)
         {
             if (itr->second->IsRemovedOnShapeLost())
-            {
                 m_target->RemoveAura(itr);
-            }
             else
-            {
                 ++itr;
-            }
         }
     }
 }