Jump to content

[Solved] [1.18.2] Overriding getContainerItem not working


RuthlessNail

Recommended Posts

I tried to implement a item that can be used multiple time in a crafting recipe, however, each usage will cause the item to lose 1 durability.

In my particular example, I made a hammer that can be used with four coal to craft compacted coal, I attempted to implement this by overriding the getContainerItem method, where I basically applied one damage to the item, then returned the damaged item. To my knowledge this should make the damaged item as the container, and that will remain on the crafting grid after crafting is completed, however in practice this implementation doesn't work, and the hammer will still be consumed along with the other ingredients.

This is my current code for the WoodenHammer class (excluding the imports):

public class ItemWoodenHammer extends Item {

    public ItemWoodenHammer(Properties pProperties) {
        super(pProperties.durability(60));

    }

    @Override
    public boolean isDamageable(ItemStack stack) {
        return true;
    }

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



    @Override
    public ItemStack getContainerItem(ItemStack itemStack) {
        itemStack.hurt(1, new Random(), null);
        System.out.println("hammer max durability:" + itemStack.getMaxDamage());
        System.out.println("hammer's current state: " + itemStack);
        return itemStack;
    }

}

Here's the compactcoalfromwoodenhammer.json for the recipe:

{
  "type": "minecraft:crafting_shapeless",
  "ingredients": [
    {
      "item": "ornithophobia:wooden_hammer"
    },
    {
      "item": "minecraft:coal"
    },
    {
      "item": "minecraft:coal"
    },
    {
      "item": "minecraft:coal"
    },
    {
      "item": "minecraft:coal"
    }
  ],
  "result": {
    "item": "ornithophobia:compacted_coal"
  }
}

As you may see I tried to print out the hammer's max durability and the current state, since I initially suspect either I messed up assigning the durability to the hammer or I messed up with the item hurt method, but the print output suggest that hammer's max durability is 60, suggesting it's applied correctly, and that 1 damage was inflicted on the itemStack when crafting occurred. In addition, the print outputted at all suggest that the getContainerItem method is triggering when the crafting happened, yet still, nothing is left behind on the crafting grid, suggesting that the returning part was somehow broken, and yet I can't figure out how. 

I am just starting out modding with forge literally two days ago so my implementation of these functions are probably pretty dumb and I probably made some real silly mistake, but I have been trying to debug this for hours and it's seriously giving me a brain aneurysm. Any amount of help/insight is very much appreciated, huge thanks in advance.

Edited by RuthlessNail
editing the title to reflect that the problem have been solved
Link to comment
Share on other sites

I don't think its documentated, but you need to return a new ItemStack not the original.

Something like: 

    @Override
    public ItemStack getContainerItem(ItemStack itemStack) {
        // Copy the original
        var result = itemStack.copy();
        // Damage it
        result.hurtAndBreak(1, ForgeHooks.getCraftingPlayer(), player -> {
            if (player != null) {
                player.level.playSound(null, player.getX(), player.getY(), player.getZ(), ITEM_BREAK, player.getSoundSource(), 1.0f, 1.0f);
            }
        });
        return result;
    }

 

Edited by warjort

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com  You should also read the support forum sticky post.

Link to comment
Share on other sites

17 minutes ago, warjort said:

I don't think its documentated, but you need to return a new ItemStack not the original.

Something like: 

    @Override
    public ItemStack getContainerItem(ItemStack itemStack) {
        // Copy the original
        var result = itemStack.copy();
        // Damage it
        result.hurtAndBreak(1, ForgeHooks.getCraftingPlayer(), player -> {
            if (player != null) {
                player.level.playSound(null, player.getX(), player.getY(), player.getZ(), ITEM_BREAK, player.getSoundSource(), 1.0f, 1.0f);
            }
        });
        return result;
    }

 

Thank you sir, you are a literal life-saver. This works flawlessly.

Link to comment
Share on other sites

  • RuthlessNail changed the title to [Solved] [1.18.2] Overriding getContainerItem not working

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.