Skip to content

Fix InventoryAction wrong for Bundles#11902

Merged
lynxplay merged 5 commits into
PaperMC:mainfrom
notTamion:fix-inventoryaction-bundles-hf
Jan 11, 2025
Merged

Fix InventoryAction wrong for Bundles#11902
lynxplay merged 5 commits into
PaperMC:mainfrom
notTamion:fix-inventoryaction-bundles-hf

Conversation

@notTamion
Copy link
Copy Markdown
Contributor

pre hardfork #11313

@notTamion notTamion requested a review from a team as a code owner January 4, 2025 12:57
Comment thread paper-api/src/main/java/org/bukkit/event/inventory/InventoryAction.java Outdated
@notTamion notTamion force-pushed the fix-inventoryaction-bundles-hf branch from 61f4a14 to def89f3 Compare January 4, 2025 21:55
@notTamion notTamion force-pushed the fix-inventoryaction-bundles-hf branch 2 times, most recently from 33cda12 to a198573 Compare January 4, 2025 22:52
@lynxplay lynxplay force-pushed the fix-inventoryaction-bundles-hf branch from 7e9339d to c65e09c Compare January 4, 2025 23:14
@kennytv kennytv added the type: bug Something doesn't work as it was intended to. label Jan 5, 2025
@lynxplay lynxplay force-pushed the fix-inventoryaction-bundles-hf branch from c65e09c to 662199a Compare January 11, 2025 18:36
@lynxplay lynxplay merged commit 19ddbef into PaperMC:main Jan 11, 2025
@notTamion notTamion deleted the fix-inventoryaction-bundles-hf branch January 11, 2025 19:22
rymiel added a commit to rymiel/Bolt that referenced this pull request Feb 3, 2025
Interactions with bundles have new InventoryAction enum entries in Paper
(PaperMC/Paper#11902). Due to this, Bolt was not
properly handling these interaction and always denied them. This meant
that you could not use bundles to withdraw or deposit items even from
containers you actually have access to.

This PR fixes these interactions. Due to this being recent API and in
Paper only, EnumUtil has to be used and so it looks a bit messy.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug Something doesn't work as it was intended to.

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

4 participants