Jake Potrebic

This really seems like too much diff in the server patch for something so simple. Hold off on merging for now

Also, I feel like this event should not be player-specific, cause villagers also compost stuff, and hoppers can insert items as well.

(see https://github.com/PaperMC/Paper/pull/6356#discussion_r685663260)

(These things can be handled by the merger so not really "required changes now")

Why limit this to only LivingEntity in the first place? Why can't we just remove that instanceof check in the first place?

Now includes fixes for Looms and Stonecutters. Also fixes https://github.com/PaperMC/Paper/issues/5825 now.

See https://github.com/PaperMC/Paper/issues/4361#issuecomment-701640428. In a similar situations, we don't change a Bee's SpawnReason when they leave a beehive, it keeps the previous one. A similar thing should be applied here. It...

I was trying to look into this issue, its related to https://github.com/PaperMC/Paper/issues/7631, but might be fixable now, or might be a later thing like the other issue. But when I...

Yeah ok, I can replicate the issue. For future reference, the item meta is reformatted lossily when you drag to place the item (which happens if you move the cursor...

Well the fix is not going to be to undo that commit. That commit fixes a bigger issue. I'm guessing your issue is actually not the same as this. Can...