From e5ae9f82f367674935e2921e96b83137534a1f63 Mon Sep 17 00:00:00 2001
From: Aragas <personal@aragas.org>
Date: Sat, 7 Jan 2023 17:11:59 +0200
Subject: [PATCH] Fixed launcher messages

---
 build/common.props                            |  2 +-
 changelog.txt                                 |  4 ++++
 .../Helpers/ModuleIssueRenderer.cs            | 22 ++++++++++---------
 3 files changed, 17 insertions(+), 11 deletions(-)

diff --git a/build/common.props b/build/common.props
index 4a21b99..25bbd57 100644
--- a/build/common.props
+++ b/build/common.props
@@ -7,7 +7,7 @@
 
   <!--Development Variables-->
   <PropertyGroup>
-    <Version>1.20.2</Version>
+    <Version>1.20.3</Version>
     <HarmonyVersion>2.2.2</HarmonyVersion>
     <BUTRSharedVersion>3.0.0.126</BUTRSharedVersion>
     <BUTRModuleManagerVersion>5.0.166</BUTRModuleManagerVersion>
diff --git a/changelog.txt b/changelog.txt
index 773bf6e..6f11eb7 100644
--- a/changelog.txt
+++ b/changelog.txt
@@ -1,4 +1,8 @@
 ---------------------------------------------------------------------------------------------------
+Version: 1.20.3
+Game Versions: v1.0.0,v1.0.1,v1.0.2,v1.0.3
+* Fixed launcher messages
+---------------------------------------------------------------------------------------------------
 Version: 1.20.2
 Game Versions: v1.0.0,v1.0.1,v1.0.2,v1.0.3
 * Fixed the Russian language issue?
diff --git a/src/Bannerlord.BUTRLoader.LauncherEx/Helpers/ModuleIssueRenderer.cs b/src/Bannerlord.BUTRLoader.LauncherEx/Helpers/ModuleIssueRenderer.cs
index b7f71a6..c5f3575 100644
--- a/src/Bannerlord.BUTRLoader.LauncherEx/Helpers/ModuleIssueRenderer.cs
+++ b/src/Bannerlord.BUTRLoader.LauncherEx/Helpers/ModuleIssueRenderer.cs
@@ -9,6 +9,12 @@ internal static class ModuleIssueRenderer
     {
         public static string Render(ModuleIssue issue) => RenderTextObject(issue).ToString();
 
+        private static string Version(ApplicationVersionRange version) => version == ApplicationVersionRange.Empty
+            ? version.ToString()
+            : version.Min == version.Max
+                ? version.Min.ToString()
+                : "";
+
         public static BUTRTextObject RenderTextObject(ModuleIssue issue) => issue.Type switch
         {
             ModuleIssueType.Missing => new BUTRTextObject("{=J3Uh6MV4}Missing '{ID}' {VERSION} in modules list")
@@ -16,12 +22,8 @@ internal static class ModuleIssueRenderer
                 .SetTextVariable("VERSION", issue.SourceVersion.Min.ToString()),
 
             ModuleIssueType.MissingDependencies => new BUTRTextObject("{=3eQSr6wt}Missing '{ID}' {VERSION}")
-                .SetTextVariable("ID", issue.Target.Id)
-                .SetTextVariable("VERSION", issue.SourceVersion == ApplicationVersionRange.Empty
-                    ? issue.SourceVersion.ToString()
-                    : issue.SourceVersion.Min == issue.SourceVersion.Max
-                        ? issue.SourceVersion.Min.ToString()
-                        : ""),
+                .SetTextVariable("ID", issue.SourceId)
+                .SetTextVariable("VERSION", Version(issue.SourceVersion)),
             ModuleIssueType.DependencyMissingDependencies => new BUTRTextObject("{=U858vdQX}'{ID}' is missing it's dependencies!")
                 .SetTextVariable("ID", issue.SourceId),
 
@@ -30,13 +32,13 @@ internal static class ModuleIssueRenderer
 
             ModuleIssueType.VersionMismatchLessThanOrEqual => new BUTRTextObject("{=Vjz9HQ41}'{ID}' wrong version <= {VERSION}")
                 .SetTextVariable("ID", issue.SourceId)
-                .SetTextVariable("VERSION", issue.SourceVersion.ToString()),
+                .SetTextVariable("VERSION", Version(issue.SourceVersion)),
             ModuleIssueType.VersionMismatchLessThan => new BUTRTextObject("{=ZvnlL7VE}'{ID}' wrong version < [{VERSION}]")
                 .SetTextVariable("ID", issue.SourceId)
-                .SetTextVariable("VERSION", issue.SourceVersion.ToString()),
+                .SetTextVariable("VERSION", Version(issue.SourceVersion)),
             ModuleIssueType.VersionMismatchGreaterThan => new BUTRTextObject("{=EfNuH2bG}'{ID}' wrong version > [{VERSION}]")
                 .SetTextVariable("ID", issue.SourceId)
-                .SetTextVariable("VERSION", issue.SourceVersion.ToString()),
+                .SetTextVariable("VERSION", Version(issue.SourceVersion)),
 
             ModuleIssueType.Incompatible => new BUTRTextObject("{=zXDidmpQ}'{ID}' is incompatible with this module")
                 .SetTextVariable("ID", issue.SourceId),
@@ -54,7 +56,7 @@ internal static class ModuleIssueRenderer
                 .SetTextVariable("SOURCEID", issue.SourceId),
 
             ModuleIssueType.DependencyNotLoadedAfterThis => new BUTRTextObject("{=2ALJB7z2}'{TARGETID}' should be loaded after '{SOURCEID}'")
-                .SetTextVariable("ID", issue.Target.Id),
+                .SetTextVariable("ID", issue.SourceId),
 
             _ => throw new ArgumentOutOfRangeException()
         };