Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stall incorrectly handles multiples of ingots traded #2

Closed
TheWhiteWolves opened this issue Dec 15, 2014 · 2 comments
Closed

Stall incorrectly handles multiples of ingots traded #2

TheWhiteWolves opened this issue Dec 15, 2014 · 2 comments

Comments

@TheWhiteWolves
Copy link

While having a play around with your mod and testing it out I found that the stall is incorrectly handling the trading of multiple ingots in a trade.

[Warehaouse and Ingot piles before the trade]
2014-12-15_00 40 08

[Stall GUI before the trade]
2014-12-15_00 40 39

[Stall GUI after trading 2 iron for 1 gold - this works correctly as expected]
2014-12-15_00 41 05

[The warehouse and ingot piles after the first trade]
2014-12-15_00 41 14

[The Stall GUI after trading 1 gold for 2 iron - it looks like it worked as expected, However...]
2014-12-15_00 41 28

[After closing and reopening the GUI - notice it the numbers are not what they should be...]
2014-12-15_00 41 35

[The ingot piles show that it has only taken 1 of the 2 iron away in the second trade and given 2 iron to the buyer and has taken 1 gold from the buyer but not given it to the seller]
2014-12-15_00 41 44

Hope this is enough for you to understand whats happening and maybe track down the cause.

@TheWhiteWolves
Copy link
Author

Also to note I was using Forge-10.13.2.1264, TerraFirmaCraft-0.79.13.468 and Merchants-1.0.8.

@Aleksey-Terzi
Copy link
Owner

Thank you for detailed description.
Bug is reproduced and fixed in 1.0.10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants