Fix NPE in CraftBanner due to null world, potentially other things fixed

Banners only load color if the world is set. I don't know why...

For some reason, the world was not set on these, so it was changing behavior.

So if we want an accurate clone, world needs to be set.
This commit is contained in:
Aikar 2018-08-15 01:21:10 -04:00
parent c235889d4a
commit 061a290937

View File

@ -0,0 +1,20 @@
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: Aikar <aikar@aikar.co>
Date: Wed, 15 Aug 2018 01:19:04 -0400
Subject: [PATCH] Pass World to Tile Entity Snapshots
diff --git a/src/main/java/org/bukkit/craftbukkit/block/CraftBlockEntityState.java b/src/main/java/org/bukkit/craftbukkit/block/CraftBlockEntityState.java
index d4d9c5fc50..a72bbdbedb 100644
--- a/src/main/java/org/bukkit/craftbukkit/block/CraftBlockEntityState.java
+++ b/src/main/java/org/bukkit/craftbukkit/block/CraftBlockEntityState.java
@@ -0,0 +0,0 @@ public class CraftBlockEntityState<T extends TileEntity> extends CraftBlockState
if (DISABLE_SNAPSHOT) {
this.snapshot = this.tileEntity;
} else {
- this.snapshot = this.createSnapshot(this.tileEntity, null);
+ this.snapshot = this.createSnapshot(this.tileEntity, tileEntity.getWorld());
}
// copy tile entity data:
if(this.snapshot != null) {
--