diff options
| author | Treeston <treeston.mmoc@gmail.com> | 2017-05-29 16:39:20 +0200 |
|---|---|---|
| committer | funjoker <funjoker109@gmail.com> | 2020-06-14 23:49:04 +0200 |
| commit | aa811d57b6dfd21689b6aaac3502c03528533222 (patch) | |
| tree | 74411c052f59ccd3c6f58babc282f37c02bf0683 /src/server/game/Scripting/ScriptMgr.cpp | |
| parent | e782799c30367cfb0be4433395cf96b052268551 (diff) | |
Partial revert of 15a207f, which was causing issues (ref #4943 and #19768). Instead, fix the underlying issue from 15a207f one level further down - Unit::getAttackerForHelper() shouldn't return units that we aren't in combat with (victim can be such a unit for players/player pets, which can startattack from a distance without entering combat). (#19814)
Fixes the following issues:
- Player pets would aggro neutral mobs as soon as they start autocasting (Imp's Firebolt) if they're in react range (due to victim != null, autocast counts as autoattack and sets victim)
- Neutral mobs would randomly evade when aggro switched between targets.
Closes #19768, #19485 and #10921.
(from PR #19814)
(cherry picked from commit 69fd6245dc41dca658cadb4e99c1ee6593fe000b)
Diffstat (limited to 'src/server/game/Scripting/ScriptMgr.cpp')
0 files changed, 0 insertions, 0 deletions