From 9f5d1e2b10013e5fecf35fdd5af70921c96d07d1 Mon Sep 17 00:00:00 2001 From: ariel- Date: Sat, 20 Jan 2018 21:00:53 -0300 Subject: Core/Scripts: fix wrong uses of SetHitDamage hook. This hook modifies damage AFTER it has been reduced by target auras/armor/resistances etc, it's useful if you want to scale damage by a factor, but not to add flat bonuses. We're fixing those by moving calculation to Launch phase, where target taken bonuses haven't been used yet. - Bronjahm: Magic's Bane - BPC: Shadow Prison - Oculus: Shock Lance - Ymiron: Dark Slash (extra fix, it was wrongly damaging half of total health, it's supposed to be half of CURRENT health!) - DK: Raise Ally Thrash spell (also extra fix: corrected formula) - Warrior: Bloodthirst (shouldn't matter much as it's damage class none and those don't get bonuses by default) - Warrior: Concussion Blow - Warlock: extra fix for Haunt, healing part shouldn't scale with spell power Closes #9560 --- .../Outland/TempestKeep/Mechanar/boss_mechano_lord_capacitus.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/server/scripts/Outland/TempestKeep') diff --git a/src/server/scripts/Outland/TempestKeep/Mechanar/boss_mechano_lord_capacitus.cpp b/src/server/scripts/Outland/TempestKeep/Mechanar/boss_mechano_lord_capacitus.cpp index cbf71d951df..65233e36cd7 100644 --- a/src/server/scripts/Outland/TempestKeep/Mechanar/boss_mechano_lord_capacitus.cpp +++ b/src/server/scripts/Outland/TempestKeep/Mechanar/boss_mechano_lord_capacitus.cpp @@ -210,7 +210,7 @@ class spell_capacitus_polarity_charge : public SpellScriptLoader Unit* target = GetHitUnit(); if (target->HasAura(GetTriggeringSpell()->Id)) - SetHitDamage(0); + PreventHitDamage(); } void Register() override -- cgit v1.2.3