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

Injected records in overhauls are not copied as standalone #95

Closed
focustense opened this issue Aug 17, 2021 · 1 comment
Closed

Injected records in overhauls are not copied as standalone #95

focustense opened this issue Aug 17, 2021 · 1 comment
Labels
bug Something isn't working easynpc Issues/requests relating the EasyNPC app

Comments

@focustense
Copy link
Owner

This was demonstrated with Citizens of Tamriel, and CoT-Visual Overhaul. The output in xEdit looks like:

Record

The overhaul proved to have invalid overrides (aka "injected records"):

Overhaul

Needs some testing to determine exactly what happened, but my guess is that EasyNPC thought that the record actually came from the master (maybe because it looks at FormKey.ModKey) rather than the overhaul, and therefore didn't copy the record, just referenced the false master which became an invalid reference once the overhaul was deactivated.

If the above is the case, then the Record Importer may need some tweaking in the "master contains" logic to check that the master actually really contains that record.

@focustense focustense added bug Something isn't working easynpc Issues/requests relating the EasyNPC app labels Aug 17, 2021
@focustense
Copy link
Owner Author

This was also shown to happen in the 3DNPC Visual Overhaul, which is actually one of the mods with official mugshots.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working easynpc Issues/requests relating the EasyNPC app
Projects
None yet
Development

No branches or pull requests

1 participant