Core/Threat: Assistance threat (healing, buffs, etc.) is now properly split (instead of copied) between all creatures threatened by the action. Thanks to Foereaper for help with testing, and my apologies to healers everywhere.

Port From (https://github.com/TrinityCore/TrinityCore/commit/73bf0e3a0b1f93d8dab719194b4aa5bf746cb969)
2021-06-23 13:31:01 -04:00
parent 9631f569fa
commit f7e09403ec
1 changed files with 6 additions and 1 deletions
@@ -549,8 +549,13 @@ namespace Game.Combat
            if (spell != null && spell.HasAttribute(SpellAttr1.NoThreat)) // shortcut, none of the calls would do anything
                return;

+            if (_threatenedByMe.Empty())
+                return;
+
+            float perTarget = baseAmount / _threatenedByMe.Count; // Threat is divided evenly among all targets (LibThreat sourced)
+
            foreach (var pair in _threatenedByMe)
-                pair.Value.GetOwner().GetThreatManager().AddThreat(assistant, baseAmount, spell, ignoreModifiers);
+                pair.Value.GetOwner().GetThreatManager().AddThreat(assistant, perTarget, spell, ignoreModifiers);
        }

        public void RemoveMeFromThreatLists()