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

CommonHooks.getCraftingRemainingItem not used #1585

Open
ewoudje opened this issue Oct 9, 2024 · 0 comments · May be fixed by #1672
Open

CommonHooks.getCraftingRemainingItem not used #1585

ewoudje opened this issue Oct 9, 2024 · 0 comments · May be fixed by #1672
Labels
1.21.1 Targeted at Minecraft 1.21.1 bug A bug or error

Comments

@ewoudje
Copy link

ewoudje commented Oct 9, 2024

Minecraft Version: 1.21.1

NeoForge Version: 21.1.65

Description of issue:
CommonHooks.getCraftingRemainingItem never gets called, instead all calls are done directly via IItemStackExtension.
I'm assuming patched calls should make use of the hook instead of the method in IItemStackExtension or in IItemStackExtension a call to CommonHooks.getCraftingRemainingItem.

Not doing this causes it to not detect broken items and thus not firing PlayerDestroyItemEvent which has in its docs that it should fire under these circumstances.

@ewoudje ewoudje added the triage Needs triaging and confirmation label Oct 9, 2024
@XFactHD XFactHD added bug A bug or error 1.21.1 Targeted at Minecraft 1.21.1 and removed triage Needs triaging and confirmation labels Oct 9, 2024
ewoudje added a commit to ewoudje/NeoForge that referenced this issue Nov 8, 2024
@ewoudje ewoudje linked a pull request Nov 8, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.21.1 Targeted at Minecraft 1.21.1 bug A bug or error
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants