veitR/mod-playerbots
1
0
Fork 0
mirror of https://github.com/liyunfan1223/mod-playerbots.git synced 2026-06-20 15:39:25 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix(Core/RPG): Per-tick re-resolve travel plan instead of advancing cached plan

Browse Source
This commit is contained in:
bash 2026-05-30 18:38:22 +02:00
parent 7ac501adb0
commit 905f550ca1
1 changed files with 12 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
src/Ai/World/Rpg/Action/NewRpgBaseAction.cpp
View File

@ -146,20 +146,20 @@ bool NewRpgBaseAction::MoveFarTo(WorldPosition dest)
((bot->GetMapId() != dest.GetMapId()) ||
(dis > sPlayerbotAIConfig.sightDistance));
// Ride the active node plan only if its dest still matches.
// A stale plan would steer the bot past a new target.
if (tryNodes && botAI->rpgInfo.HasActiveTravelPlan())
{
if (botAI->rpgInfo.travelPlan.destination.distance(dest) > 10.0f)
botAI->rpgInfo.ClearTravel();
else
return UpdateTravelPlan();
}
// PRIORITY: try the travel-node graph FIRST when the move is
// long enough to need it.
// Per-tick re-resolve (cmangos pattern). Rebuild the travel plan
// from the bot's CURRENT position every tick rather than caching
// a multi-step plan and advancing through it. Recovers naturally
// from knockback, off-route drift, mid-execution destination
// changes, and blocked waypoints. Cost: per-tick GetFullPath call;
// the lastPath cache (10% reuse block above) handles the common
// case where the cached path still ends near the same destination
// and avoids re-derivation.
if (tryNodes)
{
if (botAI->rpgInfo.HasActiveTravelPlan() &&
botAI->rpgInfo.travelPlan.destination.distance(dest) > 10.0f)
botAI->rpgInfo.ClearTravel();
StartTravelPlan(dest);
if (botAI->rpgInfo.HasActiveTravelPlan())
{