Fix a hard crash happening when UniGetUI in portable mode does not ha…

…ve write permissions on the settings directory
marticliment · Feb 8, 2025 · 104de8d · 104de8d
1 parent 1d9e025
commit 104de8d
Showing 1 changed file with 14 additions and 3 deletions.
diff --git a/src/UniGetUI.Core.Data/CoreData.cs b/src/UniGetUI.Core.Data/CoreData.cs
@@ -39,11 +39,22 @@ public static string UniGetUIDataDirectory
                 if (IS_PORTABLE is null)
                     IS_PORTABLE = File.Exists(Path.Join(UniGetUIExecutableDirectory, "ForceUniGetUIPortable"));
 
-                if (IS_PORTABLE == true)
+                if (IS_PORTABLE is true)
                 {
                     string path = Path.Join(UniGetUIExecutableDirectory, "Settings");
-                    if (!Directory.Exists(path)) Directory.CreateDirectory(path);
-                    return path;
+                    try
+                    {
+                        if (!Directory.Exists(path)) Directory.CreateDirectory(path);
+                        var testfilepath = Path.Join(path, "PermissionTestFile");
+                        File.WriteAllText(testfilepath, "https://www.youtube.com/watch?v=dQw4w9WgXcQ");
+                        return path;
+                    }
+                    catch (Exception ex)
+                    {
+                        IS_PORTABLE = false;
+                        Logger.Error($"Could not acces/write path {path}. UniGetUI will NOT be run in portable mode, and User settings will be used instead");
+                        Logger.Error(ex);
+                    }
                 }
 
                 string old_path = Path.Join(Environment.GetFolderPath(Environment.SpecialFolder.UserProfile), ".wingetui");