Merge pull request #607 from ow-mods/dev

Updated file logging

Author: misternebula

src/OWML.Common/Interfaces/IOwmlConfig.cs

@@ -1,4 +1,6 @@
-namespace OWML.Common
+using System;
+
+namespace OWML.Common
 {
 	public interface IOwmlConfig
 	{
@@ -25,5 +27,7 @@ public interface IOwmlConfig
 		bool IncrementalGC { get; set; }
 
 		int SocketPort { get; set; }
+
+		DateTime LoadTime { get; set; }
 	}
 }

src/OWML.Common/OwmlConfig.cs

@@ -1,4 +1,5 @@
-using System.IO;
+using System;
+using System.IO;
 using Newtonsoft.Json;
 
 namespace OWML.Common
@@ -17,6 +18,9 @@ public class OwmlConfig : IOwmlConfig
 		[JsonProperty("incrementalGC")]
 		public bool IncrementalGC { get; set; }
 
+		[JsonProperty("loadTime")]
+		public DateTime LoadTime { get; set; }
+
 		[JsonIgnore]
 		public bool IsSpaced => Directory.Exists(Path.Combine(GamePath, "Outer Wilds_Data"));
 

src/OWML.Launcher/OWML.Manifest.json

@@ -3,7 +3,7 @@
   "author": "Alek",
   "name": "OWML",
   "uniqueName": "Alek.OWML",
-  "version": "2.14.0",
+  "version": "2.14.1",
   "minGameVersion": "1.1.15.1018",
   "maxGameVersion": "1.1.15.1018"
 }

src/OWML.Launcher/Program.cs

@@ -1,4 +1,5 @@
 using System;
+using System.IO;
 using OWML.Abstractions;
 using OWML.Common;
 using OWML.GameFinder;
@@ -25,6 +26,7 @@ public static Container CreateContainer(string[] args)
 			var hasConsolePort = argumentHelper.HasArgument(Constants.ConsolePortArgument);
 			SaveConsolePort(owmlConfig, hasConsolePort, argumentHelper);
 			SaveOwmlPath(owmlConfig);
+			SaveCurrentLogPath(owmlConfig);
 			var owmlManifest = GetOwmlManifest();
 			var consoleWriter = CreateConsoleWriter(owmlConfig, owmlManifest, hasConsolePort);
 
@@ -79,6 +81,17 @@ private static void SaveOwmlPath(IOwmlConfig owmlConfig)
 			JsonHelper.SaveJsonObject(Constants.OwmlConfigFileName, owmlConfig);
 		}
 
+		private static void SaveCurrentLogPath(IOwmlConfig owmlConfig)
+		{
+			if (File.Exists($"{owmlConfig.LogsPath}/latest.txt"))
+			{
+				File.Delete($"{owmlConfig.LogsPath}/latest.txt");
+			}
+
+			owmlConfig.LoadTime = DateTime.Now;
+			JsonHelper.SaveJsonObject(Constants.OwmlConfigFileName, owmlConfig);
+		}
+
 		private static IModManifest GetOwmlManifest() =>
 			JsonHelper.LoadJsonObject<ModManifest>(Constants.OwmlManifestFileName);
 

src/OWML.Logging/ModLogger.cs

@@ -9,16 +9,19 @@ public class ModLogger : IModLogger
 	{
 		private readonly IModManifest _manifest;
 		private static string _logFileName;
+		private static string _latestFileName;
 
 		public ModLogger(IOwmlConfig config, IModManifest manifest)
 		{
 			_manifest = manifest;
-			_logFileName = $"{config.LogsPath}/OWML.Log.{DateTime.Now:dd-MM-yyyy-HH.mm.ss}.txt";
+			_logFileName = $"{config.LogsPath}/OWML.Log.{config.LoadTime:yyyy-MM-ddTHH.mm.ss}.txt";
 
 			if (!Directory.Exists(config.LogsPath))
 			{
 				Directory.CreateDirectory(config.LogsPath);
 			}
+
+			_latestFileName = $"{config.LogsPath}/latest.txt";
 		}
 
 		[Obsolete("Use ModHelper.Console.WriteLine with messageType = Debug instead.")]
@@ -29,7 +32,18 @@ public void Log(string s) =>
 		public void Log(params object[] objects) =>
 			Log(string.Join(" ", objects.Select(o => o.ToString()).ToArray()));
 
-		private static void LogInternal(string message) =>
-			File.AppendAllText(_logFileName, $"{DateTime.Now}: {message}{Environment.NewLine}");
+		private static void LogInternal(string message)
+		{
+			var text = $"{DateTime.Now}: {message}{Environment.NewLine}";
+			try
+			{
+				File.AppendAllText(_logFileName, text);
+				File.AppendAllText(_latestFileName, text);
+			}
+			catch
+			{
+				// ignored
+			}
+		}
 	}
 }

src/OWML.ModHelper.Events/HarmonyHelper.cs

@@ -33,7 +33,7 @@ private Harmony CreateInstance()
 				if (_owmlConfig.DebugMode)
 				{
 					_console.WriteLine("Enabling Harmony debug mode.", MessageType.Debug);
-					FileLog.logPath = $"{_owmlConfig.LogsPath}/Harmony.Log.{DateTime.Now:dd-MM-yyyy-HH.mm.ss}.txt";
+					FileLog.logPath = $"{_owmlConfig.LogsPath}/Harmony.Log.{_owmlConfig.LoadTime:yyyy-MM-ddTHH.mm.ss}.txt";
 					HarmonyFileLog.Enabled = true;
 				}
 				harmony = new Harmony(_manifest.UniqueName);