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

Alloy Recipe Loading Issue #111

Open
charlie-e09ae35d opened this issue Mar 21, 2016 · 1 comment
Open

Alloy Recipe Loading Issue #111

charlie-e09ae35d opened this issue Mar 21, 2016 · 1 comment

Comments

@charlie-e09ae35d
Copy link

minestrappolation 4.3.0 for minecraft 1.8.9

sorry if i am missing something obvious but is there a reason behind those recipes in alloy smelter?

two apples + two pie crusts = two apple pies, code link
one pumpkin + two pie crusts = two pumpkin pies, code link

[SEE BELOW DESCRIPTION FOR OVERARCHING ISSUE]

@Delocuro
Copy link
Contributor

Yeah, that's a known issue. It's actually due to some code limitations in the Alloy Furnace's recipe loader; when it's loading recipes, it only takes into account the first instance of any particular item. Thus, if you have two recipes that require the same item, it will just make both of them require the quantity of items specified in whichever one was loaded first -- ie, if you had a "1 apple + 1 pie crust = 1 apple pie" recipe, and a "1 Pumpkin + 2 Pie Crust = 2 Pumpkin Pie" recipe, the pumpkin one would only require 1 pie crust since it was specified in the apple recipe, despite the pumpkin one calling for 2.

Because we hadn't gotten around to fixing that when we added the pie recipes, and I wanted the pumpkin one to give two pies per pumpkin to compensate for the added expensiveness of the pie crust, I had to find some workaround -- hence that weirdness. We really should get that alloy loading issue fixed though, so I'll change the title of this issue to that.

@Delocuro Delocuro changed the title Alloy Smleter Apple and Pumpkin Pie Recipe Inconsistency Alloy Recipe Loading Issue Mar 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants