Tech Reborn

Tech Reborn

30M Downloads

Nei Handler does not show correct amount of forestry wood

HarrisFauntleroy opened this issue ยท 8 comments

commented

The Noteblock recipe for example, in the assembling machine it normally requires 8 planks and one redstone, though wood from Forestry, which is ore dict correctly, only requires 1, despite the fact that its still 4 planks from a log like any other wood.

First pic shows Extra Trees wood, it asks for 8, same for Vanilla wood
2015-07-20_22 39 51

Second shows Forestry wood., it only asks for 1
2015-07-20_22 39 54

Lastly, this shows both wood types are ore dictionary correctly.
2015-07-20_22 41 14

commented

Are you saying it works fine with forestry but not extra trees?

commented

Wasn't sure at first, but its Forestry wood that is working incorrectly.

commented

ok thanks

commented

hmm It it uses the ore dict so im not sure whats wrong

commented

It should work fine, RecipeHandler.addRecipe(new AssemblingMachineRecipe(new ItemStack(Blocks.planks, 8), new ItemStack(Items.redstone, 1), new ItemStack(Blocks.noteblock), 120, 5));

commented

This appears to only be an issue with the nei handler

commented

I found this and this is bad:
https://github.com/TechReborn/TechReborn/blob/master/src/main/java/techreborn/util/ItemUtils.java#L109

You override the oredictionary. Please copy the itemstack instead of changing the name of the orereference! This could be also your bug...

commented

Oh, :) that looks like it! Let me test