SaveScum / skyrim-plugin-decoding-project

Automatically exported from code.google.com/p/skyrim-plugin-decoding-project
0 stars 0 forks source link

Sort masters not updating FormIDs correctly. #123

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Download attached file.
2. Make sure that the load order of DLCs is: Dawnguard, Hearthfires, Dragonborn 
(in order of their release)
3. Sort Master (attached plugin has hearthfires.esm as a last master)
4. Check for errors.

What is the expected output?
No errors.

What do you see instead?
Errors from unresolved references.

What version of the product are you using? On what operating system?
r1254

Please provide any additional information below.

Original issue reported on code.google.com by zila...@gmail.com on 7 Mar 2013 at 1:21

Attachments:

GoogleCodeExporter commented 9 years ago
It's always the "added" cell record at index -1.
I/we may have a [for 0 to Pred(Count)] where we need a [for Low to High]
Also that is only a display issue as this record is not written to disk. 
Reloading the plugin "fixes" it.

Original comment by HuguesLe...@gmail.com on 7 Mar 2013 at 3:55

GoogleCodeExporter commented 9 years ago
The only concern is plugin's integrity. If everything is ok, then it is not a 
serious issue.

Original comment by zila...@gmail.com on 7 Mar 2013 at 4:00

GoogleCodeExporter commented 9 years ago
Best practice is to close and save after Add masters or Sort masters. Maybe 
that's why :)

Original comment by HuguesLe...@gmail.com on 7 Mar 2013 at 4:15

GoogleCodeExporter commented 9 years ago
I think this also impacts AddMasterIfMissing for scripts. When I use that the 
new masters are not inserted at the end of the list and thus cause existing 
references to break.

Original comment by cehoff...@gmail.com on 16 Mar 2013 at 8:22

GoogleCodeExporter commented 9 years ago
Use AddRequiredElementMasters(), always works for me.

Original comment by zila...@gmail.com on 16 Mar 2013 at 9:17