From 76d95fe7edabcb7ac6138655877579c3a88851d9 Mon Sep 17 00:00:00 2001
From: Jishuna <joshl5324@gmail.com>
Date: Sat, 17 Feb 2024 14:02:38 +1100
Subject: [PATCH] SPIGOT-7417, #1343: Add Sign#getTargetSide and
 Sign#getAllowedEditor

---
 .../bukkit/craftbukkit/block/CraftSign.java   | 23 +++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/src/main/java/org/bukkit/craftbukkit/block/CraftSign.java b/src/main/java/org/bukkit/craftbukkit/block/CraftSign.java
index 4f45365df..19830d02a 100644
--- a/src/main/java/org/bukkit/craftbukkit/block/CraftSign.java
+++ b/src/main/java/org/bukkit/craftbukkit/block/CraftSign.java
@@ -1,8 +1,10 @@
 package org.bukkit.craftbukkit.block;
 
 import com.google.common.base.Preconditions;
+import java.util.UUID;
 import net.minecraft.network.chat.IChatBaseComponent;
 import net.minecraft.world.level.block.entity.TileEntitySign;
+import org.bukkit.Bukkit;
 import org.bukkit.DyeColor;
 import org.bukkit.World;
 import org.bukkit.block.Sign;
@@ -93,6 +95,27 @@ public class CraftSign<T extends TileEntitySign> extends CraftBlockEntityState<T
         }
     }
 
+    @Override
+    public SignSide getTargetSide(Player player) {
+        ensureNoWorldGeneration();
+        Preconditions.checkArgument(player != null, "player cannot be null");
+
+        if (getSnapshot().isFacingFrontText(((CraftPlayer) player).getHandle())) {
+            return front;
+        }
+
+        return back;
+    }
+
+    @Override
+    public Player getAllowedEditor() {
+        ensureNoWorldGeneration();
+
+        // getPlayerWhoMayEdit is always null for the snapshot, so we use the wrapped TileEntity
+        UUID id = getTileEntity().getPlayerWhoMayEdit();
+        return (id == null) ? null : Bukkit.getPlayer(id);
+    }
+
     @Override
     public DyeColor getColor() {
         return front.getColor();