Skip to content

Fix PlayerItemConsumeEvent cancelling (fixes #4682)#5383

Merged
zachbr merged 1 commit into
PaperMC:masterfrom
chickeneer:4682
Apr 17, 2021
Merged

Fix PlayerItemConsumeEvent cancelling (fixes #4682)#5383
zachbr merged 1 commit into
PaperMC:masterfrom
chickeneer:4682

Conversation

@chickeneer
Copy link
Copy Markdown
Contributor

When the active item is not cleared, the item is still readied for use and will repeatedly trigger the PlayerItemConsumeEvent till their item is switched.
This patch clears the active item when the event is cancelled

@chickeneer chickeneer requested a review from a team as a code owner March 19, 2021 05:45
@MiniDigger MiniDigger linked an issue Apr 10, 2021 that may be closed by this pull request
@electronicboy
Copy link
Copy Markdown
Member

needs rebasing, but, want this mergarino'd soon

@chickeneer
Copy link
Copy Markdown
Contributor Author

This is rebased to be mergeable at the moment. In general, just runs the risk of needing an updated patch number if not acted on. Low Diff patch, so low likeliness of conflicts.

@zachbr zachbr merged commit ca7111d into PaperMC:master Apr 17, 2021
@electronicboy
Copy link
Copy Markdown
Member

I know, I just generally prefer to avoid merging outdated stuff in, giving aikar free commits, smh, gets spammy too :D

ExcessiveAmountsOfZombies pushed a commit to ExcessiveAmountsOfZombies/Paper that referenced this pull request May 14, 2021
LeonTG pushed a commit to LeonTG/Paper that referenced this pull request May 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PlayerItemConsumeEvent is fired when it shouldn't be

4 participants