Detect Player Blocks

Author: Prince527GitHub

.github/workflows/build-dev.yml

@@ -0,0 +1,48 @@
+name: Build Development
+
+on:
+  push:
+    branches: [main]
+
+permissions:
+  contents: write
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Set up JDK 21
+        uses: actions/setup-java@v4
+        with:
+          java-version: 21
+          distribution: temurin
+
+      - name: Download GriefPrevention
+        run: curl -o GriefPrevention.jar https://api.serversmp.xyz/upload/69f601b7986e1f4bd6f7cf46.jar
+
+      - name: Download LogBlock
+        run: curl -o LogBlock.jar https://api.serversmp.xyz/upload/69f601b7a022054e454d11a4.jar
+
+      - name: Install GriefPrevention to Maven
+        run: mvn install:install-file -Dfile=GriefPrevention.jar -DgroupId=com.github.TechFortress -DartifactId=GriefPrevention -Dversion=16.16.0 -Dpackaging=jar -DgeneratePom=true
+
+      - name: Install LogBlock to Maven
+        run: mvn install:install-file -Dfile=LogBlock.jar -DgroupId=de.diddiz -DartifactId=logblock -Dversion=1.16.1.2-SNAPSHOT -Dpackaging=jar -DgeneratePom=true
+
+      - name: Build
+        run: mvn package --file pom.xml
+
+      - name: Extract repository name
+        run: echo "NAME=$(basename ${{ github.repository }})" >> $GITHUB_ENV
+
+      - name: Upload Build
+        uses: marvinpinto/action-automatic-releases@master
+        with:
+          title: "${{ env.NAME }}"
+          automatic_release_tag: "latest"
+          repo_token: "${{ secrets.GITHUB_TOKEN }}"
+          files: "target/*.jar"
+          prerelease: false

.gitignore

@@ -9,3 +9,4 @@
 /.apt_generated/
 /.apt_generated_tests/
 dependency-reduced-pom.xml
+.DS_Store

src/main/java/com/syntaxphoenix/spigot/smoothtimber/SmoothTimber.java

@@ -13,10 +13,12 @@
 import com.syntaxphoenix.spigot.smoothtimber.event.AsyncPlayerChoppedTreeEvent;
 import com.syntaxphoenix.spigot.smoothtimber.event.AsyncPlayerTreeFallEvent;
 import com.syntaxphoenix.spigot.smoothtimber.event.reason.DefaultReason;
+import com.syntaxphoenix.spigot.smoothtimber.listener.BlockPlaceListener;
 import com.syntaxphoenix.spigot.smoothtimber.platform.Platform;
 import com.syntaxphoenix.spigot.smoothtimber.toggle.ToggleStorage;
 import com.syntaxphoenix.spigot.smoothtimber.utilities.PluginUtils;
 import com.syntaxphoenix.spigot.smoothtimber.utilities.cooldown.CooldownHelper;
+import com.syntaxphoenix.spigot.smoothtimber.utilities.locate.PlacedBlockTracker;
 import com.syntaxphoenix.spigot.smoothtimber.version.manager.VersionChanger;
 import com.syntaxphoenix.syntaxapi.command.CommandManager;
 import com.syntaxphoenix.syntaxapi.reflection.ClassCache;
@@ -38,6 +40,8 @@ public void onEnable() {
             return;
         }
         STORAGE = new ToggleStorage();
+        PlacedBlockTracker.setup(this);
+        getServer().getPluginManager().registerEvents(new BlockPlaceListener(), this);
 
         // Load classes
         CooldownHelper.class.getClass();

src/main/java/com/syntaxphoenix/spigot/smoothtimber/listener/BlockBreakListener.java

@@ -45,6 +45,10 @@ public void onBlockBreak(final BlockBreakEvent event) {
         }
 
         final Player player = event.getPlayer();
