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

Add CrafterCraftItemEvent #11082

Merged
merged 6 commits into from
Jul 19, 2024
Merged

Conversation

lynxplay
Copy link
Contributor

Adds an event fired when the crafter crafts an item. The event only exposes the recipe and the potential result for event modifications.

The PR does not contain a preparation related event along the lines of PrepareItemCraftEvent, as exposing the relative players in such an event is out of scope of the PR. It is also not necessary for plugins to prevent crafting specific items.

Future improvements to this event may include boolean flags to prevent items from being consumed, but this is also out of scope for the PR.

@lynxplay lynxplay requested a review from a team as a code owner July 15, 2024 13:40
@lynxplay lynxplay force-pushed the feature/crafter-craft-event branch from bdd7810 to 2d64f83 Compare July 15, 2024 14:55
@lynxplay lynxplay linked an issue Jul 15, 2024 that may be closed by this pull request
@lynxplay
Copy link
Contributor Author

Unfortunately, in all of their wisdom, upstream has begun work on this event to.
Will monitor their progress, but presumably their PR will supersede this one.

I'll try my best to ensure all methods present on this PR will be in the upstream version too.

@lynxplay lynxplay force-pushed the feature/crafter-craft-event branch from 2d64f83 to 39883ac Compare July 18, 2024 14:49
lynxplay added 5 commits July 19, 2024 09:02
Adds an event fired when the crafter crafts an item.
The event only exposes the recipe and the potential result
for event modifications.
@lynxplay lynxplay force-pushed the feature/crafter-craft-event branch from 8ea242e to f7851eb Compare July 19, 2024 07:14
@lynxplay lynxplay force-pushed the feature/crafter-craft-event branch from 7b9a2b5 to 3161d1d Compare July 19, 2024 09:15
@lynxplay lynxplay merged commit 2a39276 into PaperMC:master Jul 19, 2024
3 checks passed
@lynxplay lynxplay deleted the feature/crafter-craft-event branch July 19, 2024 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

Crafters lead to duplication with custom crafting plugins
4 participants