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

unified inventory (or unified inventory plus) does insane things when crafting items with replacements #166

Closed
fluxionary opened this issue Dec 15, 2019 · 6 comments
Labels
bug Something isn't working exploitable something that players can exploit to gain an unfair advantage high priority Critical issue affecting the daily functioning of the server

Comments

@fluxionary
Copy link
Member

fluxionary commented Dec 15, 2019

  1. if you hit the "All" button to craft things, you will only get 1 side effect item, no matter how many items you craft.
  2. if you craft something that has multiple side effects (like cucina vegana's rice flour), when not using the "All" button, you get way too many items back (probably 2x, didn't count)
@fluxionary fluxionary added bug Something isn't working exploitable something that players can exploit to gain an unfair advantage labels Dec 15, 2019
@catwhisperer77
Copy link
Collaborator

For the rice flour, is that something to do with how it's replacing the first stone with 3 stones?

@fluxionary
Copy link
Member Author

ah. nice catch. its replacing each of 3 stones w/ 3 stones. so only point 1 is valid, and cucina vegana needs its own fix.

@catwhisperer77
Copy link
Collaborator

homedecor dye-ables (beds/glowlights) fail to craft when using the all button (it consumes the dye and makes the default/white version)

@fluxionary
Copy link
Member Author

fluxionary commented Dec 27, 2019

@fluxionary fluxionary changed the title unified inventory (or unified inventory plus) does insane things when crafting items with side effect items unified inventory (or unified inventory plus) does insane things when crafting items with replacements Dec 27, 2019
@fluxionary fluxionary added the high priority Critical issue affecting the daily functioning of the server label May 18, 2020
@fluxionary
Copy link
Member Author

This issue is still unresolved upstream. We may have to take it over.

@fluxionary
Copy link
Member Author

I rewrote unified_inventory_plus, and fixed all of the major issues:
mt-mods/unified_inventory_plus#16
bousket doesn't want to actively maintain the mod anymore, so if I don't hear back from them, I'm going to try to get the mod moved into minetest-mods.

@oversword: feel free to point at my repo for the time being, if you wish.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working exploitable something that players can exploit to gain an unfair advantage high priority Critical issue affecting the daily functioning of the server
Projects
None yet
Development

No branches or pull requests

2 participants