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

1 files changed, 8 insertions, 0 deletions

diff --git a/sql/updates/world/3.3.5/2018_01_21_00_world.sql b/sql/updates/world/3.3.5/2018_01_21_00_world.sql
new file mode 100644
index 00000000000..fe27557969e
--- /dev/null
+++ b/sql/updates/world/3.3.5/2018_01_21_00_world.sql
@@ -0,0 +1,8 @@
+DELETE FROM `spell_bonus_data` WHERE `entry`=48210;
+INSERT INTO `spell_bonus_data` (`entry`, `direct_bonus`, `dot_bonus`, `ap_bonus`, `ap_dot_bonus`, `comments`) VALUES
+(48210, 0, 0, 0, 0, 'Warlock - Haunt heal');
+
+DELETE FROM `spell_script_names` WHERE `ScriptName`='spell_gen_50pct_count_pct_from_max_hp' AND `spell_id`=48292;
+DELETE FROM `spell_script_names` WHERE `ScriptName`='spell_dark_slash';
+INSERT INTO `spell_script_names` (`spell_id`, `ScriptName`) VALUES
+(48292, 'spell_dark_slash');