Enhanced logging with additional levels and distinct log file format

Flow.Launcher.Infrastructure/Logger/Log.cs

@@ -3,10 +3,10 @@
 using System.Runtime.CompilerServices;
 using System.Runtime.ExceptionServices;
 using Flow.Launcher.Infrastructure.UserSettings;
 using NLog;
 using NLog.Config;
 using NLog.Targets;
 using NLog.Targets.Wrappers;
 
 namespace Flow.Launcher.Infrastructure.Logger
 {
@@ -16,7 +16,7 @@
 
         public static string CurrentLogDirectory { get; }
 
-        static Log()
+static Log()
         {
             CurrentLogDirectory = DataLocation.VersionLogDirectory;
             if (!Directory.Exists(CurrentLogDirectory))
@@ -27,14 +27,14 @@
             var configuration = new LoggingConfiguration();
 
             const string layout = 
                 @"${date:format=HH\:mm\:ss.ffffK} - " +
                 @"${level:uppercase=true:padding=-5} - ${logger} - ${message:l}" +
                 @"${onexception:${newline}" +
                     @"EXCEPTION OCCURS\: ${exception:format=tostring}${newline}}";
 
             var fileTarget = new FileTarget
             {
-                FileName = CurrentLogDirectory.Replace(@"\", "/") + "/${shortdate}.txt",
+                FileName = CurrentLogDirectory.Replace(@"\", "/") + "/Flow.Launcher.${date:format=yyyyMMdd}.log",
                 Layout = layout
             };
 
@@ -45,6 +45,8 @@
                 Layout = layout
             };
 
+
+
             configuration.AddTarget("file", fileTargetASyncWrapper);
             configuration.AddTarget("debug", debugTarget);
 
@@ -65,26 +67,22 @@
 
         public static void SetLogLevel(LOGLEVEL level)
         {
-            switch (level)
+            var rule = LogManager.Configuration.FindRuleByName("file");
+
+            var nlogLevel = level switch
             {
-                case LOGLEVEL.DEBUG:
-                    UseDebugLogLevel();
-                    break;
-                default:
-                    UseInfoLogLevel();
-                    break;
-            }
-            Info(nameof(Logger), $"Using log level: {level}.");
-        }
+                LOGLEVEL.None => LogLevel.Off,
+                LOGLEVEL.Error => LogLevel.Error,
+                LOGLEVEL.Debug => LogLevel.Debug,
+                _ => LogLevel.Info
+            };
 
-        private static void UseDebugLogLevel()
-        {
-            LogManager.Configuration.FindRuleByName("file").SetLoggingLevels(LogLevel.Debug, LogLevel.Fatal);
-        }
+            rule.SetLoggingLevels(nlogLevel, LogLevel.Fatal);
 
-        private static void UseInfoLogLevel()
-        {
-            LogManager.Configuration.FindRuleByName("file").SetLoggingLevels(LogLevel.Info, LogLevel.Fatal);
+            LogManager.ReconfigExistingLoggers();
+
+            // We can't log Info when level is set to Error or None, so we use Debug
+            Debug(nameof(Logger), $"Using log level: {level}.");
         }
 
         private static void LogFaultyFormat(string message)
@@ -169,7 +167,9 @@
 
     public enum LOGLEVEL
     {
-        DEBUG,
-        INFO
+        None,
+        Error,
+        Info,
+        Debug
     }
 }

Flow.Launcher.Infrastructure/UserSettings/Settings.cs

@@ -340,7 +340,7 @@ public CustomBrowserViewModel CustomBrowser
         public DialogJumpFileResultBehaviours DialogJumpFileResultBehaviour { get; set; } = DialogJumpFileResultBehaviours.FullPath;
 
         [JsonConverter(typeof(JsonStringEnumConverter))]
-        public LOGLEVEL LogLevel { get; set; } = LOGLEVEL.INFO;
+        public LOGLEVEL LogLevel { get; set; } = LOGLEVEL.Info;
 
         /// <summary>
         /// when false Alphabet static service will always return empty results

Flow.Launcher/Languages/en.xaml

@@ -459,11 +459,13 @@
     <system:String x:Key="welcomewindow">Wizard</system:String>
     <system:String x:Key="userdatapath">User Data Location</system:String>
     <system:String x:Key="userdatapathToolTip">User settings and installed plugins are saved in the user data folder. This location may vary depending on whether it's in portable mode or not.</system:String>
-    <system:String x:Key="userdatapathButton">Open Folder</system:String>
+<system:String x:Key="userdatapathButton">Open Folder</system:String>
     <system:String x:Key="advanced">Advanced</system:String>
     <system:String x:Key="logLevel">Log Level</system:String>
-    <system:String x:Key="LogLevelDEBUG">Debug</system:String>
-    <system:String x:Key="LogLevelINFO">Info</system:String>
+    <system:String x:Key="LogLevelNone">None</system:String>
+    <system:String x:Key="LogLevelError">Error</system:String>
+    <system:String x:Key="LogLevelInfo">Info</system:String>
+    <system:String x:Key="LogLevelDebug">Debug</system:String>
     <system:String x:Key="settingWindowFontTitle">Setting Window Font</system:String>
 
     <!--  Release Notes Window  -->