Can you git rebase -i and squash these into one?

Squashed in to one commit

Added documentation that result item stack may be null if nothing would normally be produced.

Change the title - this is the event to handle anvils choosing the result.

Documented that this event can be triggered multiple times when an item is placed in the Javadoc. That seems unavoidable as the client sends multiple packets, each triggering that call when an item is initially placed.

Oh yeah, do we have a test plugin to verify that anything changed actually displays right on the client?

My test was tacked on to another plugin I happened to be writing at the time - can pull it out in to a test plugin soon & get back to you.

Well, I can take your word for it too. Just make sure to test that it shows up on the client.

Sorry, lost track and forgot to respond. Yes, changes here show up on the client.

@gmcferrin added it to description, it seems he linked this one from the CB but not the other way around

From @amaranth:

We wouldn't want to change the cost without that showing on the client so this just becomes a named replacement for InventoryClickEvent that erroneously gets called 3 times

Unless this is no longer true, we see no reason to pull this.