[1.9] Updating Mod from 1.6 to 1.9 [Unsolved]

[Zodsmar]

I decided to get back into modding along with the 1.9 and I see how much it has changed. I am trying to correct a few issues I have come across but have not been able to figure out how too because of how much has changed.

    public ItemStack getContainerItemStack(ItemStack stack) {
        if (stack == this.itemID) {
            stack.setItemDamage(stack.getItemDamage() + 1);
            return stack;
        } else {
            ItemStack newStack = new ItemStack(this);
            newStack.setItemDamage(newStack.getMaxDamage());
            return newStack;
        }
    }

 

I am trying to update my item so that it has crafting damage when crafted. And I have most of it figured it out but since the changes I cannot figure what to replace itemID with and BlockID with even across my code. Any help would be appreciated. I am sure I will come across many more issues with the code however this is one that has been an issue for a long time and even put me off programming for 1.8

[UberAffe]

when you use GameRegistry#addRecipe you should be giving a new ItemStack as result of the recipe (first parameter in addRecipe). There are three params to the ItemStack constructor, instance of the item, number of items in the stack, and metadata(damage).

[coolAlias]

IDs have been completely removed (well, hidden). You should not have any reason to use them.

 

In your particular case:

if (stack == this.itemID) {

// becomes:

if (stack.getItem() == this) {

[Draco18s]

Also, pretty sure that statement will never return false.

 

//somewhere...

stack new ItemStack(yourItem);

//somewhere else...

stack.getItem().getContainerItem(stack)

 

[Zodsmar]

Thank you guys for the quick replies. I was able to get my issues fixed however I came across a crash now.

 

net.minecraft.util.ReportedException: Rendering screen
at net.minecraft.client.renderer.EntityRenderer.updateCameraAndRender(EntityRenderer.java:1174) ~[EntityRenderer.class:?]
at net.minecraft.client.Minecraft.runGameLoop(Minecraft.java:1135) ~[Minecraft.class:?]
at net.minecraft.client.Minecraft.run(Minecraft.java:401) [Minecraft.class:?]
at net.minecraft.client.main.Main.main(Main.java:118) [Main.class:?]
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.7.0_79]
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.7.0_79]
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.7.0_79]
at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.7.0_79]
at net.minecraft.launchwrapper.Launch.launch(Launch.java:135) [launchwrapper-1.12.jar:?]
at net.minecraft.launchwrapper.Launch.main(Launch.java:28) [launchwrapper-1.12.jar:?]
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.7.0_79]
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.7.0_79]
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.7.0_79]
at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.7.0_79]
at net.minecraftforge.gradle.GradleStartCommon.launch(GradleStartCommon.java:97) [start/:?]
at GradleStart.main(GradleStart.java:26) [start/:?]
Caused by: java.lang.NullPointerException
at net.minecraft.client.renderer.RenderItem.renderItemOverlayIntoGUI(RenderItem.java:425) ~[RenderItem.class:?]
at net.minecraft.client.gui.inventory.GuiContainer.drawSlot(GuiContainer.java:319) ~[GuiContainer.class:?]
at net.minecraft.client.gui.inventory.GuiContainer.drawScreen(GuiContainer.java:118) ~[GuiContainer.class:?]
at net.minecraftforge.client.ForgeHooksClient.drawScreen(ForgeHooksClient.java:309) ~[ForgeHooksClient.class:?]
at net.minecraft.client.renderer.EntityRenderer.updateCameraAndRender(EntityRenderer.java:1147) ~[EntityRenderer.class:?]

 

This crash happens when I try to craft using durability on my tool.

