Hey guys,

 

I've created a  Hammer Item which uses NBT tags to create sub-items for each material it's made of and also has durability, which gets decreased when metal plates are crafted with it. The problem is that the crafting recipe only accepts the undamaged hammer item although i specified it as an ingredient with the Wildcard as data. I looked over my code, but can't find a solution on how to get it to accept hammers of every damage value.

 

ItemHammer.java:

Spoiler

package piggo.industrialization.objects.items;

import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.util.NonNullList;
import piggo.industrialization.Main;
import piggo.industrialization.utils.HammerUtils;
import piggo.industrialization.utils.HammerUtils.EnumHammerMaterial;
import piggo.industrialization.utils.IHasModel;

public class ItemHammer extends Item implements IHasModel{
      
    public ItemHammer(String name, CreativeTabs tab){
        setMaxDamage(0);
        setUnlocalizedName(name);
        setRegistryName(name);       
        setCreativeTab(tab);
    }
    
    @Override
    public void registerModel() {        
        Main.proxy.registerItemRenderer(this, 0, "inventory");
    }
    
    @Override
    public void getSubItems(CreativeTabs tab, NonNullList<ItemStack> items){
        if(this.isInCreativeTab(tab)){
            for(EnumHammerMaterial material : EnumHammerMaterial.values()){
                items.add(HammerUtils.createHammer(new ItemStack(this), material));
            }
        }
    }
    
    @Override
    public String getUnlocalizedName(ItemStack stack){
        return super.getUnlocalizedName(stack) + "." + HammerUtils.getRegistryNameFromNBT(stack);
    }
    
    @Override
    public int getMaxDamage(ItemStack stack){
        return super.getMaxDamage(stack) + HammerUtils.getDurabilityFromNBT(stack);
    }
    
    @Override
    public boolean hasContainerItem(ItemStack stack){
        return true;
    }
    
    @Override
    public ItemStack getContainerItem(ItemStack stack){
        if(stack.getMaxDamage() == stack.getItemDamage()){
            return (ItemStack)null;
        }
        else{
            ItemStack stack2 = stack.copy();
            stack2.setItemDamage(stack.getItemDamage() + 1);
            return stack2;
       }        
    }
    
}

 

JSON recipe:

Spoiler

{
    "type": "minecraft:crafting_shapeless",
    
    "ingredients" : [
        {
            "item": "idtn:hammer",
            "data": 32767
        },
        {
            "item": "idtn:ingot_steel"
        }
    ],
    
    "result": {
        "item": "idtn:plate_steel"
    }
}

 

 

It works that the recipe accepts hammer's with every nbt tag though.

 

Thanks for every helpful answer :)

 

Greets,  MrPiggo

Hmm, okay. I will do so, but is there an other way of keeping the amount of recipes as low as possible? Because when I create an Item for every Hammer I need to create a seperate recipe for each of them and each of my plate items.

Edited April 24, 20196 yr by MrPiggo

Tough.

Sad that there isn't any "short" solution, but at least there is one for my original problem