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

Device Linker not safe to add to Morph-o-Tool/Can't rename Device Linker #13

Open
DoomRater opened this issue Dec 5, 2020 · 1 comment
Labels
bug Something isn't working

Comments

@DoomRater
Copy link

Issue Description:

The Device Linker does not modify its NBT stack, instead choosing to recreate it from scratch when in use, resulting in NBT data that the Morph-o-Tool uses to keep track of something being inside of a Morph-o-Tool (or Akashic Tome, if you're crazy enough) being deleted. The rest of the contents of the Morph-o-Tool will be deleted in this case.

This also has the side effect of making it impossible to rename the Device Linker, as it deletes the NBT tag that contains its name!

Steps to reproduce:

  1. Add thuttech:devicelinker to whitelist of Morph-o-Tool config list
  2. Craft a Morphing Tool with a device linker
  3. Start using the Device Linker to make an elevator
  4. Poof, no more Morph-o-Tool!
    

Affected Versions:

While this is noticed on 1.12.2 versions of the mod, specifically ThutCore 5.21.0 and Elevators 6.1.6, I'd be surprised if anything has changed about NBT handling of the item in later versions where Morph-o-Tool is also available. Only other note is that I'm speicifcally using Akashic Tome, but the code is over 90% the same, but since this affects renaming the item as well...

@DoomRater DoomRater added the Unverified needs triage to determine if is actually a bug label Dec 5, 2020
@Thutmose
Copy link
Contributor

Thutmose commented Dec 8, 2020

Hmm, I will look into a better way to do the removal of the tag then, I was simply removing the entire tag, as that was the easiest way to do it, also for the names, the item re-names itself when linked vs unlinked.

@Thutmose Thutmose added bug Something isn't working and removed Unverified needs triage to determine if is actually a bug labels Dec 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants