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

Dragon Breath leaves empty Glass Bottle in Brewing Stand when used #2873

Closed
DDaMAGEr opened this issue Jan 31, 2023 · 4 comments · Fixed by #2874
Closed

Dragon Breath leaves empty Glass Bottle in Brewing Stand when used #2873

DDaMAGEr opened this issue Jan 31, 2023 · 4 comments · Fixed by #2874
Labels
bug Something isn't working priority:high High priority PRs that need review and work now. Review these first.

Comments

@DDaMAGEr
Copy link

Dragon Breath leaves empty Glass Bottle in ingredient slot of Brewing Stand when used.

For some strange reason, this previously happened with Forge pretty long ago, and now it happens with Fabric, but only when Fabric API is present among mods. This is pretty bad, because it may break brewing automatization, as there seems to be no way of removing empty glass bottle from ingredient slot automatically. Tested it on 1.19.3, fabric version 0.14.13, fabric api 0.73.0+1.19.3 (seems to be the latest release), the problem exists.

@haykam821
Copy link
Contributor

I can reproduce the issue by running the following command and observing the item in the upper slot of the brewing stand:

setblock ~2 ~ ~ minecraft:brewing_stand{Fuel: 20s, Items: [{Slot: 1, id: "minecraft:splash_potion", Count: 1}, {Slot: 3, id: "minecraft:dragon_breath", Count: 1}]}
Initial Expected Actual
image image image

@Shnupbups
Copy link
Contributor

Funnily enough, this seems to be actually fixing a vanilla bug. But I'd agree that we should un-fix said bug to retain vanilla parity.

@modmuss50 modmuss50 added bug Something isn't working priority:high High priority PRs that need review and work now. Review these first. labels Jan 31, 2023
@modmuss50
Copy link
Member

For some strange reason, this previously happened with Forge pretty long ago

I have just tested, this is still an issue in Forge. I have been struggling to fix this so vanilla behaviour is unchanged but this works with modded items, had hoped I could get an idea of how to fix it from looking at Forge but nope!

This is pretty bad, because it may break brewing automatization

Yes this is very bad, we should never fix vanilla bugs. I have prepared a fix here: #2874 this does apply the same vanilla bug to our modded items.

The vanilla bug can clearly be reproduced by putting 2 dragons breath in the top slot, and a single potion in the bottom slot. On the first brew an empty bottle is spawned into the world, on the 2nd the empty bottle is lost.

@modmuss50
Copy link
Member

I have opened a new vanilla bug report about this: https://bugs.mojang.com/browse/MC-259583

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority:high High priority PRs that need review and work now. Review these first.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants