fix: bug where directory won't be created if it doesn't exist

dylan-george-field · Jan 22, 2023 · 9298a44 · 9298a44
1 parent e63cb29
commit 9298a44
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 10 deletions.
diff --git a/src/wfh-log-wpf/App.xaml.cs b/src/wfh-log-wpf/App.xaml.cs
@@ -1,18 +1,13 @@
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Hosting;
-using Microsoft.Win32;
 using System;
 using System.ComponentModel;
 using System.Diagnostics;
-using System.IO;
-using System.Linq;
 using System.Reflection;
 using System.Windows;
 using wfh_log_wpf.Logger;
-using wfh_log_wpf.Models;
 using wfh_log_wpf.Settings;
 using wfh_log_wpf.Timer;
-using wfh_log_wpf.Uninstaller;
 
 namespace wfh_log_wpf
 {
@@ -114,7 +109,7 @@ private void ShowMainWindow()
 
         private void OpenLogFile()
         {
-            var absoluteFilePath = BaseLog.GetLogPath();
+            var absoluteFilePath = BaseLog.GetLogFilePath();
 
             var processStartInfo = new ProcessStartInfo
             {

diff --git a/src/wfh-log-wpf/Logger/BaseLog.cs b/src/wfh-log-wpf/Logger/BaseLog.cs
@@ -12,16 +12,24 @@ public abstract class BaseLog
 
         public BaseLog()
         {
-            AbsoluteFilePath = GetLogPath();
+            AbsoluteFilePath = GetLogFilePath();
+
+            Directory.CreateDirectory(GetLogDirectoryPath());
         }
 
-        public static string GetLogPath()
+        public static string GetLogFilePath()
+        {
+            var filename = "wfh.log";
+
+            return Path.Combine(GetLogDirectoryPath(), filename);
+        }
+
+        public static string GetLogDirectoryPath()
         {
             var appDataPath = Environment.GetFolderPath(Environment.SpecialFolder.LocalApplicationData);
             var directory = "wfh-log";
-            var filename = "wfh.log";
 
-            return Path.Combine(appDataPath, directory, filename);
+            return Path.Combine(appDataPath, directory);
         }
     }
 }