Merge lp:~widelands-dev/widelands/economy-target-profiles into lp:widelands
Proposed by
Benedikt Straub
Status: | Merged | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Merged at revision: | 9179 | ||||||||||||||||
Proposed branch: | lp:~widelands-dev/widelands/economy-target-profiles | ||||||||||||||||
Merge into: | lp:widelands | ||||||||||||||||
Diff against target: |
75 lines (+10/-8) 2 files modified
src/wui/economy_options_window.cc (+10/-7) src/wui/economy_options_window.h (+0/-1) |
||||||||||||||||
To merge this branch: | bzr merge lp:~widelands-dev/widelands/economy-target-profiles | ||||||||||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
GunChleoc | Approve | ||
Review via email: mp+371251@code.launchpad.net |
Commit message
Forbid changing economy targets by spectators
Description of the change
Added some missing checks for can_act and got rid of an unused variable.
It is still allowed to save another player´s targets to disk though as this does not affect the game.
To post a comment you must log in.
LGTM :)
Do we want to disable the save button too?