Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[1.20.1] EvilCraft & GTCEu-Modern Potion Bottle Added NBT #1084

Open
loving2 opened this issue Jan 8, 2025 · 4 comments
Open

[1.20.1] EvilCraft & GTCEu-Modern Potion Bottle Added NBT #1084

loving2 opened this issue Jan 8, 2025 · 4 comments

Comments

@loving2
Copy link

loving2 commented Jan 8, 2025

Issue type:

  • 🐛 Bug

Short description:

Installing both GTCEu-Modern v1.6.2 and EvilCraft 1.2.49 results in Minecraft:Potion items getting an additional NBT on them. This causes issues with any other mod that adds crafting recipes utilizing a Minecraft:Potion because they expect the one NBT specifying the Potion type.

Steps to reproduce the problem:

  1. Install GTCEu-Modern 1.6.2, EvilCraft 1.2.49, JEI v15.20.0.105, and for example Draconic Evolution v3.1.2.604
  2. Start a world in Creative
  3. Use JEI to bookmark the ingredients for the Damage Module from Draconic Evolution
  4. Give yourself the ingredients
  5. Hit F3+H and confirm that there are 2 NBT tags on the strength potion
  6. Place a Crafting Table down and try to craft the Damage Module

Expected behaviour:

No added NBT to a Minecraft:Potion item so that crafting recipes using the potion still work


Versions:

  • This mod: 1.2.49
  • GTCEu-Modern: 1.6.2
  • Minecraft: 1.20.1
  • Mod loader version: Forge 47.3.11

Additional Information:

You don't need to install JEI or Draconic Evolution to reproduce this. Having just GTCEu-Modern 1.6.2 and EvilCraft 1.2.49 is enough to see that potions have an additional NBT stored on them.

@rubensworks
Copy link
Member

Thanks for reporting!

@rubensworks
Copy link
Member

AFAIK, EvilCraft does not add any tags to potions.
Can you report this to the GTCEu-Modern issue tracker and link back here?
Happy to make changes here if the GTCEu-Modern mod author considers it relevant.

@rubensworks rubensworks moved this from To Do to On hold (awaiting input) in Maintenance Jan 9, 2025
@loving2
Copy link
Author

loving2 commented Jan 9, 2025

Sure thing - I reported it here first since it doesn't happen with just GTCEu-Modern installed and appears to be a mod interaction, but I'll make an issue there too. Issue here

GTCEu-Modern recently added functionality where Potions are now a Fluid, and my thought was the additional NBT could be getting added so the Dark Tank would work properly with it.

@loving2
Copy link
Author

loving2 commented Jan 13, 2025

What causes the poison bottle to have the NBT Fluid: {FluidName: "evilcraft:poison", Amount: 1000}? With just EvilCraft installed and no other mods. JEI shows a poison bottle with no NBT, cheating one into your inventory changes it to have that additional NBT.

This is what the added NBT looks like on the potions when GTCEu-Modern is also installed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: On hold (awaiting input)
Development

No branches or pull requests

2 participants