Jump to content

Recommended Posts

Posted

Hopefully my last thread as this is the last TODO on my list before release..

 

NBT on items seems to be messed up for creative mode. It doesn't affect the host of the server, but for anyone else, switching to creative mode with /gamemode c seems to reset all the NBT of the items in their inventory. In particular, this happens only when they can see the item in their inventory.

 

So in creative mode, when you press the "inventory button" it shows a big list of items for you to grab and hides your inner inventory, but shows your hotbar. Things on the hotbar have their NBT reset, but things in the inner inventory are fine.. until you switch to look at the inner inventory, and then they get their NBT reset. Also, it seems like this NBT reset happens so frequently that items on the hotbar gets stuck in the "use item" animation, making items with NBT essentially unusable in creative. 

 

I'm using  getNBTShareTag and readNBTShareTag to sync capabilities, like this, cause I'm showing the data in the tooltip: 

 

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

@Override
public NBTTagCompound getNBTShareTag(ItemStack stack) {
    IGemMahou gemMahou = stack.getCapability(GemMahouProvider.MAHOU, null);
    NBTBase nbt = GemMahouProvider.MAHOU.getStorage().writeNBT(GemMahouProvider.MAHOU, gemMahou, null);
    NBTTagCompound ret = super.getNBTShareTag(stack);
    if (ret == null) ret = new NBTTagCompound();
    ret.setTag("mahou", nbt);
    return ret;
}

@Override
public void readNBTShareTag(ItemStack stack, NBTTagCompound nbtt) {
    if (nbtt != null && nbtt.hasKey("mahou")) {
        IGemMahou gemMahou = stack.getCapability(GemMahouProvider.MAHOU, null);
        IGemMahou serverGemMahou = new GemMahou();
        GemMahouProvider.MAHOU.getStorage().readNBT(GemMahouProvider.MAHOU, serverGemMahou, null, nbtt.getTag("mahou"));
        if (gemMahou != null)
            gemMahou.setStoredMana(serverGemMahou.getStoredMana());
    }
}

 

I've only noticed this on items which have the ShareTag, but basically all my items which have NBT show it in the tooltip, so it may be unrelated, too.

 

I've found only one thing which looks similar to my problem here:

https://github.com/MinecraftForge/MinecraftForge/issues/4580

 

But this seems to be much less of a problem than mine. Even opening creative at all resets NBT, not just moving items around.

 

Any ideas on what can be done to fix it? 

Posted
42 minutes ago, stepsword said:

I've found only one thing which looks similar to my problem here:

The solution to your problem is to not modify the NBTTagCompound in getShareTag. Just send all of the data. There was a PR here, but it seems to have been forgotten about. Maybe making a comment on the PR and tagging Mezz in it will bring it back to their attention. Though the problem isn't that big of one, because in most cases creative mode won't be used.

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Posted (edited)
15 minutes ago, Animefan8888 said:

The solution to your problem is to not modify the NBTTagCompound in getShareTag. Just send all of the data. There was a PR here, but it seems to have been forgotten about. Maybe making a comment on the PR and tagging Mezz in it will bring it back to their attention. Though the problem isn't that big of one, because in most cases creative mode won't be used.

When you say send all of the data do you mean with packets or just something like this?

 

Edit: don't know what happened to my last code, seems like it all disappeared

@Override
public NBTTagCompound getNBTShareTag(ItemStack stack) {
    IGemMahou gemMahou = stack.getCapability(GemMahouProvider.MAHOU, null);
    NBTBase nbt = GemMahouProvider.MAHOU.getStorage().writeNBT(GemMahouProvider.MAHOU, gemMahou, null);
    return nbt;
}
Edited by stepsword
Posted
Just now, stepsword said:

@Override public getNBTShareTagMAHOU, null; MAHOUMAHOU, , null; ;

