From af2e88fbb9e0dc7ab735952482aa2db714d54dc5 Mon Sep 17 00:00:00 2001
From: Shane Freeder <theboyetronic@gmail.com>
Date: Wed, 17 Apr 2019 00:53:55 +0100
Subject: [PATCH] Fix NPE from sign placement (Closes #1963) (Fixes #1917)

This fixes issues with upstreams changes to solve a private issue on
their side, as signs are placed, they may replace existing blocks, e.g.
grass, which breaks upstreams assumption that the sign is always placed
adjacent to a surface
---
 .../Fix-NPE-from-sign-placement.patch         | 53 +++++++++++++++++++
 1 file changed, 53 insertions(+)
 create mode 100644 Spigot-Server-Patches/Fix-NPE-from-sign-placement.patch

diff --git a/Spigot-Server-Patches/Fix-NPE-from-sign-placement.patch b/Spigot-Server-Patches/Fix-NPE-from-sign-placement.patch
new file mode 100644
index 000000000..8e5b3a1c6
--- /dev/null
+++ b/Spigot-Server-Patches/Fix-NPE-from-sign-placement.patch
@@ -0,0 +1,53 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Shane Freeder <theboyetronic@gmail.com>
+Date: Wed, 17 Apr 2019 00:48:59 +0100
+Subject: [PATCH] Fix NPE from sign placement
+
+This fixes issues with upstreams changes to solve a private issue on
+their side, as signs are placed, they may replace existing blocks, e.g.
+grass, which breaks upstreams assumption that the sign is always placed
+adjacent to a surface
+
+diff --git a/src/main/java/net/minecraft/server/EntityHuman.java b/src/main/java/net/minecraft/server/EntityHuman.java
+index f5d9b4abc2..fdbe9a2adc 100644
+--- a/src/main/java/net/minecraft/server/EntityHuman.java
++++ b/src/main/java/net/minecraft/server/EntityHuman.java
+@@ -0,0 +0,0 @@ public abstract class EntityHuman extends EntityLiving {
+     public EntityFishingHook hookedFish;
+     // Paper start
+     public boolean affectsSpawning = true;
++    public BlockPosition openingSign = null; // Paper - fix NPE when opening signs
+     // Paper end
+     // Paper start - Player view distance API
+     private int viewDistance = -1;
+diff --git a/src/main/java/net/minecraft/server/ItemSign.java b/src/main/java/net/minecraft/server/ItemSign.java
+index 11045ee1e1..7808aed0c0 100644
+--- a/src/main/java/net/minecraft/server/ItemSign.java
++++ b/src/main/java/net/minecraft/server/ItemSign.java
+@@ -0,0 +0,0 @@ public class ItemSign extends ItemBlockWallable {
+         if (!world.isClientSide && !flag && entityhuman != null) {
+             // CraftBukkit start - SPIGOT-4678
+             // entityhuman.openSign((TileEntitySign) world.getTileEntity(blockposition));
++            entityhuman.openingSign = blockposition; // Paper - fix NPE when opening signs
+             ItemSign.openSign = true;
+             // CraftBukkit end
+         }
+diff --git a/src/main/java/net/minecraft/server/ItemStack.java b/src/main/java/net/minecraft/server/ItemStack.java
+index eb130c0121..993de2bded 100644
+--- a/src/main/java/net/minecraft/server/ItemStack.java
++++ b/src/main/java/net/minecraft/server/ItemStack.java
+@@ -0,0 +0,0 @@ public final class ItemStack {
+                     // SPIGOT-4678
+                     if (this.item instanceof ItemSign && ItemSign.openSign) {
+                         ItemSign.openSign = false;
+-                        entityhuman.openSign((TileEntitySign) world.getTileEntity(new BlockActionContext(itemactioncontext).getClickPosition()));
++                        // Paper start - fix NPE when opening signs
++                        if (entityhuman.openingSign != null) {
++                            entityhuman.openSign((TileEntitySign) world.getTileEntity(entityhuman.openingSign));
++                            entityhuman.openingSign = null;
++                        }
++                        // Paper end
+                     }
+ 
+                     // SPIGOT-1288 - play sound stripped from ItemBlock
+--
\ No newline at end of file