Jake Potrebic

@fhnaumann yeah, the issue about `create...` vs `open...` is really a whole separate thing. I have an open PR that kinda tries to do something about it. It's the same...

So I did start up on https://github.com/PaperMC/Paper/pull/10732, which is my new idea for handling this. Some basic testing with the 2 inventory types I implemented seemed to suggest they function...

Rebased and tested for 1.20.6

So `PlayerPortalEvent#getTo` isn't null ever from server use. The constructor param is marked as nullable, but event constructor's aren't API, so that doesn't really matter. `PlayerTeleportEvent#getTo` does return null sometime....

I think this is "works as intended" due to the documentation on `Inventory#getStorageContents`. The javadocs for that method say > In most cases this will represent the entire inventory, but...

It looks like previously, Paper has added specific API to cover "all slots". There is a Inventory#removeItemAnySlot method to account for the other Inventory#removeItem only removing from the "storage" contents....

I feel like despite what CB's comment says about being unable to call the click event, we could call it by just mimicking some of the checks in the `case...

I think this duplicates a lot of what is proposed in https://github.com/PaperMC/Paper/pull/7377.

> EDIT 2: Haven't looked in detail but for what i want i need access to the dispenser BE that fired the projectile or spawned the tnt, not having it...

> Ok, took a look, I can use that for any projectile, but i still need something for the tnt being dispensed. Yes, the mentioned PR does add an event...