From f56636d2c8c78d06b150c4f942f2b791228c560b Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Tue, 18 Sep 2018 21:03:05 -0400
Subject: fix trace logs not showing path for invalid mods

---
 src/SMAPI/Framework/SCore.cs | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

(limited to 'src/SMAPI')

diff --git a/src/SMAPI/Framework/SCore.cs b/src/SMAPI/Framework/SCore.cs
index 14380f7b..d05f5bb9 100644
--- a/src/SMAPI/Framework/SCore.cs
+++ b/src/SMAPI/Framework/SCore.cs
@@ -837,7 +837,7 @@ namespace StardewModdingAPI.Framework
             // unlock mod integrations
             this.ModRegistry.AreAllModsInitialised = true;
         }
-        
+
         /// <summary>Load a given mod.</summary>
         /// <param name="mod">The mod to load.</param>
         /// <param name="mods">The mods being loaded.</param>
@@ -855,13 +855,14 @@ namespace StardewModdingAPI.Framework
             errorDetails = null;
 
             // log entry
-            if (mod.IsContentPack)
-                this.Monitor.Log($"   {mod.DisplayName} (content pack, {PathUtilities.GetRelativePath(this.ModsPath, mod.DirectoryPath)})...", LogLevel.Trace);
-            else
             {
-                this.Monitor.Log(mod.Manifest?.EntryDll != null
-                    ? $"   {mod.DisplayName} ({PathUtilities.GetRelativePath(this.ModsPath, mod.DirectoryPath)}{Path.DirectorySeparatorChar}{mod.Manifest.EntryDll})..." // don't use Path.Combine here, since EntryDLL might not be valid
-                    : $"   {mod.DisplayName}...", LogLevel.Trace);
+                string relativePath = PathUtilities.GetRelativePath(this.ModsPath, mod.DirectoryPath);
+                if (mod.IsContentPack)
+                    this.Monitor.Log($"   {mod.DisplayName} ({relativePath}) [content pack]...", LogLevel.Trace);
+                else if (mod.Manifest.EntryDll != null)
+                    this.Monitor.Log($"   {mod.DisplayName} ({relativePath}{Path.DirectorySeparatorChar}{mod.Manifest.EntryDll})...", LogLevel.Trace); // don't use Path.Combine here, since EntryDLL might not be valid
+                else
+                    this.Monitor.Log($"   {mod.DisplayName} ({relativePath})...", LogLevel.Trace);
             }
 
             // add warning for missing update key
-- 
cgit