-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Final CraftItemEvent result #8439
Copy link
Copy link
Closed
Labels
status: acceptedDisputed bug is accepted as valid or Feature accepted as desired to be added.Disputed bug is accepted as valid or Feature accepted as desired to be added.type: featureRequest for a new Feature.Request for a new Feature.
Metadata
Metadata
Assignees
Labels
status: acceptedDisputed bug is accepted as valid or Feature accepted as desired to be added.Disputed bug is accepted as valid or Feature accepted as desired to be added.type: featureRequest for a new Feature.Request for a new Feature.
Type
Fields
Give feedbackNo fields configured for Feature.
Is your feature request related to a problem?
The
CraftItemEventdoes not allow you to see how many items the player actually crafted. It seems like the result acts more like a copy. It would be a lot more intuitive if the event offered the number of items the player actually crafted.Describe the solution you'd like.
Some way to access the final result of the craft event, like getFinalResult or just altering the current method that already exists to make it reflect the result of the event.
Describe alternatives you've considered.
Hacky & imperfect guessing logic to see how many items the player crafted based on their inventory.
Other
The original issue explains this pretty well: #7383. I am opening this issue because that one was closed due to being stale.
Also, IIRC the
PlayerTradeEventoffers a proper result that tells you how many emeralds or other items the player received, and works when the item is shift-clicked into the player's inventory. Maybe theCraftItemEventcould be updated to look more like that, if that's possible? Thank you for reading!