Fix potential issue on Paper 1.21.8+

Author: Lauriichan

pom.xml

@@ -5,7 +5,7 @@
 
     <groupId>com.syntaxphoenix.spigot</groupId>
     <artifactId>smoothtimber-legacy</artifactId>
-    <version>1.27.6</version>
+    <version>1.27.7</version>
     <name>SmoothTimber</name>
     <packaging>jar</packaging>
 

src/main/java/com/syntaxphoenix/spigot/smoothtimber/compatibility/coreprotect/CoreProtectChopListener.java

@@ -1,6 +1,7 @@
 package com.syntaxphoenix.spigot.smoothtimber.compatibility.coreprotect;
 
 import org.bukkit.Location;
+import org.bukkit.block.BlockState;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.Listener;
 
@@ -28,7 +29,11 @@ public void onChopEvent(final AsyncPlayerChoppedTreeEvent event) {
 
     private void handleChopEvent(final AsyncPlayerChoppedTreeEvent event) {
         for (final Location location : event.getBlockLocations()) {
-            compat.logRemoval(event.getPlayer().getName(), location, Locator.getBlockState(location));
+            BlockState state = Locator.getBlockState(location);
+            if (state == null) {
+                continue;
+            }
+            compat.logRemoval(event.getPlayer().getName(), location, state);
         }
     }
 

src/main/java/com/syntaxphoenix/spigot/smoothtimber/platform/Platform.java

@@ -4,7 +4,9 @@
 
 import com.syntaxphoenix.spigot.smoothtimber.SmoothTimber;
 import com.syntaxphoenix.spigot.smoothtimber.utilities.Container;
+import com.syntaxphoenix.spigot.smoothtimber.version.manager.gen.MCVersion;
 import com.syntaxphoenix.syntaxapi.reflection.ClassCache;
+import com.syntaxphoenix.syntaxapi.reflection.Reflect;
 
 public abstract class Platform {
 
@@ -26,13 +28,23 @@ public static void shutdown() {
     private static Platform createPlatform(final SmoothTimber plugin) {
         if (ClassCache.getOptionalClass("io.papermc.paper.threadedregions.RegionizedServer")
             .or(() -> ClassCache.getOptionalClass("io.papermc.paper.threadedregions.RegionizedServerInitEvent")).isPresent()) {
-            return new FoliaPlatform(plugin);
+            switch (MCVersion.getCoreVersion()) {
+            case v1_21x:
+                // On 1.21+ we can't only use the Regionized Server cause Paper for some reason includes that API now.
+                if (ClassCache.getOptionalClass("io.papermc.paper.plugin.configuration.PluginMeta")
+                    .filter(clz -> new Reflect(clz).searchMethod("isFolia", "isFoliaSupported").containsMethod("isFolia")).isPresent()) {
+                    return new FoliaPlatform(plugin);
+                }
+                break;
+            default:
+                return new FoliaPlatform(plugin);
+            }
         }
         return new SpigotPlatform(plugin);
     }
 
     protected abstract void internalShutdown();
-    
+
     public boolean isRegional() {
         return false;
     }

src/main/java/com/syntaxphoenix/spigot/smoothtimber/version/changer/v1_13xChanger.java

@@ -63,7 +63,7 @@ public ItemStack removeDurabilityFromItem(final ItemStack stack) {
             }
         }
         final ItemMeta meta = stack.getItemMeta();
-        if (meta.isUnbreakable()) {
+        if (meta == null || meta.isUnbreakable()) {
             return stack;
         }
         if (meta instanceof Damageable) {

src/main/java/com/syntaxphoenix/spigot/smoothtimber/version/changer/v1_16xChanger.java

@@ -63,7 +63,7 @@ public ItemStack removeDurabilityFromItem(final ItemStack stack) {
             }
         }
         final ItemMeta meta = stack.getItemMeta();
-        if (meta.isUnbreakable()) {
+        if (meta == null || meta.isUnbreakable()) {
             return stack;
         }
         if (meta instanceof Damageable) {

src/main/java/com/syntaxphoenix/spigot/smoothtimber/version/changer/v1_19xChanger.java

@@ -57,7 +57,7 @@ public ItemStack removeDurabilityFromItem(final ItemStack stack) {
             }
         }
         final ItemMeta meta = stack.getItemMeta();
-        if (meta.isUnbreakable()) {
+        if (meta == null || meta.isUnbreakable()) {
             return stack;
         }
         if (meta instanceof Damageable) {

src/main/java/com/syntaxphoenix/spigot/smoothtimber/version/changer/v1_20xChanger.java

@@ -62,7 +62,7 @@ public ItemStack removeDurabilityFromItem(final ItemStack stack) {
             }
         }
         final ItemMeta meta = stack.getItemMeta();
-        if (meta.isUnbreakable()) {
+        if (meta == null || meta.isUnbreakable()) {
             return stack;
         }
         if (meta instanceof Damageable) {

src/main/java/com/syntaxphoenix/spigot/smoothtimber/version/changer/v1_21xChanger.java

@@ -55,7 +55,7 @@ public ItemStack removeDurabilityFromItem(final ItemStack stack) {
             }
         }
         final ItemMeta meta = stack.getItemMeta();
-        if (meta.isUnbreakable()) {
+        if (meta == null || meta.isUnbreakable()) {
             return stack;
         }
         if (meta instanceof Damageable) {