move post-load fmginfo edits

soulsmods · May 24, 2024 · 6980550 · 6980550
1 parent 64d97b6
commit 6980550
Showing 1 changed file with 13 additions and 17 deletions.
diff --git a/src/StudioCore/TextEditor/FMGBank.cs b/src/StudioCore/TextEditor/FMGBank.cs
@@ -101,23 +101,6 @@ private bool LoadItemMenuMsgBnds(AssetDescription itemMsgPath, AssetDescription
             return false;
         }
 
-        foreach (FMGInfo info in FmgInfoBank)
-        {
-            ApplyGameDifferences(info);
-            if (CFG.Current.FMG_NoGroupedFmgEntries)
-            {
-                info.EntryType = FmgEntryTextType.TextBody;
-            }
-        }
-
-        if (!CFG.Current.FMG_NoFmgPatching)
-        {
-            foreach (FMGInfo info in FmgInfoBank)
-            {
-                SetFMGInfoPatchParent(info);
-            }
-        }
-
         if (CFG.Current.FMG_NoGroupedFmgEntries)
         {
             FmgInfoBank = FmgInfoBank.OrderBy(e => e.EntryCategory).ThenBy(e => e.FmgID).ToList();
@@ -173,6 +156,19 @@ private bool LoadMsgBnd(string path, string msgBndType = "UNDEFINED")
         {
             FmgInfoBank.Add(GenerateFMGInfo(file));
         }
+        // I hate this 2 parse system. Solve game differences by including game in the get enum functions. Maybe parentage is solvable by pre-sorting binderfiles but that does seem silly. FMG patching sucks. 
+        foreach (FMGInfo info in FmgInfoBank)
+        {
+            ApplyGameDifferences(info);
+            if (CFG.Current.FMG_NoGroupedFmgEntries)
+            {
+                info.EntryType = FmgEntryTextType.TextBody;
+            }
+            if (!CFG.Current.FMG_NoFmgPatching)
+            {
+                SetFMGInfoPatchParent(info);
+            }
+        }
 
         fmgBinder.Dispose();
         return true;