public/azerothcore-wotlk
1
0
Fork 0
mirror of https://github.com/liyunfan1223/azerothcore-wotlk.git synced 2026-08-04 13:57:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(Core/Movement): let melee pets attack targets whose chase point is unpathable (#26726)

Browse Source
This commit is contained in:
Andrew
2026-07-22 09:07:02 -03:00
committed by GitHub
parent 3e741cc44f
commit 007f866cbf
1 changed files with 33 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

41
src/server/game/Movement/MovementGenerators/TargetedMovementGenerator.cpp
View File

@@ -104,15 +104,40 @@ bool ChaseMovementGenerator<T>::DispatchSplineToPosition(T* owner, float x, floa
if (owner->IsHovering())
owner->UpdateAllowedPositionZ(x, y, z);
bool success = i_path->CalculatePath(x, y, z, forceDest);
uint32 pathType = i_path->GetPathType();
bool pathFailed = !success || (pathType & PATHFIND_NOPATH);
auto isPathUsable = [&]()
{
uint32 pathType = i_path->GetPathType();
if (pathType & PATHFIND_NOPATH)
return false;
// For pets, treat incomplete paths as failures to avoid clipping through geometry
// Players and Player-controlled units have more erratic movement, skip failure
if (cOwner && (cOwner->IsPet() || cOwner->IsControlledByPlayer()) && !GetTarget()->IsCharmedOwnedByPlayerOrPlayer())
if (pathType & PATHFIND_INCOMPLETE)
pathFailed = true;
// For pets, treat incomplete paths as failures to avoid clipping through geometry
// Players and Player-controlled units have more erratic movement, skip failure
if (cOwner && (cOwner->IsPet() || cOwner->IsControlledByPlayer())
&& !GetTarget()->IsCharmedOwnedByPlayerOrPlayer())
if (pathType & PATHFIND_INCOMPLETE)
return false;
return true;
};
bool pathFailed = !i_path->CalculatePath(x, y, z, forceDest) || !isPathUsable();
// Targets with an oversized combat reach can stand entirely over unwalkable space
// (e.g. Kologarn) so pathing to their center or to an angled near point (pets chase
// to behind the target, which may hang over the void) fails even though the melee
// ring covers the navmesh. Retry against the nearest point on the ring, ignoring the
// chase angle, via GetNearPoint2D: GetNearPoint's LoS repositioning must be avoided
// here, it can rotate the point to the far side of the target.
if (pathFailed && (!_range || _range->MaxRange <= CONTACT_DISTANCE)
&& !GetTarget()->IsCharmedOwnedByPlayerOrPlayer() && GetTarget()->GetCombatReach() > NOMINAL_MELEE_RANGE)
{
GetTarget()->GetNearPoint2D(owner, x, y, 0.0f, GetTarget()->GetAngle(owner));
z = GetTarget()->GetPositionZ();
owner->UpdateAllowedPositionZ(x, y, z);
pathFailed = !i_path->CalculatePath(x, y, z, forceDest) || !isPathUsable();
// the destination already lies on the melee ring, nothing to cut
cutPath = false;
}
if (pathFailed)
{