+        final Location location = event.getBlock().getLocation();
+
+        // Check if block is player-placed (PDC automatically removes the marker when block is destroyed)
+        final boolean isPlaced = Locator.isPlayerPlaced(location);
 
         if (!PlayerState.isPermitted(player)) {
             return;
@@ -77,8 +81,7 @@ public void onBlockBreak(final BlockBreakEvent event) {
             if (!change.hasPermissionForCuttingItem(player, tool)) {
                 return;
             }
-            final Location location = event.getBlock().getLocation();
-            if (Locator.isPlayerPlaced(location)) {
+            if (isPlaced) {
                 return;
             }
             event.setCancelled(true);

src/main/java/com/syntaxphoenix/spigot/smoothtimber/listener/BlockPlaceListener.java

@@ -0,0 +1,21 @@
+package com.syntaxphoenix.spigot.smoothtimber.listener;
+
+import org.bukkit.event.EventHandler;
+import org.bukkit.event.EventPriority;
+import org.bukkit.event.Listener;
+import org.bukkit.event.block.BlockPlaceEvent;
+
+import com.syntaxphoenix.spigot.smoothtimber.utilities.PluginUtils;
+import com.syntaxphoenix.spigot.smoothtimber.utilities.locate.PlacedBlockTracker;
+import com.syntaxphoenix.spigot.smoothtimber.version.manager.VersionChanger;
+
+public class BlockPlaceListener implements Listener {
+
+    @EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true)
+    public void onBlockPlace(BlockPlaceEvent event) {
+        VersionChanger changer = PluginUtils.CHANGER;
+        if (changer.isWoodBlock(event.getBlock().getState())) {
+            PlacedBlockTracker.markAsPlaced(event.getBlock());
+        }
+    }
+}

src/main/java/com/syntaxphoenix/spigot/smoothtimber/utilities/locate/DefaultResolver.java

@@ -47,7 +47,7 @@ public List<Location> resolve(final Location start, final int radius, final List
 
     @Override
     public boolean isPlayerPlaced(final Location location) {
-        return false;
+        return PlacedBlockTracker.isPlayerPlaced(location);
     }
 
 }

src/main/java/com/syntaxphoenix/spigot/smoothtimber/utilities/locate/PlacedBlockTracker.java

@@ -0,0 +1,101 @@
+package com.syntaxphoenix.spigot.smoothtimber.utilities.locate;
+
+import org.bukkit.Chunk;
+import org.bukkit.Location;
+import org.bukkit.NamespacedKey;
+import org.bukkit.block.Block;
+import org.bukkit.persistence.PersistentDataType;
+import org.bukkit.plugin.Plugin;
+
+/**
+ * Tracks player-placed blocks using Minecraft's PersistentDataContainer (PDC).
+ * This approach stores metadata directly on blocks, which automatically handles:
+ * - Block removal (any method: player break, explosion, fire, WorldEdit, etc.)
+ * - Server restarts (data persists in chunk files)
+ * - No memory overhead or file I/O
+ * - No cleanup needed
+ */
+public class PlacedBlockTracker {
+
+    private static NamespacedKey PLAYER_PLACED_KEY;
+
+    public static void setup(Plugin plugin) {
+        PLAYER_PLACED_KEY = new NamespacedKey(plugin, "player_placed");
+    }
+
+    /**
+     * Marks a block as player-placed.
+     * @param block The block to mark
+     */
+    public static void markAsPlaced(Block block) {
+        if (PLAYER_PLACED_KEY == null) {
+            return;
+        }
+        block.getChunk().getPersistentDataContainer().set(
+            getChunkKey(block.getLocation()), 
+            PersistentDataType.BYTE, 
+            (byte) 1
+        );
+    }
+    
+    /**
+     * Gets a unique key for a block location within a chunk.
+     */
+    private static NamespacedKey getChunkKey(Location location) {
+        String key = "player_placed_" + location.getBlockX() + "_" + location.getBlockY() + "_" + location.getBlockZ();
+        return new NamespacedKey(PLAYER_PLACED_KEY.getNamespace(), key);
+    }
+
+    /**
+     * Marks a block at the given location as player-placed.
+     * @param location The location of the block to mark
+     */
+    public static void markAsPlaced(Location location) {
+        markAsPlaced(location.getBlock());
+    }
+
+    /**
+     * Checks if a block was placed by a player.
+     * @param block The block to check
+     * @return true if the block was placed by a player, false otherwise
+     */
+    public static boolean isPlayerPlaced(Block block) {
+        if (PLAYER_PLACED_KEY == null) {
+            return false;
+        }
+        return block.getChunk().getPersistentDataContainer().has(
+            getChunkKey(block.getLocation()), 
+            PersistentDataType.BYTE
+        );
+    }
+
+    /**
+     * Checks if a block at the given location was placed by a player.
+     * @param location The location of the block to check
+     * @return true if the block was placed by a player, false otherwise
+     */
+    public static boolean isPlayerPlaced(Location location) {
+        return isPlayerPlaced(location.getBlock());
+    }
+
+    /**
+     * Removes the player-placed marker from a block.
+     * Note: This is usually not needed as the PDC is automatically removed when blocks are destroyed.
+     * @param block The block to unmark
+     */
+    public static void unmark(Block block) {
+        if (PLAYER_PLACED_KEY == null) {
+            return;
+        }
+        block.getChunk().getPersistentDataContainer().remove(getChunkKey(block.getLocation()));
+    }
+
+    /**
+     * Removes the player-placed marker from a block at the given location.
+     * Note: This is usually not needed as the PDC is automatically removed when blocks are destroyed.
+     * @param location The location of the block to unmark
+     */
+    public static void unmark(Location location) {
+        unmark(location.getBlock());
+    }
+}