PlayerInteractEvent only returns early for InventoryHolders

[ErrinDev]

Describe the bug

PlayerInteractEvent listener only returns early for InventoryHolders, causing block placement issues where the block placement gets cancelled.

Used a plugin to log PlayerInteractEvents at each priority level, saw it was EventPriority.HIGH and tried disabling each of those few plugins until it started working again.

How to reproduce

Place a barrel on another barrel and observe the placement getting cancelled

Screenshots / Videos

2025-08-15.15-33-02.mp4

Server Log

No response

[dxnnv]

Adding onto this, I diagnosed this issue by logging all events for PlayerInteractEvent. I was able to determine that the issue via the output from this:

[01:18:38 INFO]: [InteractionLogger] [EventStack - LOWEST] InteractedBlock=ALLOW | ItemInHand=DEFAULT
[01:18:38 INFO]: [InteractionLogger] [EventStack - MONITOR] InteractedBlock=ALLOW ItemInHand=DENY
[01:18:38 INFO]: [InteractionLogger] [Interact] Sneaking=true | ClickedBlockType=BARREL | ItemType=BARREL | InteractedBlock=ALLOW | ItemInHand=DENY

I added logging for each EventPriority and narrowed it down to AdvancedSkills, and was able to find the offending class in the four listeners it registers.

This should probably be refactored to at least check if the item the player is holding is placeable and if they're sneaking. Otherwise, something more robust would be even better.