Improved GriefDefender support

GriefDefender support

eco-api/src/main/java/com/willfp/eco/core/display/Display.java

@@ -150,7 +150,6 @@ public class Display {
      * @param player    The player.
      * @param args      The args.
      */
-    @SuppressWarnings("checkstyle:FinalParameters")
     @ApiStatus.Internal
     public static void callDisplayModule(@NotNull final DisplayModule module,
                                          @NotNull final ItemStack itemStack,

eco-api/src/main/java/com/willfp/eco/util/StringUtils.java

@@ -336,10 +336,14 @@ public class  StringUtils {
      * @return The JSON String.
      */
     @NotNull
-    public String legacyToJson(@NotNull final String legacy) {
+    public String legacyToJson(@Nullable final String legacy) {
+        String processed = legacy;
+        if (legacy == null) {
+            processed = "";
+        }
         return GsonComponentSerializer.gson().serialize(
                 Component.empty().decoration(TextDecoration.ITALIC, false).append(
-                        LEGACY_COMPONENT_SERIALIZER.deserialize(legacy)
+                        LEGACY_COMPONENT_SERIALIZER.deserialize(processed)
                 )
         );
     }

eco-core/core-plugin/build.gradle

@@ -13,7 +13,7 @@ dependencies {
     compileOnly 'com.github.TechFortress:GriefPrevention:16.14.0'
     compileOnly 'com.massivecraft:Factions:1.6.9.5-U0.5.10'
     compileOnly 'com.github.cryptomorin:kingdoms:1.10.14'
-    compileOnly 'com.github.TownyAdvanced:Towny:0.96.2.0'
+    compileOnly 'com.github.TownyAdvanced:Towny:0.97.1.0'
     compileOnly 'com.github.angeschossen:LandsAPI:4.7.3'
     compileOnly 'fr.neatmonster:nocheatplus:3.16.1-SNAPSHOT'
     compileOnly 'com.github.jiangdashao:matrix-api-repo:317d4635fd'

eco-core/core-plugin/src/main/kotlin/com/willfp/eco/spigot/EcoSpigotPlugin.kt

@@ -102,6 +102,7 @@ abstract class EcoSpigotPlugin : EcoPlugin(
             // AntiGrief
             IntegrationLoader("WorldGuard") { AntigriefManager.register(AntigriefWorldGuard()) },
             IntegrationLoader("GriefPrevention") { AntigriefManager.register(AntigriefGriefPrevention()) },
+            IntegrationLoader("GriefDefender") { AntigriefManager.register(AntigriefGriefDefender()) },
             IntegrationLoader("FactionsUUID") { AntigriefManager.register(AntigriefFactionsUUID()) },
             IntegrationLoader("Towny") { AntigriefManager.register(AntigriefTowny()) },
             IntegrationLoader("Lands") { AntigriefManager.register(AntigriefLands(this)) },

eco-core/core-plugin/src/main/kotlin/com/willfp/eco/spigot/integrations/antigrief/AntigriefGriefDefender.kt

@@ -0,0 +1,81 @@
+package com.willfp.eco.spigot.integrations.antigrief
+
+import com.flowpowered.math.vector.Vector3i
+import com.griefdefender.api.GriefDefender
+import com.griefdefender.api.data.PlayerData
+import com.griefdefender.api.permission.flag.Flags
+import com.willfp.eco.core.integrations.antigrief.AntigriefWrapper
+import org.bukkit.Location
+import org.bukkit.block.Block
+import org.bukkit.entity.LivingEntity
+import org.bukkit.entity.Player
+import java.util.*
+
+class AntigriefGriefDefender : AntigriefWrapper {
+    override fun canBreakBlock(player: Player, block: Block): Boolean {
+        val data = GriefDefender.getCore().getPlayerData(player.world.uid, player.uniqueId) ?: return true
+
+        val claim = GriefDefender.getCore().getClaimAt(Vector3i(block.location.x, block.location.y, block.location.z))
+            ?: return true
+
+        return if (data.canIgnoreClaim(claim)) {
+            true
+        } else {
+            claim.getActiveFlagPermissionValue(Flags.BLOCK_BREAK, data.user, Collections.emptySet(), true).asBoolean()
+        }
+    }
+
+    override fun canCreateExplosion(player: Player, location: Location): Boolean {
+        val data = GriefDefender.getCore().getPlayerData(player.world.uid, player.uniqueId) ?: return true
+
+        val claim = GriefDefender.getCore().getClaimAt(Vector3i(location.x, location.y, location.z)) ?: return true
+
+        return if (data.canIgnoreClaim(claim)) {
+            true
+        } else {
+            claim.getActiveFlagPermissionValue(Flags.EXPLOSION_BLOCK, data.user, Collections.emptySet(), true)
+                .asBoolean()
+        }
+    }
+
+    override fun canPlaceBlock(player: Player, block: Block): Boolean {
+
+        val data: PlayerData = GriefDefender.getCore().getPlayerData(player.world.uid, player.uniqueId) ?: return true
+
+        val claim =
+            GriefDefender.getCore().getClaimAt(Vector3i(block.location.x, block.location.y, block.location.z))
+                ?: return true
+
+        return if (data.canIgnoreClaim(claim)) {
+            true
+        } else {
+            claim.getActiveFlagPermissionValue(Flags.BLOCK_PLACE, data.user, Collections.emptySet(), true)
+                .asBoolean()
+        }
+    }
+
+    override fun canInjure(player: Player, victim: LivingEntity): Boolean {
+
+        val data = GriefDefender.getCore().getPlayerData(player.world.uid, player.uniqueId) ?: return true
+
+        if (data.canIgnoreClaim(data.currentClaim)) {
+            return true
+        }
+
+        return if (victim is Player) {
+            val vicData = GriefDefender.getCore().getPlayerData(victim.world.uid, victim.uniqueId)
+            if (vicData != null) {
+                data.canPvp(data.currentClaim) && data.canPvp(vicData.currentClaim)
+            } else data.canPvp(data.currentClaim)
+        } else {
+            val claim =
+                GriefDefender.getCore().getClaimAt(Vector3i(victim.location.x, victim.location.y, victim.location.z))
+            claim?.getActiveFlagPermissionValue(Flags.ENTITY_DAMAGE, data.user, Collections.emptySet(), true)
+                ?.asBoolean() ?: true
+        }
+    }
+
+    override fun getPluginName(): String {
+        return "GriefDefender"
+    }
+}

eco-core/core-plugin/src/main/kotlin/com/willfp/eco/spigot/integrations/antigrief/AntigriefTowny.kt

@@ -2,9 +2,7 @@ package com.willfp.eco.spigot.integrations.antigrief
 
 import com.palmergames.bukkit.towny.TownyAPI
 import com.palmergames.bukkit.towny.TownyUniverse
-import com.palmergames.bukkit.towny.`object`.Town
 import com.palmergames.bukkit.towny.`object`.TownyPermission
-import com.palmergames.bukkit.towny.`object`.WorldCoord
 import com.palmergames.bukkit.towny.utils.PlayerCacheUtil
 import com.willfp.eco.core.integrations.antigrief.AntigriefWrapper
 import org.bukkit.Location
@@ -67,22 +65,8 @@ class AntigriefTowny : AntigriefWrapper {
                 true
             }
         }
-        if (victim is Player) {
+        val townBlock = TownyAPI.getInstance().getTownBlock(victim.location) ?: return true
-            try {
+        return townBlock.permissions.pvp
-                val town: Town = WorldCoord.parseWorldCoord(victim.getLocation()).townBlock.town
-                return town.isPVP
-            } catch (ignored: Exception) {
-                // If exception, no town was found, thus return true.
-            }
-        } else {
-            try {
-                val town: Town = WorldCoord.parseWorldCoord(victim.location).townBlock.town
-                return town.hasMobs()
-            } catch (ignored: Exception) {
-                // If exception, no town was found, thus return true.
-            }
-        }
-        return true
     }
 
     override fun getPluginName(): String {

eco-core/core-plugin/src/main/resources/plugin.yml

@@ -25,6 +25,7 @@ softdepend:
   - ItemsAdder
   - Oraxen
   - HeadDatabase
+  - GriefDefender
 libraries:
   - org.reflections:reflections:0.9.12
   - org.apache.maven:maven-artifact:3.0.3

gradle.properties

@@ -1,2 +1,2 @@
-version = 6.8.1
+version = 6.8.4
 plugin-name = eco