OpenClonk Bugtracker - OpenClonk
View Issue Details
0001867OpenClonkEngine - GUIpublic2016-12-17 19:192016-12-17 23:53
ReporterZapper 
Assigned ToClonkonaut 
PrioritynormalSeverityminorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformOSOS Version
Product Versiongit master 
Target Version8.0Fixed in Version 
Summary0001867: It is not possible to transfer earth inside a bucket from Clonk to foundry
DescriptionThe behavior was broken by https://git.openclonk.org/openclonk.git/commitdiff/7ef0fc5568a4e4ee28f1838cfbb4380f5c771937

Previously, you could transfer contained earth from a bucket to a production facility by clicking on the bucket. Only an empty bucket would remain in the Clonk's inventory.

This was not only the intuitive behavior but also served as a hint to the player how earth is handled.
Now you have to go through the completely unnecessary additional hassle of opening the bucket inventory first.

I opt for reverting the commit.
TagsNo tags attached.
Attached Files

Notes
(0005540)
Maikel   
2016-12-17 19:48   
We can revert that commit, if we find a solution for the dynamite box.
(0005541)
occ   
2016-12-17 23:51   
Hi! There's been a check-in that references this bug. For more information you can visit the repository browser at this address:
https://git.openclonk.org/openclonk.git/commitdiff/afc565a41935131493f2e69db716ef7f21fd16e4

Changeset afc565a by Clonkonaut <Armag3ddon@web.de>
Re-enabled automatic bucket emptying (0001867).

(0005542)
Clonkonaut   
2016-12-17 23:53   
I wouldn't opt for reverting that commit since the change sounds very useful indeed.
I just made an exception for the bucket since that item is a bit special compared to all other items with extra slots and special cases shouldn't define the standard case.

Curiously enough, you can put earth into any building but that's not a problem with my change as it was also possible with the previous code.

Issue History
2016-12-17 19:19ZapperNew Issue
2016-12-17 19:48MaikelNote Added: 0005540
2016-12-17 23:51occNote Added: 0005541
2016-12-17 23:53ClonkonautAssigned To => Clonkonaut
2016-12-17 23:53ClonkonautStatusnew => resolved
2016-12-17 23:53ClonkonautResolutionopen => fixed
2016-12-17 23:53ClonkonautNote Added: 0005542