diff options
| author | Treeston <treeston.mmoc@gmail.com> | 2018-02-15 12:23:05 +0100 |
|---|---|---|
| committer | Shauren <shauren.trinity@gmail.com> | 2021-06-21 00:07:24 +0200 |
| commit | f982f3019a1a12bdd9d85e59faf5ddb53c10761f (patch) | |
| tree | 97221768b19bcc65a0290823255aaab167f77ddc /src/server/game | |
| parent | 37298ca99cce901bd358c536b34aee879f72f900 (diff) | |
Core/Combat: Unify combat permitted checks between CvC and PvC - only require "not friendly" in all cases. Fixes Yogg-Saron encounter. Closes #21380.
Also some bonus cleanup to Yogg's script to make transition cleaner.
(cherry picked from commit 4da9321d2bdde17ddf1a55b92ab130216e9af27b)
Diffstat (limited to 'src/server/game')
| -rw-r--r-- | src/server/game/Combat/CombatManager.cpp | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/src/server/game/Combat/CombatManager.cpp b/src/server/game/Combat/CombatManager.cpp index 7630e3c1d2d..fbcad98cb50 100644 --- a/src/server/game/Combat/CombatManager.cpp +++ b/src/server/game/Combat/CombatManager.cpp @@ -43,16 +43,8 @@ return false; if (a->HasUnitState(UNIT_STATE_IN_FLIGHT) || b->HasUnitState(UNIT_STATE_IN_FLIGHT)) return false; - if (a->IsControlledByPlayer() || b->IsControlledByPlayer()) - { // PvSomething, only block friendly fire - if (a->IsFriendlyTo(b) || b->IsFriendlyTo(a)) - return false; - } - else - { // CvC, need hostile reaction to start a fight - if (!a->IsHostileTo(b) && !b->IsHostileTo(a)) - return false; - } + if (a->IsFriendlyTo(b) || b->IsFriendlyTo(a)) + return false; Player const* playerA = a->GetCharmerOrOwnerPlayerOrPlayerItself(); Player const* playerB = b->GetCharmerOrOwnerPlayerOrPlayerItself(); // ...neither of the two units must be (owned by) a player with .gm on |