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

Cannot equip items with NBT in turtles #106

Closed
osmarks opened this issue Feb 9, 2019 · 4 comments
Closed

Cannot equip items with NBT in turtles #106

osmarks opened this issue Feb 9, 2019 · 4 comments
Labels
bug A problem or unexpected behaviour with the mod. wontfix A bug which I won't fix or an enhancement which won't be implemented.

Comments

@osmarks
Copy link
Contributor

osmarks commented Feb 9, 2019

Minecraft 1.12.2, whatever CC: Tweaked is on Switchcraft.

It seems that if an item has any NBT, it can't be equipped on turtles.
To reproduce: name a laser in an anvil, attempt to equip it on a turtle - it doesn't work.

@SquidDev SquidDev added bug A problem or unexpected behaviour with the mod. invalid Issues caused by user error, another mod, etc... labels Feb 9, 2019
@SquidDev
Copy link
Member

SquidDev commented Feb 9, 2019

Working as intended.

@SquidDev SquidDev closed this as completed Feb 9, 2019
@osmarks
Copy link
Contributor Author

osmarks commented Feb 9, 2019

Um... why? This causes significant problems with stuff like introspection modules, which keep NBT data forever if bound.

@SquidDev SquidDev added wontfix A bug which I won't fix or an enhancement which won't be implemented. and removed invalid Issues caused by user error, another mod, etc... labels Feb 9, 2019
@SquidDev
Copy link
Member

SquidDev commented Feb 9, 2019

Introspection modules keeping NBT data when they shouldn't is a separate bug, and already has an issue filed on the plethora repo.

As far as changing NBT by enchanting, setting the label, etc... - this has always been a limitation of CC. Equipping and then unequipping an item should give leave you with the original, and that is no longer the case when you start ignoring NBT.

@osmarks
Copy link
Contributor Author

osmarks commented Feb 9, 2019

Maybe it should keep the NBT instead of ignoring it, then? Or is that not possible?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A problem or unexpected behaviour with the mod. wontfix A bug which I won't fix or an enhancement which won't be implemented.
Projects
None yet
Development

No branches or pull requests

2 participants