Jump to content

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


Guest Toinan67

Recommended Posts

  • * 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

Link to comment
Share on other sites

  • 5 months later...
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. Privacy Policy Terms of Use