Barbarian weaving mill produces endless cloth
Bug #988870 reported by
wl-zocker
This bug affects 4 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
widelands |
Fix Released
|
Wishlist
|
Unassigned | ||
widelands (Debian) |
Fix Released
|
Unknown
|
Bug Description
The Barbarian weaving mill checks only if enough thatch reed is in the economy, not if cloth is needed. As the default target quantity of thatch reed is 1, one can easily run out of it when some buildings are built simultaneously.
I suggest to set the default target quanity of both thatch reed and cloth to 10 (so that one always has some of it if one wants to build a building or a ship) and to add a line "return=skipped unless economy needs cloth" to the weaving mill's conf file.
Related branches
lp:~widelands-dev/widelands/bug988870
- Nasenbaer: Approve
- Widelands Developers: Pending requested
-
Diff: 425 lines (+52/-49)28 files modifiedtribes/atlanteans/burners_house/conf (+1/-1)
tribes/atlanteans/coalmine/conf (+2/-2)
tribes/atlanteans/crystalmine/conf (+6/-6)
tribes/atlanteans/goldmine/conf (+2/-2)
tribes/atlanteans/horsefarm/conf (+2/-2)
tribes/atlanteans/ironmine/conf (+2/-2)
tribes/atlanteans/mill/conf (+1/-1)
tribes/atlanteans/smokery/conf (+2/-2)
tribes/atlanteans/spiderfarm/conf (+2/-2)
tribes/atlanteans/spideryarn/conf (+1/-0)
tribes/barbarians/bakery/conf (+2/-2)
tribes/barbarians/brewery/conf (+2/-2)
tribes/barbarians/burners_house/conf (+1/-1)
tribes/barbarians/cattlefarm/conf (+2/-2)
tribes/barbarians/thatchreed/conf (+2/-0)
tribes/barbarians/weaving-mill/conf (+1/-1)
tribes/empire/brewery/conf (+2/-2)
tribes/empire/burners_house/conf (+1/-1)
tribes/empire/coalmine/conf (+2/-2)
tribes/empire/deep_coalmine/conf (+2/-2)
tribes/empire/deep_goldmine/conf (+2/-2)
tribes/empire/deep_oremine/conf (+2/-2)
tribes/empire/donkeyfarm/conf (+2/-2)
tribes/empire/goldmine/conf (+2/-2)
tribes/empire/mill/conf (+1/-1)
tribes/empire/oremine/conf (+2/-2)
tribes/empire/piggery/conf (+2/-2)
tribes/empire/wine/conf (+1/-1)
tags: | added: economy |
Changed in widelands (Debian): | |
status: | Unknown → New |
Changed in widelands (Debian): | |
status: | New → Fix Released |
Changed in widelands: | |
status: | Confirmed → Fix Committed |
To post a comment you must log in.
sounds like a reasonable suggestion. :)