Fix stacking for items after setting empty meta. Fixes BUKKIT-5331

ItemStacks do not stack if one has null for a tag, while the other has an
empty tag. In CraftItemStack, if you set an item to an empty ItemMeta, it
will create an empty tag for the internal ItemStack.

This changes the setItemMeta function to check for empty meta, and then
use null for the tag instead of an empty NBTTagCompound.

By: Wesley Wolfe <weswolf@aol.com>
This commit is contained in:
CraftBukkit/Spigot 2014-01-23 23:02:40 -06:00
parent ba1adfb79b
commit d04ca26007

View file

@ -360,7 +360,7 @@ public final class CraftItemStack extends ItemStack {
if (item == null) {
return false;
}
if (itemMeta == null) {
if (CraftItemFactory.instance().equals(itemMeta, null)) {
item.tag = null;
return true;
}