← Back to team overview

widelands-dev team mailing list archive

[Merge] lp:~widelands-dev/widelands/bug1132466 into lp:widelands

 

The proposal to merge lp:~widelands-dev/widelands/bug1132466 into lp:widelands has been updated.

Commit Message changed to:

Fix bug #1132466 and a potential cheat related to evicting soldiers

The worker leavebuilding task now deals more robustly with situations in which the evicted worker is currently outside of the building.

Furthermore, it is no longer possible to evict soldiers whose current position on the map is not their assigned building (i.e., their location). Note that this was not possible with the standard user interface anyway, but a cheater could have modified her version of the game such that an eviction player command would be sent automatically for her soldiers when they reach low HP during a battle. Such soldiers would have immediately stopped battle and returned home.

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/bug1132466/+merge/150244
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug1132466/+merge/150244
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug1132466 into lp:widelands.


References