Use (Neo)Forge permissions system too

Author: BluSpring

build.gradle.kts

@@ -90,7 +90,9 @@ dependencies {
         modImplementation("com.terraformersmc:modmenu:$modMenuVersion")
     }
 
-    includeOrShade(modImplementation("me.lucko:fabric-permissions-api:0.3.1")!!)
+    if (mcData.isFabric) {
+        includeOrShade(modImplementation("me.lucko:fabric-permissions-api:0.3.1")!!)
+    }
 
     val useSVC = false
 

src/main/kotlin/xyz/bluspring/unitytranslate/PlatformProxy.kt

@@ -8,6 +8,14 @@ import net.minecraft.network.FriendlyByteBuf
 //#endif
 import net.minecraft.resources.ResourceLocation
 import net.minecraft.server.level.ServerPlayer
+import net.minecraft.world.entity.player.Player
+
+//#if FORGE
+//$$ import net.minecraftforge.server.permission.events.PermissionGatherEvent
+//#elseif NEOFORGE
+//$$ import net.neoforged.neoforge.server.permission.events.PermissionGatherEvent
+//#endif
+
 import java.nio.file.Path
 
 interface PlatformProxy {
@@ -40,4 +48,10 @@ interface PlatformProxy {
         NetworkManager.sendToPlayer(player, id, buf)
     //#endif
     }
+
+    //#if FORGE-LIKE
+    //$$ fun registerPermissions(event: PermissionGatherEvent.Nodes)
+    //#endif
+
+    fun hasTranscriptPermission(player: Player): Boolean
 }

src/main/kotlin/xyz/bluspring/unitytranslate/PlatformProxyImpl.kt

@@ -1,20 +1,30 @@
 package xyz.bluspring.unitytranslate
 
 //#if FABRIC
+import me.lucko.fabric.api.permissions.v0.Permissions
 import net.fabricmc.api.EnvType
 import net.fabricmc.loader.api.FabricLoader
 //#endif
+import net.minecraft.world.entity.player.Player
 import java.nio.file.Path
 //#if FORGE
 //$$ import net.minecraftforge.api.distmarker.Dist
 //$$ import net.minecraftforge.fml.ModList
 //$$ import net.minecraftforge.fml.loading.FMLLoader
 //$$ import net.minecraftforge.fml.loading.FMLPaths
+//$$ import net.minecraftforge.server.permission.PermissionAPI
+//$$ import net.minecraftforge.server.permission.events.PermissionGatherEvent
+//$$ import net.minecraftforge.server.permission.nodes.PermissionNode
+//$$ import net.minecraftforge.server.permission.nodes.PermissionTypes
 //#elseif NEOFORGE
 //$$ import net.neoforged.api.distmarker.Dist
 //$$ import net.neoforged.fml.ModList
 //$$ import net.neoforged.fml.loading.FMLLoader
 //$$ import net.neoforged.fml.loading.FMLPaths
+//$$ import net.neoforged.neoforge.server.permission.PermissionAPI
+//$$ import net.neoforged.neoforge.server.permission.events.PermissionGatherEvent
+//$$ import net.neoforged.neoforge.server.permission.nodes.PermissionNode
+//$$ import net.neoforged.neoforge.server.permission.nodes.PermissionTypes
 //#endif
 
 class PlatformProxyImpl : PlatformProxy {
@@ -61,4 +71,21 @@ class PlatformProxyImpl : PlatformProxy {
         //$$ return FMLLoader.getDist() == Dist.CLIENT
         //#endif
     }
+
+    // how did Forge manage to overcomplicate permissions of all things
+
+    //#if FORGE-LIKE
+    //$$ val requestTranslationsNode = PermissionNode(UnityTranslate.MOD_ID, "request_translations", PermissionTypes.BOOLEAN, { _, _, _ -> true })
+    //$$ override fun registerPermissions(event: PermissionGatherEvent.Nodes) {
+    //$$    event.addNodes(requestTranslationsNode)
+    //$$ }
+    //#endif
+
+    override fun hasTranscriptPermission(player: Player): Boolean {
+        //#if FABRIC
+        return Permissions.check(player, "unitytranslate.request_translations", true)
+        //#elseif FORGE-LIKE
+        //$$ return PermissionAPI.getOfflinePermission(player.uuid, requestTranslationsNode)
+        //#endif
+    }
 }

src/main/kotlin/xyz/bluspring/unitytranslate/neoforge/UnityTranslateNeoForge.kt

@@ -11,6 +11,7 @@ package xyz.bluspring.unitytranslate.neoforge
 //$$ import net.minecraftforge.fml.common.Mod
 //$$ import net.minecraftforge.fml.event.lifecycle.FMLClientSetupEvent
 //$$ import net.minecraftforge.fml.javafmlmod.FMLJavaModLoadingContext
+//$$ import net.minecraftforge.server.permission.events.PermissionGatherEvent
 //#elseif NEOFORGE
 //$$ import net.neoforged.api.distmarker.Dist
 //$$ import net.neoforged.api.distmarker.OnlyIn
@@ -19,6 +20,7 @@ package xyz.bluspring.unitytranslate.neoforge
 //$$ import net.neoforged.fml.common.Mod
 //$$ import net.neoforged.fml.event.lifecycle.FMLClientSetupEvent
 
+//$$ import net.neoforged.neoforge.server.permission.events.PermissionGatherEvent
 //$$ import net.neoforged.neoforge.client.event.RegisterKeyMappingsEvent
 //#endif
 
@@ -48,6 +50,10 @@ package xyz.bluspring.unitytranslate.neoforge
 //$$         ConfigScreenHelper.createConfigScreen()
 //$$     }
 //$$
+//$$     fun onPermissionsGather(event: PermissionGatherEvent.Nodes) {
+//$$         UnityTranslate.instance.proxy.registerPermissions(event)
+//$$     }
+//$$
 //$$     @OnlyIn(Dist.CLIENT)
 //$$     @SubscribeEvent
 //$$     fun onClientKeybinds(ev: RegisterKeyMappingsEvent) {

src/main/kotlin/xyz/bluspring/unitytranslate/network/UTServerNetworking.kt

@@ -10,7 +10,6 @@ package xyz.bluspring.unitytranslate.network
 //#endif
 import dev.architectury.event.events.common.PlayerEvent
 import dev.architectury.networking.NetworkManager
-import me.lucko.fabric.api.permissions.v0.Permissions
 import net.minecraft.ChatFormatting
 import net.minecraft.network.FriendlyByteBuf
 import net.minecraft.network.chat.Component
@@ -231,7 +230,7 @@ object UTServerNetworking {
     }
 
     fun canPlayerRequestTranslations(player: Player): Boolean {
-        return Permissions.check(player, "unitytranslate.request_translations", true)
+        return UnityTranslate.instance.proxy.hasTranscriptPermission(player)
     }
 
     //#if MC >= 1.20.6