More consistency checks and validation on ensuring models are always exported with the correct skin material (and that skin material is correctly passed to the exporters)

Author: Lunaretic

xivModdingFramework/Models/FileTypes/Mdl.cs

@@ -217,6 +217,12 @@ public async Task ExportModel(TTModel model, string outputFilePath, int mtrlVari
             // Pop the textures out so the exporters can reference them.
             if (includeTextures)
             {
+                // Fix up our skin references in the model before exporting, to ensure
+                // we supply the right material names to the exporters down-chain.
+                if (model.IsInternal)
+                {
+                    ModelModifiers.FixUpSkinReferences(model, model.Source, null);
+                }
                 await ExportMaterialsForModel(model, outputFilePath, _gameDirectory, mtrlVariant);
             }
 

xivModdingFramework/Models/Helpers/ModelModifiers.cs

@@ -1144,6 +1144,10 @@ public static void CalculateTangentsFromBinormals(TTModel model, Action<bool, st
         /// <param name="newInternalPath"></param>
         public static void FixUpSkinReferences(TTModel model, string newInternalPath, Action<bool, string> loggingFunction = null)
         {
+            if (loggingFunction == null)
+            {
+                loggingFunction = NoOp;
+            }
 
             // Here we should to go in and correct any Skin Material references to point to the skin material for this race.
             // It's not actually -NEEDED-, as the game will dynamically resolve them anyways to the player's skin material, but it's good for user expectation and sanity.