{SOLVED}Help with ContainerItem

[RedBullSlurpie]

In 1.8.9 I had the code listed in pastebin below

 

http://pastebin.com/Xha5TMKP

 

That returned the Blender item to my inventory so people did not have to re-craft it over and over again. However in 1.9, it's not working, when you use it, once taking said item from crafting, say the blender and an apple make Apple Juice, once taking the Apple Juice, the blender would have a 0 in the bottom of the texture of it, and once closed, the item would be consumed and not returned. How can I fix that in 1.9? Thanks.

[danielm59]

I use this function

    
    public ItemStack getContainerItem(ItemStack itemStack)
    {
        
        ItemStack copiedStack = itemStack.copy();
        
        copiedStack.stackSize = 1;
        
        return copiedStack;
        
    }

[RedBullSlurpie]

I use this function

    
    public ItemStack getContainerItem(ItemStack itemStack)
    {
        
        ItemStack copiedStack = itemStack.copy();
        
        copiedStack.stackSize = 1;


So this should work on 1.9?
        
        return copiedStack;
        
    }

[Draco18s]

I use this function

    
    public ItemStack getContainerItem(ItemStack itemStack) {
        ItemStack copiedStack = itemStack.copy();
        copiedStack.stackSize = 1;
        return copiedStack;
    }

 

Why are you cloning the input stack, setting its size to 1, and returning it?

 

If its size wasn't already 1 you just destroyed (an unknown number of) items.  If it was, there was no point in cloning and setting the size.

[RedBullSlurpie]

Never call setMaxStackSize after the item has been registered.

 

Okay, but how would I get this to work on 1.9?

[Draco18s]

@Override
public ItemStack getContainerItem(ItemStack itemStack) {
    return itemStack;
}

[RedBullSlurpie]

@Override
public ItemStack getContainerItem(ItemStack itemStack) {
    return itemStack;
}

 

This method doesn't work in 1.9 for me, when I did the copied one that the other person shared with me, it seems to work, so ill stick with that one.

[coolAlias]

That's strange that it requires a copy to be made, but that still doesn't negate the fact that you are reducing a stack of unknown size to a size of 1 for no apparent reason.

 

Why not just return the copy directly?

@Override
public ItemStack getContainerItem(ItemStack itemStack) {
    return itemStack.copy();
}

[RedBullSlurpie]

That's strange that it requires a copy to be made, but that still doesn't negate the fact that you are reducing a stack of unknown size to a size of 1 for no apparent reason.

 

Why not just return the copy directly?

@Override
public ItemStack getContainerItem(ItemStack itemStack) {
    return itemStack.copy();
}

 

 

Doesn't work in 1.9, for me anyways, this seems to be the only method that works for me in 1.9, so not gonna question it.