← Back to team overview

widelands-dev team mailing list archive

[Merge] lp:~widelands-dev/widelands/bug-1809838-blackroot-target into lp:widelands

 

Notabilis has proposed merging lp:~widelands-dev/widelands/bug-1809838-blackroot-target into lp:widelands.

Commit message:
Adding missing check whether blackroot flour is required.

Requested reviews:
  Widelands Developers (widelands-dev)
Related bugs:
  Bug #1809838 in widelands: "Blackroot flour economy target is ignored"
  https://bugs.launchpad.net/widelands/+bug/1809838

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/bug-1809838-blackroot-target/+merge/361290

Checking in the production program whether blackroot flour is required before grinding blackroot, so the economic target is respected.

It might be that this isn't checked on purpose, since different from corn there is no other use for blackroot. If the check shouldn't be in the production program, the economy target for blackroot flour should be removed instead.
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1809838-blackroot-target into lp:widelands.


Follow ups