Merge #2493 Set default Exception.Message string for ModuleNotFoundKr…

…aken
KSP-CKAN · Aug 22, 2018 · 86d1f57 · 86d1f57
2 parents b3d3bc9 + 536bb90
commit 86d1f57
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -14,6 +14,7 @@ All notable changes to this project will be documented in this file.
 - [Multiple] Fix GUIMod crash when module doesn't have a compatible game version (#2486 by: HebaruSan; reviewed: cculianu, politas)
 - [Core] Fix crash on invalid portable or Steam folder (#2506 by: HebaruSan; reviewed: politas)
 - [GUI] Fix red X on HideTab (#2501 by: HebaruSan; reviewed: politas)
+- [Core] Set default Exception.Message string for ModuleNotFoundKraken (#2493 by: HebaruSan; reviewed: politas)
 
 ## v1.25.2 (Goddard)
 

diff --git a/Core/Types/Kraken.cs b/Core/Types/Kraken.cs
@@ -59,12 +59,17 @@ public class ModuleNotFoundKraken : Kraken
         public string version;
 
         // TODO: Is there a way to set the stringify version of this?
-        public ModuleNotFoundKraken(string module, string version = null, string reason = null, Exception innerException = null)
+        public ModuleNotFoundKraken(string module, string version, string reason, Exception innerException = null)
             : base(reason, innerException)
         {
-            this.module = module;
+            this.module  = module;
             this.version = version;
         }
+
+        public ModuleNotFoundKraken(string module, string version = null)
+            : this(module, version, $"Module not found: {module} {version ?? ""}")
+        { }
+
     }
 
     /// <summary>