From 4da12422ebf1e212fe0feb37190d4c6a77d50d91 Mon Sep 17 00:00:00 2001 From: jackpoz Date: Sun, 28 Apr 2019 14:50:33 +0200 Subject: [PATCH] Core/MMaps: Revert PATHFIND_INCOMPLETE handling change in ChaseMovementGenerator in 386b5a6fd948b4866cbd9a8a9892e8777922e9c9 since the current hp reset code in Creature::Update() will reset the hp of bosses as soon as CanNotReachTarget() is true. In case of boss mechanics like Abub'Rekhan at Naxx where the boss throws the player in the air, a PATHFIND_INCOMPLETE path is returned. The code in Creature::Update() should be modified to wait a few seconds before resetting the hp. PATHFIND_NOPATH / PATHFIND_INCOMPLETE don't really mean that the Creature can't attack the player anyway either. # Conflicts: # src/server/game/Movement/MovementGenerators/ChaseMovementGenerator.cpp --- .../Movement/MovementGenerators/ChaseMovementGenerator .cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/server/game/Movement/MovementGenerators/ChaseMovementGenerator .cpp b/src/server/game/Movement/MovementGenerators/ChaseMovementGenerator .cpp index 9426dac2876..83f68de3bf8 100644 --- a/src/server/game/Movement/MovementGenerators/ChaseMovementGenerator .cpp +++ b/src/server/game/Movement/MovementGenerators/ChaseMovementGenerator .cpp @@ -166,7 +166,7 @@ bool ChaseMovementGenerator::Update(Unit* owner, uint32 diff) owner->UpdateAllowedPositionZ(x, y, z); bool success = _path->CalculatePath(x, y, z, owner->CanFly()); - if (!success || (_path->GetPathType() & (PATHFIND_NOPATH | PATHFIND_INCOMPLETE))) + if (!success || (_path->GetPathType() & (PATHFIND_NOPATH /* | PATHFIND_INCOMPLETE*/))) { if (cOwner) cOwner->SetCannotReachTarget(true);