[fix] ICC, FoS, Boss Bronjahm, Magic's Bane Spell

[Toinan67]

• * What bug does the patch fix? What features does the patch add?
  Fixes the spell damage : adds half the target max mana and limits the spell dmg
  * For which repository revision was the patch created?
  10410
  * 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
  

Patch :

diff --git a/src/game/SpellEffects.cpp b/src/game/SpellEffects.cpp
index 3ad5b93..3cef9e9 100644
--- a/src/game/SpellEffects.cpp
+++ b/src/game/SpellEffects.cpp
@@ -364,6 +364,26 @@ void Spell::EffectSchoolDMG(SpellEffectIndex effect_idx)
                    case 67485:
                        damage += uint32(0.5f * m_caster->GetTotalAttackPowerValue(BASE_ATTACK));
                        break;
+                    //Magic Bane normal (Forge of Souls - Bronjahm)
+                    case 68793:
+                    {
+                        damage += uint32(unitTarget->GetMaxPower(POWER_MANA) / 2);
+
+                        if (damage > 10000)
+                            damage = 10000;
+
+                        break;
+                    }
+                    //Magic Bane heroic (Forge of Souls - Bronjahm)
+                    case 69050:
+                    {
+                        damage += uint32(unitTarget->GetMaxPower(POWER_MANA) / 2);
+
+                        if (damage > 15000)
+                            damage = 15000;
+
+                        break;
+                    }
                }
                break;
            }

Edit : could have done the same case for 2 spells with a little modif for max damage, but I prefer keep it clear

[Toinan67]

bump :)

[Ambal]

In [11146]. Thank you