Merge lp://staging/~widelands-dev/widelands/bug-1658456-quantity-empire-soldier into lp://staging/widelands
Proposed by
Notabilis
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Merged at revision: | 8281 | ||||
| Proposed branch: | lp://staging/~widelands-dev/widelands/bug-1658456-quantity-empire-soldier | ||||
| Merge into: | lp://staging/widelands | ||||
| Diff against target: |
12 lines (+1/-1) 1 file modified
src/logic/playercommand.cc (+1/-1) |
||||
| To merge this branch: | bzr merge lp://staging/~widelands-dev/widelands/bug-1658456-quantity-empire-soldier | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| SirVer | Approve | ||
|
Review via email:
|
|||
Description of the change
Fixed bug that the empire soldier target quantity could not be changed. Turned out to be a simple copy&paste bug: The worker id was compared against the known ware ids. The carrier animals and the other soldiers just got lucky and were accepted since there exists wares with the same id.
To post a comment you must log in.
Revision history for this message
| bunnybot (widelandsofficial) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Nic find!
@bunnybot merge