[PATCH] Invalid commands should be ignored

[Josef Kufner]

When starting to build a building, all selected units go to building site, including units that cannot build. These units should ignore build command and leave space for workers. The same problem is with garrisoning into siege weapons... siege weapon tries to stay in formation and runs away from soldiers trying to garrison (#3442).

[Itms]

Idea from #2946: The units that cannot build could also move towards the target but stay at some distance to avoid crowding.

[Imarok]

​https://github.com/Imarok/0ad/tree/2159_invalid_commands

[fatherbushido]

That's not my own idea as I have never thought about that problem but what follow should be taken into account: I don't know if we must make the units so intelligent. When selecting something and clicking somewhere what do we expect ? If we don't expect what happens, the player just have to select something more accurate.

[elexis]

I think it might be just as unexpected if the other units remain there. But I agree that it were nice if they would not block the foundation but only walk near that, so perhaps a WalkToRange thing might be better suited.

[wraitii]

I definitely agree the units should walk there, and stay a small distance away. Removing rfc. I think we can remove "design" too.

[elexis]

Agree, either won't fix or keep min distance to the target if we absolutely have to. With regards to siege engines reaching the target, the idea to push units away was also mentioned, but would require more elaborate pathfinder changes.

[Imarok]

Move tickets to Simulation as UI & Simulation got some sub components.