public class ItemCraftingDamage extends Item {
public ItemCraftingDamage() {

	super();
	setMaxStackSize(1);
	setNoRepair();
}

@Override
public boolean hasContainerItem() {
	return true;
}

public ItemStack getContainerItemStack(ItemStack stack) {
	if (stack.getItem() == this) {
		stack.setItemDamage(stack.getItemDamage() + 1);
		return stack;
	} else {
		ItemStack newStack = new ItemStack(this);
		newStack.setItemDamage(newStack.getMaxDamage());
		return newStack;
	}
}

public boolean doesContainerItemLeaveCraftingGrid(ItemStack is) {
	return false;
}

This is the actual code for the durability then I extend my item separately. In the recipes I have

new ItemStack(ItemsAF.hammer, 1, OreDictionary.WILDCARD_VALUE)

(Obviously the *MYITEM* is different in my code it)

However the idea is there. I am not sure as to why it would be causing this crash any help would be appreciated!

 

While I am here I may as well go back to my original question on block ids and item ids and ask for help on fixing on of the main functionalities of my mod. The hammer is my tool which could be used in a crafting table or u could right click it on blocks to see a change in the world. This was working but with all the id issues I have tried rewriting it but I cannot figure out how to get around the ids. Here is the code I have right now:

 

Inside the Hammer class:

@Override
public boolean onItemUse(ItemStack par1ItemStack,
		EntityPlayer par2EntityPlayer, World par3World, int par4, int par5,
		int par6, int par7, float par8, float par9, float par10) {
	int id = par3World.getBlockId(par4, par5, par6);
	int meta = par3World.getBlockMetadata(par4, par5, par6);
	ItemStack output = Recipes.getHammerRecipe(id, meta);

	if(output != null) {
		String paricle = "explode";
		par3World.spawnParticle(paricle, par4, par5 + 1D, par6, 0D, 0D, 0D);
		par3World.spawnParticle(paricle, par4 + 1, par5 + 1D, par6, 0D, 0D, 0D);
		par3World.spawnParticle(paricle, par4 + 1, par5 + 1D, par6 + 1, 0D, 0D, 0D);
		par3World.spawnParticle(paricle, par4, par5 + 1D, par6 + 1, 0D, 0D, 0D);

		par3World.setBlock(par4, par5, par6, output.getItem(), Recipes.getIgroneMeta(id, meta) ? meta : output.getItemDamage(), 2);

		if(!par2EntityPlayer.capabilities.isCreativeMode)
			par1ItemStack.damageItem(2, par2EntityPlayer);

		return true;
	}

	return false;
}

 

Then inside the actual recipe class:

public static HashMap<HammerRecipe, ItemStack> transformBlocks = new HashMap<HammerRecipe, ItemStack>(); 
public static ItemStack getHammerRecipe(int id, int meta) {
	for (Entry<HammerRecipe, ItemStack> entry : transformBlocks.entrySet())
	    if(entry.getKey().id == id && (entry.getKey().meta == meta || entry.getKey().igroneData))
	    	return entry.getValue();
	return null;
}

public static boolean getIgroneMeta(int id, int meta) {
	for (HammerRecipe recipe : transformBlocks.keySet())
	    if(recipe.id == id && (recipe.meta == meta || recipe.igroneData))
	    	return recipe.igroneData;
	return false;
}
public static void addRecipe(HammerRecipe input, HammerRecipe output) {
	if(!input.isItems && !output.isItems)
		transformBlocks.put(input, new ItemStack(output.id, 1, output.meta));

	CraftingManager.getInstance().addShapelessRecipe(new ItemStack(output.id, 1, output.meta),
			new Object[] {new ItemStack(ModItems.hammer, 1, Short.MAX_VALUE), 
		new ItemStack(input.id, 1, input.meta)});
}

public static void addRecipe(HammerRecipe output, HammerRecipe ... input) {
	Object[] objInput = new Object[input.length + 1];
	for(int i = 0; i < input.length; i++) {
		objInput[i] = new ItemStack(input[i].id, 1, input[i].meta);
	}

	objInput[input.length] = new ItemStack(ModItems.hammer, 1, Short.MAX_VALUE);
	CraftingManager.getInstance().addShapelessRecipe(new ItemStack(output.id, 1, output.meta),
			objInput);	
}

public static class HammerRecipe {
	public boolean isItems, igroneData;
	public int id, meta;

	public HammerRecipe(Block Blocks, int meta) {
		this.id = Blocks.BlockID;
		this.meta = meta;
		this.isItems = false;
	}

	public HammerRecipe(Block Blocks) {
		this.id = Blocks.BlockID;
		this.meta = 0;
		this.isItems = false;
	}

	public HammerRecipe(Item Items) {
		this.id = Items.ItemID;
		this.meta = 0;
		this.isItems = true;
	}

	public HammerRecipe(Item Items, int meta) {
		this.id = Items;
		this.meta = meta;
		this.isItems = true;
	}

	public HammerRecipe setIgroneMeta() {
		this.igroneData = true;
		return this;
	}
}

 

So basically I could then create a Hammer Recipe which would now not only work in a crafting table but when I right click for example planks it would convert it to the new plank.

This is what the recipe would look like just for reference:

addRecipe(new HammerRecipe(Blocks.planks, 0), 				new HammerRecipe(Blocks.planks, 1));
	addRecipe(new HammerRecipe(Blocks.planks, 1), 				new HammerRecipe(Blocks.planks, 2));
	addRecipe(new HammerRecipe(Blocks.planks, 2), 				new HammerRecipe(Blocks.planks, 3));
	addRecipe(new HammerRecipe(Blocks.planks, 3), 				new HammerRecipe(Blocks.planks, 0));

 

I hope what I am trying to do even makes sense. In 1.6 I had all this working and really would love to fix it up and expand with the many more ideas I have for the mod.

(But again you can see how before with planks it was all meta data so I am not even sure if this is possible anymore the way I want to do it. If it is not do you have any suggestions?)

[Zodsmar]

Anyone got any idea's?

[Zodsmar]

Do people not realize I asked another question? Like should I make a new thread? And up 

[coolAlias]

Do people not realize I asked another question? Like should I make a new thread? And up 

People probably do but either, don't know the answer and don't have time, inclination, or ability to research the answer, or they know the answer but haven't had the time to reply.

 

Personally, I fall into the first category since I'm using a work computer without Forge installed