Merge lp:~widelands-dev/widelands/empire04_unused_key_return_on_dismantle_no_ui into lp:widelands
Proposed by
GunChleoc
Status: | Merged |
---|---|
Merged at revision: | 8927 |
Proposed branch: | lp:~widelands-dev/widelands/empire04_unused_key_return_on_dismantle_no_ui |
Merge into: | lp:widelands |
Diff against target: |
45 lines (+15/-10) 1 file modified
src/logic/map_objects/tribes/building.cc (+15/-10) |
To merge this branch: | bzr merge lp:~widelands-dev/widelands/empire04_unused_key_return_on_dismantle_no_ui |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
hessenfarmer | test review | Approve | |
Review via email: mp+358305@code.launchpad.net |
Commit message
Allow return on dismantle values without buildcost.
UI fix has been postponed until after Build 20, c.f.
To post a comment you must log in.
I am not sure whether this fix or the other branch has any influence on the Scenario, and I can't test this at least until next monday.
In the current implementation I managed to have the buildings in the right state some should be dismantable, some not. Some should be enhanceable, some not or only after a quest has been fulfillled. This state of the Scenario most probably is affected by these changes. If so the text is not matching the gameplay properly, so I recommend to postpone the whole thing to b21.
If you wan't to have it anyway it is your decision. I will have a look as soon as I have time for this.