widelands-dev team mailing list archive

[SirVer <SirVer@xxxxxx>]

Review: Needs Fixing

That is not a proper fix. Assume this order of action:

- you set 10 fish on water.
- you switch the terrain to be greenland. Now you have 10 fish on greenland and the map is broken.
- you now click the reduce resource tool -> you now have 9 fish on water, but that is still wrong.

Also, your fix should then also be applied to the set resources tool. 

I left a comment on the bug on how I believe this should be fixed.
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-977980/+merge/242410
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-977980.