This won't even compile so I definitely didn't mean anything like that.Just don't override getNBTShareTag or getShareTag. That way all of the NBT will be sent to the client. Then you can override readNBTShareTag and use it to update your ItemStack's capability.

  • Thanks 1

VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING

I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect.

Forge and vanilla BlockState generator.

Posted
1 minute ago, Animefan8888 said:

This won't even compile so I definitely didn't mean anything like that.Just don't override getNBTShareTag or getShareTag. That way all of the NBT will be sent to the client. Then you can override readNBTShareTag and use it to update your ItemStack's capability.

Sorry - I tried pasting/editing from the forum and it came out like that. I see what you mean now though - I didn't realize NBT was shared whether or not I overrode getNBTShareTag. Thanks for the help with this!

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



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • @Tsuk1 Also, new note, you can use blockbench to make the custom item model for when it is not on the head.   EDIT: Funny story, I am making a mod similar to yours! Mine is called NorseMC.
    • @Nood_dev Could you send a screenshot of your weapon code? Here is the one I made (for a dagger): The specific UUID does not matter, just that it is the same every time, which is why UUID#randomUUID does not work public class DaggerItem extends TieredItem implements Vanishable { protected static final double REACH_MODIFIER = -1.5D; protected final Multimap<Attribute, AttributeModifier> defaultModifiers; protected final UUID BASE_ATTACK_REACH_UUID = UUID.fromString("6fe75b5c-9d1b-4e83-9eea-a1d5a94e8dd5") public DaggerItem(Tier pTier, int pAttackDamageModifier, float pAttackSpeedModifier, Properties pProperties) { super(pTier, pAttackDamageModifier, pAttackSpeedModifier, pProperties); this.attackDamage = (float) pAttackDamageModifier + pTier.getAttackDamageBonus(); ImmutableMultimap.Builder<Attribute, AttributeModifier> builder = ImmutableMultimap.builder(); builder.put(Attributes.ATTACK_DAMAGE, new AttributeModifier(BASE_ATTACK_DAMAGE_UUID, "Weapon modifier", this.attackDamage, AttributeModifier.Operation.ADDITION)); builder.put(Attributes.ATTACK_SPEED, new AttributeModifier(BASE_ATTACK_SPEED_UUID, "Weapon modifier", pAttackSpeedModifier, AttributeModifier.Operation.ADDITION)); // THE ONE YOU WANT: builder.put(ForgeMod.ENTITY_REACH.get(), new AttributeModifier(BASE_ATTACK_REACH_UUID, "Weapon modifier", REACH_MODIFIER, AttributeModifier.Operation.ADDITION)); this.defaultModifiers = builder.build(); } @Override public Multimap<Attribute, AttributeModifier> getDefaultAttributeModifiers(EquipmentSlot pEquipmentSlot) { return pEquipmentSlot == EquipmentSlot.MAINHAND ? this.defaultModifiers : super.getDefaultAttributeModifiers(pEquipmentSlot); } }
    • https://images.app.goo.gl/1PxFKdxByTgkxvSu6
    • That's what we'll try out. I could never figure out how to recreate the crash, so I'll just have to wait and see.
    • Ok, I updated to the latest version and now the models are visible, the problem now is that the glowing eyes are not rendered nor any texture I render there when using shaders, even using the default Minecraft eyes RenderType, I use entityTranslucent and entityCutout, but it still won't render. Something I noticed when using shaders is that a texture, instead of appearing at the world position, would appear somewhere on the screen, following a curved path, it was strange, I haven't been able to reproduce it again. I thought it could be that since I render the texture in the AFTER ENTITIES stage which is posted after the batches used for entity rendering are finished, maybe that was the reason why the render types were not being drawn correctly, so I tried injecting code before finishing the batches but it still didn't work, plus the model was invisible when using shaders, there was a bug where if I look at the model from above it is visible but if I look at it from below it is invisible. So in summary, models are now visible but glowing eyes and textures are not rendered, that hasn't changed.
  • Topics

×
×
  • Create New...

Important Information

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