19972e09b8
5a0150f586ed3eb15fe6f1f596d1a5a7d806f0f9 Fix ITEM_BREAK e6a3911057bd94d8bd7021cbb4923fb84fb106d1 Upstream merge d1cdcf8d4c3639f956474f02ed662517cffbe23e Remove old patch 068df64aeee368377e1673667bffc7a6dcf90554 Rebuild all patches
28 lines
1.2 KiB
Diff
28 lines
1.2 KiB
Diff
From 5bc259a6567c04944c072dd5866eb94606b34741 Mon Sep 17 00:00:00 2001
|
|
From: md_5 <md_5@live.com.au>
|
|
Date: Mon, 18 Mar 2013 20:01:44 +1100
|
|
Subject: [PATCH] Prevent NPE in CraftSign
|
|
|
|
This commit prevents the constructor of CraftSign throwing an NPE when it cannot get the sign tile entity. Instead it will fallback to a 4 empty lined sign, and not try to do anything to those lines on .update().
|
|
|
|
diff --git a/src/main/java/org/bukkit/craftbukkit/block/CraftSign.java b/src/main/java/org/bukkit/craftbukkit/block/CraftSign.java
|
|
index 724dbd1..1b603b2 100644
|
|
--- a/src/main/java/org/bukkit/craftbukkit/block/CraftSign.java
|
|
+++ b/src/main/java/org/bukkit/craftbukkit/block/CraftSign.java
|
|
@@ -17,6 +17,12 @@ public class CraftSign extends CraftBlockState implements Sign {
|
|
|
|
CraftWorld world = (CraftWorld) block.getWorld();
|
|
sign = (TileEntitySign) world.getTileEntityAt(getX(), getY(), getZ());
|
|
+ // Spigot start
|
|
+ if (sign == null) {
|
|
+ lines = new String[]{"", "", "", ""};
|
|
+ return;
|
|
+ }
|
|
+ // Spigot end
|
|
lines = new String[sign.lines.length];
|
|
System.arraycopy(revertComponents(sign.lines), 0, lines, 0, lines.length);
|
|
}
|
|
--
|
|
2.1.0
|
|
|