As the title says, if a market is destroyed/captured f.e. by an invading army, and the traders are not assigned to a new trade route, they will
stop moving where they are (which may be towards the territory edge) and stay idle; they then may become surrounded by enemies, and/or they may end up in neutral or enemy territory.
Details
Details
See that the diff causes land traders to move to the closest base, if there is one, instead of stopping to move when there is no route to take.
Possibly test by destroying an AI's market early in a game when it only has two.
Diff Detail
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
Comment Actions
Build is green
Updating workspaces. Build (release)... Build (debug)... Running release tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK! Running debug tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK! Checking XML files...
http://jw:8080/job/phabricator/1724/ for more details.
Comment Actions
Build is green
Updating workspaces. Build (release)... Build (debug)... Running release tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK! Running debug tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK! Checking XML files...
http://jw:8080/job/phabricator/1728/ for more details.
Comment Actions
Looks good. There is only one inline comment, but i can fix it when commiting.
binaries/data/mods/public/simulation/ai/petra/tradeManager.js | ||
---|---|---|
546 ↗ | (On Diff #2883) | should be removed and use access from line 535 |