From 6d88d545e9a516bf5e7fed3190fc5680e4d9b908 Mon Sep 17 00:00:00 2001 From: Travis Watkins Date: Sat, 16 Mar 2013 13:14:09 -0500 Subject: Filter data for items that shouldn't have it and filter wool. We used to fall Item.filterData() for this but that method is meant for converting item data to block data during placement and does the wrong thing for this case. Instead we just see if the item should have data and if not set it to zero. We also have to filter wool data explicitly because clients crash when given invalid wool data. --- src/main/java/net/minecraft/server/ItemStack.java | 13 +++++++++++++ src/main/java/net/minecraft/server/PlayerConnection.java | 2 +- 2 files changed, 14 insertions(+), 1 deletion(-) (limited to 'src/main/java/net/minecraft') diff --git a/src/main/java/net/minecraft/server/ItemStack.java b/src/main/java/net/minecraft/server/ItemStack.java index cd6dd070..09b46789 100644 --- a/src/main/java/net/minecraft/server/ItemStack.java +++ b/src/main/java/net/minecraft/server/ItemStack.java @@ -149,6 +149,19 @@ public final class ItemStack { } public void setData(int i) { + // CraftBukkit start - filter out data for items that shouldn't have it + if (!this.usesData()) { + this.damage = 0; + return; + } + + // Filter wool to avoid confusing the client + if (this.id == Block.WOOL.id) { + this.damage = Math.min(15, i); + return; + } + // CraftBukkit end + this.damage = i; if (this.damage < 0) { this.damage = 0; diff --git a/src/main/java/net/minecraft/server/PlayerConnection.java b/src/main/java/net/minecraft/server/PlayerConnection.java index 10fedeb4..13c27051 100644 --- a/src/main/java/net/minecraft/server/PlayerConnection.java +++ b/src/main/java/net/minecraft/server/PlayerConnection.java @@ -1228,7 +1228,7 @@ public class PlayerConnection extends Connection { this.player.a(this.player.activeContainer, arraylist); // CraftBukkit start - send a Set Slot to update the crafting result slot - if(type == SlotType.RESULT && itemstack != null) { + if (type == SlotType.RESULT && itemstack != null) { this.player.playerConnection.sendPacket((Packet) (new Packet103SetSlot(this.player.activeContainer.windowId, 0, itemstack))); } // CraftBukkit end -- cgit v1.2.3