Fixed the Ghost Blocks appearing when a packing up a block.
The main issue here, was the desynced return between server and client.
I moved the client check to a spot where no desync will happen, and additionally I adjusted the return value.
ActionResult.SUCCESS and ActionResult.CONSUME are currently equal in Minecraft. But to avoid potential future issues, I switched it over to CONSUME, as semantically SUCCESS doesn't necessarily have to actually consume the action and might still forward it.
Fixed the Ghost Blocks appearing when a packing up a block. The main issue here, was the desynced return between server and client. I moved the client check to a spot where no desync will happen, and additionally I adjusted the return value. ActionResult.SUCCESS and ActionResult.CONSUME are currently equal in Minecraft. But to avoid potential future issues, I switched it over to CONSUME, as semantically SUCCESS doesn't necessarily have to actually consume the action and might still forward it.