-
Notifications
You must be signed in to change notification settings - Fork 4
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
Crafting Supplier takes more materials than single item recipe requires, when recipe specified has more than 1 item per slot #18
Comments
Eh, I should have probably added here instead:
Fixed in #19 |
Release 1.2.4 should now fix this, along with improving the crafting inventory usability by allowing moving items within it, and replacement of items. Limiting the stacksize to 1 would probably have been easier, but instead I made both the Crafting Supplier and Autocrafter (both affected by this bug) take only the minimum required recipes. This was a bit more difficult because of replacements, but it should now work correctly, while keeping replacements accurate. Also has been published on ContentDB. |
Confirming it works. |
Yes, that's possible, minetest provides a function to get the crafting recipes for an item. Two things make it a bit more challenging to implement though:
Still, it's possible, and for most items it could work just fine. Will keep it in mind to implement, but want to try and take care of some previous bugs/requests first. |
Going to close this as done, as it should be now fixed correctly. |
E.g. if the recipe is like so:
Then when requested to craft 1 item the crafting supplier will erroniously:
Two ways to solve this:
Created from discussion in #17
The text was updated successfully, but these errors were encountered: