Allow for separate blocks in PlayerBlockBreakEvent

src/main/java/net/minestom/server/event/player/PlayerBlockBreakEvent.java

@@ -7,32 +7,36 @@ public class PlayerBlockBreakEvent extends CancellableEvent {
 
     private BlockPosition blockPosition;
 
-    private short blockId;
+    private short resultBlockId;
-    private boolean customBlock;
+    private short resultCustomBlockId;
 
-    public PlayerBlockBreakEvent(BlockPosition blockPosition) {
+    public PlayerBlockBreakEvent(BlockPosition blockPosition, short resultBlockId, short resultCustomBlockId) {
         this.blockPosition = blockPosition;
+        this.resultBlockId = resultBlockId;
+        this.resultCustomBlockId = resultCustomBlockId;
     }
 
     public BlockPosition getBlockPosition() {
         return blockPosition;
     }
 
-    public short getResultBlock() {
+    public short getResultBlockId() {
-        return blockId;
+        return resultBlockId;
     }
 
-    public void setResultBlock(short blockId) {
+    public void setResultBlockId(short resultBlockId) {
-        this.blockId = blockId;
+        this.resultBlockId = resultBlockId;
-        this.customBlock = false;
+    }
+
+    public short getResultCustomBlockId() {
+        return resultCustomBlockId;
+    }
+
+    public void setResultCustomBlockId(short resultCustomBlockId) {
+        this.resultCustomBlockId = resultCustomBlockId;
     }
 
     public boolean isResultCustomBlock() {
-        return customBlock;
+        return resultCustomBlockId != 0;
-    }
-
-    public void setResultCustomBlock(short customBlockId) {
-        this.blockId = customBlockId;
-        this.customBlock = true;
     }
 }

src/main/java/net/minestom/server/instance/InstanceContainer.java

@@ -213,23 +213,19 @@ public class InstanceContainer extends Instance {
         int z = blockPosition.getZ();
 
         short blockId = chunk.getBlockId(x, y, z);
+
+        // The player probably have a wrong version of this chunk section, send it
         if (blockId == 0) {
             sendChunkSectionUpdate(chunk, ChunkUtils.getSectionAt(y), player);
             return;
         }
 
-        PlayerBlockBreakEvent blockBreakEvent = new PlayerBlockBreakEvent(blockPosition);
+        PlayerBlockBreakEvent blockBreakEvent = new PlayerBlockBreakEvent(blockPosition, (short) 0, (short) 0);
         player.callEvent(PlayerBlockBreakEvent.class, blockBreakEvent);
         if (!blockBreakEvent.isCancelled()) {
 
             // Break or change the broken block based on event result
-            short resultBlockId = blockBreakEvent.getResultBlock();
+            setSeparateBlocks(x, y, z, blockBreakEvent.getResultBlockId(), blockBreakEvent.getResultCustomBlockId());
-            boolean custom = blockBreakEvent.isResultCustomBlock();
-            if (custom) {
-                setCustomBlock(x, y, z, resultBlockId);
-            } else {
-                setBlock(x, y, z, resultBlockId);
-            }
 
             ParticlePacket particlePacket = ParticleCreator.createParticlePacket(Particle.BLOCK, false,
                     x + 0.5f, y, z + 0.5f,
@@ -240,6 +236,7 @@ public class InstanceContainer extends Instance {
 
             chunk.sendPacketToViewers(particlePacket);
         } else {
+            // Cancelled so we need to refresh player chunk section
             sendChunkSectionUpdate(chunk, ChunkUtils.getSectionAt(blockPosition.getY()), player);
         }
     }

src/main/java/net/minestom/server/registry/RegistryMain.java

@@ -395,8 +395,6 @@ public class RegistryMain {
     }
 
     private static JsonObject parse(String path, String key) {
-        List<RegistryStat> registryStats = new ArrayList<>();
-
         BufferedReader bufferedReader = null;
         try {
             bufferedReader = new BufferedReader(new FileReader(path));