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

[TES4] Copy as overrides fails for MGEF #788

Open
MacSplody opened this issue Mar 2, 2020 · 8 comments
Open

[TES4] Copy as overrides fails for MGEF #788

MacSplody opened this issue Mar 2, 2020 · 8 comments

Comments

@MacSplody
Copy link

What is the version of xEdit you are using?

TES4Edit 4.0.3 (1CDF3AC7)

Describe the bug
When using "Copying as Override" on a MGEF record, an error message will show for an assertion. The record is created in the overriding module, but it has no content.

To reproduce
Steps to reproduce the behavior:
Go to Oblivion.esm (Or any module with MGEF records).
Right click any MGEF record and select copy as Override.
Target module can be new or existing plugin.
After hitting confirm, you'll then receive an error message.
Assertion failure (P:\TES5Edit\wbImplementation.pas, line 15543).
Open the newly created override record.
You should now see a record containing only the signature and formid.
The rest of its content will be empty.

Expected behavior
The new record should match the original record selected for "Copy as Override".

Screenshots
image

image

Additional context

Logs.zip

@MacSplody
Copy link
Author

Just to add to this, if I make a new record.
The only sub records I can't add to are EDID - Magic Effect Code and ESCE - Counter Effects.

image

@ElminsterAU
Copy link
Collaborator

Oblivion does not support adding new MGEF. The only way for new MGEF is using OBME (oblivion magic extender). Such new MGEF are not identified by a 4 character EDID but with magic effects integer. So not being able to add an EDID to an MGEF record with OBME header is normal.

@MacSplody
Copy link
Author

Oblivion does not support adding new MGEF. The only way for new MGEF is using OBME (oblivion magic extender). Such new MGEF are not identified by a 4 character EDID but with magic effects integer. So not being able to add an EDID to an MGEF record with OBME header is normal.

Could the user be prevented from trying to add new MGEF or informed of that limitation?

@ElminsterAU
Copy link
Collaborator

Well, xEdit DOES support OBME (at least it's supposed to), so adding new MGEF is supported. And your original issue is that copy as override doesn't work right, which I definitely need to look at.

@Infernio
Copy link
Contributor

Infernio commented Apr 10, 2020

Probably related: when I copy as override any record that contains OBME info (tested with ALCH, ENCH and SPEL - haven't been able to find one with INGR or SGST yet), I get an access violation:
https://gist.github.com/Infernio/e0734de4373ec1cbdccc8827c2a91b21

This is on 4.1.3b. Same thing also happens when I just try to drag and drop the effects portion of one those records over into the partial override.
Can be tested with the ESM plugin from this mod: https://www.nexusmods.com/oblivion/mods/49096

Edit: And trying to add an EFIX subrecord takes xEdit down completely, no assertion failure or anything. Just closes instantly.

@Infernio
Copy link
Contributor

Also, the decoding for OBME's SCIT/EFIX seems to be wrong? At least, according to OBME's documentation, the efixParam etc. is in SCIT, not in EFIX. As silly as that seems.

That, or OBME's docs are wrong 🤷‍♀️

ElminsterAU added a commit that referenced this issue Jan 24, 2021
…r MGEF / TES4Edit 4.0.3 not able to copy Magic Effect Records
ElminsterAU added a commit that referenced this issue Jan 24, 2021
…r MGEF / TES4Edit 4.0.3 not able to copy Magic Effect Records

(cherry picked from commit 3d201e9)
@ElminsterAU ElminsterAU reopened this Jul 8, 2021
@ElminsterAU
Copy link
Collaborator

OBME support is currently incomplete. OBME documentation is a mess.

@MacSplody
Copy link
Author

MacSplody commented Aug 28, 2021

Confirming that copy as override now works without issue in TES4Edit 4.0.3h (Hotfix 1).
Editing the copied record and saving changes also works fine.

@ElminsterAU Is there anything else I should test out for this issue?

@MacSplody MacSplody changed the title For Oblivion Copy as overrides fails for MGEF [TES4] Copy as overrides fails for MGEF Aug 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants