Jump to content
View in the app

A better way to browse. Learn more.

Forge Forums

A full-screen app on your home screen with push notifications, badges and more.

To install this app on iOS and iPadOS
  1. Tap the Share icon in Safari
  2. Scroll the menu and tap Add to Home Screen.
  3. Tap Add in the top-right corner.
To install this app on Android
  1. Tap the 3-dot menu (⋮) in the top-right corner of the browser.
  2. Tap Add to Home screen or Install app.
  3. Confirm by tapping Install.

[Forge 1.11.2] "Dynamic" item lore that shows a (custom) NBT tag?

Featured Replies

Posted

I have an item with a custom NBT tag called "ConsumedAmount". This works, but I also want some form of a 'dynamic' lore, that shows the NBT tag. Like "Amount: >ConsumedAmount<". You can increase the NBT tag (integer) by 1 every time you right click using an overridden onItemRightClick. I've tried to convert the integer into a string and concatenate it with "Amount: " in my addInformation, but it stays at 0.

 

I've tried to change the NBT tag manually to 1000 from the code as a debug inside of the addInformation, and it appears to 'flicker' between 1000 and 0 rapidly. But it doesn't show any changes from outside the addInformation and in the onItemRightClick at all.

 

I've been trying to get this to work for a while now, but no dice. It stays at 0. Any help?

 

    @Override
    public void addInformation(ItemStack par1ItemStack, EntityPlayer par2EntityPlayer, List par3List, boolean par4)
    {
    	int ConsumedAmountNBT = (SelfStack.getTagCompound().getInteger("ConsumedAmount"));
    	
        par3List.add("\u00A77Hold right click to consume \u00A7fIron Ingots\u00A77, and gain buffs.");
        par3List.add("");
        par3List.add("\u00A7eAmount consumed: \u00A76" + Integer.toString(ConsumedAmountNBT));
    }

 

I can send the entire code including the onItemRightClick if necessary (just ask).

6 minutes ago, SnakeBlock said:

int ConsumedAmountNBT = (SelfStack.getTagCompound().getInteger("ConsumedAmount"));

What is SelfStack, where are you getting that from? If you want to add information based on the NBT of the stack in question, you need to get the NBT from the stack in question - the one the method gives you.

  • Author

It's just an item stack of the item the code is running in.

 ItemStack SelfStack = new ItemStack(this);

 

Here's every part of the code that relates to my problem/what I want, in question:

Pastebin

Not to mention, Items are Singletons.

There exists only 1 of each item. Any data stored in the Item class, will be global for each ItemStack containing said Item.

The way you intend to work using this SelfStack will thus not work.

 

You are given the correct ItemStack in almost all possible method parameters (here, it's par1ItemStack¹), either directly, or if need ever be, through the EntityPlayer.

 

¹Please edit your parameter names as well. par1, par2...par27 looks horrible and are easily mistaken for each other by the human eye.

Also previously known as eAndPi.

"Pi, is there a station coming up where we can board your train of thought?" -Kronnn

Published Mods: Underworld

Handy links: Vic_'s Forge events Own WIP Tutorials.

  • Author
2 hours ago, Matryoshika said:

Not to mention, Items are Singletons.

There exists only 1 of each item. Any data stored in the Item class, will be global for each ItemStack containing said Item.

The way you intend to work using this SelfStack will thus not work.

 

You are given the correct ItemStack in almost all possible method parameters (here, it's par1ItemStack¹), either directly, or if need ever be, through the EntityPlayer.

 

¹Please edit your parameter names as well. par1, par2...par27 looks horrible and are easily mistaken for each other by the human eye.

So. I've been playing around with your suggestion, along with some extra help from Google.

https://pastebin.com/Xfh8fp3F

 

I've removed the onUpdate thing and did this instead. Although, Eclipse gives me an error...

"The method onItemRightClick(ItemStack, World, EntityPlayer) of type Item_IronInfusedGemstone must override or implement a supertype method"

 

I know why this is happening - it's because the onItemRightClick line should be "public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer p, EnumHand h)", like it is in the method I'm actually overriding.

 

But this raises the question for me, how would I use a method parameter for the right click along with the addInformation? Just doing "World worldIn, EntityPlayer p, EnumHand h, ItemStack stack" gives me an error.

 

I'd imagine I'm probably talking nonsense right now - apologies, seriously. I just want to get this working. :P

  • Author
5 minutes ago, diesieben07 said:

addInformation already has the stack. In onItemRightClick you can get it from the player based on the hand using getHeldItem.

Ha, can't believe I never thought of that! Thank you all so much, it works perfectly! :D

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...

Important Information

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

Configure browser push notifications

Chrome (Android)
  1. Tap the lock icon next to the address bar.
  2. Tap Permissions → Notifications.
  3. Adjust your preference.
Chrome (Desktop)
  1. Click the padlock icon in the address bar.
  2. Select Site settings.
  3. Find Notifications and adjust your preference.