Paper
Paper copied to clipboard
PlayerStonecutterRecipeSelectEvent not firing on API-created inventories
Expected behavior
When creating a Stonecutter inventory like so:
Inventory inventory = Bukkit.createInventory(null, InventoryType.STONECUTTER);
PlayerStonecutterRecipeSelectEvent is never fired for API-created inventories.
Observed/Actual behavior
The event is not fired.
Steps/models to reproduce
- Create a listener that listens on
PlayerStonecutterRecipeSelectEvent. - Create a Stonecutter inventory using Bukkit#createInventory
- Recipes selected within this inventory will not fire the event.
Plugin list
Just a basic test plugin.
Paper version
This server is running Paper version git-Paper-777 (MC: 1.16.5) (Implementing API version 1.16.5-R0.1-SNAPSHOT) You are running the latest version Previous version: git-Paper-775 (MC: 1.16.5)
Agreements
- [X] I am running the latest version of Paper available from https://papermc.io/downloads.
- [X] I have searched for and ensured there isn't already an open issue regarding this.
- [X] My version of Minecraft is supported by Paper.
Other
No response
What if you use HumanEntity#openStoneCutter instead? The Bukkit.createInventory API has many flaws at the moment due to how some of the inventories are implemented in NMS. Some of these flaws are documented in the method's JavaDocs, some possibly aren't.
What if you use
HumanEntity#openStoneCutterinstead? TheBukkit.createInventoryAPI has many flaws at the moment due to how some of the inventories are implemented in NMS. Some of these flaws are documented in the method's JavaDocs, some possibly aren't.
Yea, I see that method now - guess that means I can't create a stonecutter using custom titles and stuff. Thanks for the heads up. I can create what I need to with packet listeners, anyways.
Not sure if this issue should be closed, or remain open to track this bug if its intended to be fixed?
Fixed by https://github.com/PaperMC/Paper/pull/7265