Lone traders fail to follow rally points

[zoot]

Steps to reproduce:

1. Build two markets, market A and market B.
2. Draw a zig-zagging line of several rally points from market A to market B.
3. Train a trader in market A.

What happens:

The trader walks straight ("as the crow flies") from market A to market B.

What should happen:

The trader should follow the rally points.

[zoot]

Trader walking as the crow flies

[leper]

Fixing this is just adding the cmd.queued parameter to the call to SetupTradeRoute in Commands.js, but I'll try to fix this and #1822 in one go.

[mimo]

Replying to leper:

Fixing this is just adding the cmd.queued parameter to the call to SetupTradeRoute in Commands.js, but I'll try to fix this and #1822 in one go.

With such a change, I think the trader will only follow the required path for the first journey, but when he'll come back and after, he'd go back to the straight route.? while requiring such a route would be useful to avoid ennemies, so all journeys should follow it.

[leper]

Replying to mimo:

With such a change, I think the trader will only follow the required path for the first journey, but when he'll come back and after, he'd go back to the straight route.?

Currently yes, the wip patch at #1207 fixed that by introducing a cycle property that cycles some commands (walk, trade) that would make the trader follow that route every time.

[mimo]

In 14697:

allow the possibility to define route with waypoints when doing trade, fixes #1207, #1821 and #1822