[3.3.5] Core/Spells: Health Funnel removes more life than it should

[Keader]

Current script remove warlock health in 2 points.
Here: https://github.com/TrinityCore/TrinityCore/blob/3.3.5/src/server/game/Spells/Auras/SpellAuraEffects.cpp#L5421
And Here: https://github.com/TrinityCore/TrinityCore/blob/3.3.5/src/server/game/Spells/Auras/SpellAuras.cpp#L706
The second point, is called 2 times: First call is caused by Warlock Aura. The second is caused by Pet Aura.
And current script, dont check any SPELLMOD_COST, so reduce cost part of Improved Health Funnel not work.

PS: If you try fix it, please read #20559 (comment)

:)

More info here: #20559

Branch(es): 3.3.5

TC rev. hash/commit: 527ef19

[ariel-]

Needs #21395 first (because of two aura objects applied, one on pet, other on warlock)

[CraftedRO]

8ad4314 not really sure if still valid but it can kill you and I doubt it shoud.