I want to be able to set the container item for only select metadata items but so far it just adds the container item for all of the metadata items

 

Here is my code:

package ocraftyone.MoreEverything.items;

import java.util.List;

import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.util.NonNullList;
import net.minecraftforge.fml.common.Mod.Metadata;
import ocraftyone.MoreEverything.handlers.EnumHandler.CupcakePanTypes;
import ocraftyone.MoreEverything.init.ModItemBasic;

public class ItemCupcakePan extends ModItemBasic {

public ItemCupcakePan(String name) {
	super(name);
	this.setHasSubtypes(true);
	this.setMaxStackSize(1);
}

@Override
public void getSubItems(Item itemIn, CreativeTabs tab, NonNullList<ItemStack> subItems) {
	for(int i = 0; i < CupcakePanTypes.values().length; i++) {
		subItems.add(new ItemStack(itemIn, 1, i));
	}
}

@Override
public String getUnlocalizedName(ItemStack stack) {
	for(int i = 0; i < CupcakePanTypes.values().length; i++) {
		if(stack.getItemDamage() == i) {
			return this.getUnlocalizedName() + "." + CupcakePanTypes.values()[i].getName();
		}
		else {
			continue;
		}
	}
	return this.getUnlocalizedName() + "." + CupcakePanTypes.EMPTY.getName();
}

@Override
public void addInformation(ItemStack stack, EntityPlayer playerIn, List<String> tooltip, boolean advanced) {
	switch(stack.getItemDamage()) {
	case 0: break;
	case 1: tooltip.add("1/6 Cupcakes"); break;
	case 2: tooltip.add("2/6 Cupcakes"); break;
	case 3: tooltip.add("3/6 Cupcakes"); break;
	case 4: tooltip.add("4/6 Cupcakes"); break;
	case 5: tooltip.add("5/6 Cupcakes"); break;
	case 6: tooltip.add("6/6 Cupcakes"); break;
	case 7: tooltip.add("1/6 Dough"); break;
	case 8: tooltip.add("2/6 Dough"); break;
	case 9: tooltip.add("3/6 Dough"); break;
	case 10: tooltip.add("4/6 Dough"); break;
	case 11: tooltip.add("5/6 Dough"); break;
	case 12: tooltip.add("6/6 Dough"); break;
	}
}

}

 

I want it to be that metadata items 7-12 have the .setContainerItem to be set to ModItems.itemCupcakePan

 

some example code would be greatly appreciated

Use getContainerItem(ItemStack stack)

so what you are saying to do is this:

this.getContainerItem(new ItemStack(ModItems.itemCupcakePan, 1, metadata)

is this right?

so what you are saying to do is this:

this.getContainerItem(new ItemStack(ModItems.itemCupcakePan, 1, metadata)

is this right?

 

No, you need to override

getContainerItem(ItemStack stack)

to return the container item for the passed in

ItemStack

&

hasContainerItem(ItemStack stack)

to return true if the passed in

ItemStack

has a container item.

i am a very visual person. would you mind typing out that block of code with place holders?

@Override
public ItemStack getContainerItem(ItemStack stack) {
     //do something
}

i got that part. i just need help with what to do in the //do something area

...check the stack's metadata and either return a new item stack or not?

i feel like i am being dumb. especially for working on this one problem for many days.

Do you see a problem with my code?

i want metadata 7-12 to have the container item

@Override
public ItemStack getContainerItem(ItemStack itemStack) {
	if(7 > itemStack.getMetadata()) {
		return super.getContainerItem(new ItemStack(ModItems.itemCupcakePan));
	}
	return null;
}

Override

Item#hasContainerItem

to return

true

if your item has a container

Item

. Then in

Item#getContainerItem

return the correct container

Item

for each metadata.

i feel like i am being dumb. especially for working on this one problem for many days.

Do you see a problem with my code?

i want metadata 7-12 to have the container item

@Override
public ItemStack getContainerItem(ItemStack itemStack) {
	if(7 > itemStack.getMetadata()) {
		return super.getContainerItem(new ItemStack(ModItems.itemCupcakePan));
	}
	return null;
}

If 7 is greater than the metadata then it will be 6 and below for one, and if you return super.getContainerItem you are just going to get null. Then override Item#hasContainerItem and make it return true when there is a container item.

Thank you everyone who helped me! Here was my final code:

@Override
public boolean hasContainerItem(ItemStack stack) {
	if(7 > stack.getMetadata()) {
		if(0 != stack.getMetadata()) {
			return true;
		}
	}
	return false;
}

@Override
public ItemStack getContainerItem(ItemStack itemStack) {
	if(7 > itemStack.getMetadata()) {
		if(0 != itemStack.getMetadata()) {
			return new ItemStack(ModItems.itemCupcakePan);
		}
	}
	return null;
}