Reworking the half baked log support I did years ago to something decent. No more gambiarras.

Author: Lisias

Source/ModuleManager/CustomConfigsManager.cs

@@ -36,7 +36,7 @@ internal void Start()
 			if (start_techtree_loaded)
 			{
 				if (HighLogic.CurrentGame.Parameters.Career.TechTreeUrl != TECHTREE_CONFIG.KspPath)
-					Log(string.Format("Tech tree was changed by third party to [{0}].", HighLogic.CurrentGame.Parameters.Career.TechTreeUrl));
+					Log("Tech tree was changed by third party to [{0}].", HighLogic.CurrentGame.Parameters.Career.TechTreeUrl);
 			}
 			else if (TECHTREE_CONFIG.IsLoadable)
 			{
@@ -47,9 +47,9 @@ internal void Start()
 		}
 
 		private static readonly KSPe.Util.Log.Logger log = KSPe.Util.Log.Logger.CreateThreadUnsafeForType<CustomConfigsManager>(); // No need to use thread safe logging. Yet.
-		private static void Log(String s)
+		private static void Log(String s, params object[] @params)
 		{
-			log.info(s);
+			log.info(s, @params);
 		}
 
 	}

Source/ModuleManager/Extensions/IBasicLoggerExtensions.cs

@@ -14,16 +14,19 @@ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
 	You should have received a copy of the GNU General Public License 3.0
 	along with Module Manager /L. If not, see <https://www.gnu.org/licenses/>.
 */
-using System;
 using UnityEngine;
 using ModuleManager.Logging;
 
+using K = KSPe.Util.Log;
+
 namespace ModuleManager.Extensions
 {
     public static class IBasicLoggerExtensions
     {
-        public static void Info(this IBasicLogger logger, string message) => logger.Log(LogType.Log, message);
-        public static void Warning(this IBasicLogger logger, string message) => logger.Log(LogType.Warning, message);
-        public static void Error(this IBasicLogger logger, string message) => logger.Log(LogType.Error, message);
+        public static void Trace(this IBasicLogger logger, string message, params object[] @params) => logger.Log(K.Level.TRACE, message, @params);
+        public static void Detail(this IBasicLogger logger, string message, params object[] @params) => logger.Log(K.Level.DETAIL, message, @params);
+        public static void Info(this IBasicLogger logger, string message, params object[] @params) => logger.Log(K.Level.INFO, message, @params);
+        public static void Warning(this IBasicLogger logger, string message, params object[] @params) => logger.Log(K.Level.WARNING, message, @params);
+        public static void Error(this IBasicLogger logger, string message, params object[] @params) => logger.Log(K.Level.ERROR, message, @params);
     }
 }

Source/ModuleManager/Logging/IBasicLogger.cs

@@ -15,14 +15,15 @@ You should have received a copy of the GNU General Public License 3.0
 	along with Module Manager /L. If not, see <https://www.gnu.org/licenses/>.
 */
 using System;
-using UnityEngine;
+
+using K = KSPe.Util.Log;
 
 namespace ModuleManager.Logging
 {
     // Stripped down version of UnityEngine.ILogger
     public interface IBasicLogger
     {
-        void Log(LogType logType, string message);
+        void Log(K.Level logType, string message, params object[] @params);
         void Exception(string message, Exception exception);
         void Finish();
     }

Source/ModuleManager/Logging/ModLogger.cs

@@ -39,12 +39,7 @@ private ModLogger()
             this.methods[i++] = new LogMethod(LOG.error);
         }
 
-        // Gambiarra porque eu não previ essa possibilidade no KSPe.Util.Log!
-        private static readonly object[] NONE = new object[0];
-        public void Log(LogType logType, string message)
-        {
-            this.methods[(int)logType](message, NONE);
-        }
+        public void Log(K.Level logType, string message, params object[] @params) => this.methods[(int)logType](message, @params);
 
         public void Exception(string message, Exception exception)
         {

Source/ModuleManager/Logging/PatchLogger.cs

@@ -22,6 +22,7 @@ namespace ModuleManager.Logging
     public class PatchLogger : IBasicLogger
     {
         internal readonly K.Logger log;
+        internal static bool DebugMode => KSPe.Globals<ModuleManager>.DebugMode;
 
         private delegate void LogMethod(string message, params object[] @parms);
         private readonly LogMethod[] methods;
@@ -36,16 +37,11 @@ internal PatchLogger(string filename)
             this.methods[i++] = new LogMethod(this.log.warn);
             this.methods[i++] = new LogMethod(this.log.info);
             this.methods[i++] = new LogMethod(this.log.detail);
-            this.methods[i++] = new LogMethod(this.log.error);
+            this.methods[i++] = new LogMethod(this.log.trace);
             this.log.level = K.Level.TRACE;
         }
 
-        // Gambiarra porque eu não previ essa possibilidade no KSPe.Util.Log!
-        private static readonly object[] NONE = new object[0];
-        public void Log(LogType logType, string message)
-        {
-            this.methods[(int)logType](message, NONE);
-        }
+        public void Log(K.Level logType, string message, object[] @params) => this.methods[(int)logType](message, @params);
 
         public void Exception(string message, Exception exception)
         {