Fix crash when saving peferences

DSPaul · Sep 12, 2024 · 524f2bf · 524f2bf
1 parent 3c1ecb1
commit 524f2bf
Show file tree

Hide file tree

Showing 4 changed files with 36 additions and 14 deletions.
diff --git a/src/Resources/Controls/EnhancedDataGrid.cs b/src/Resources/Controls/EnhancedDataGrid.cs
@@ -1,6 +1,7 @@
 //https://bengribaudo.com/blog/2012/03/14/1942/saving-restoring-wpf-datagrid-columns-size-sorting-and-order
 
 using COMPASS.Services;
+using COMPASS.Tools;
 using Newtonsoft.Json;
 using System;
 using System.Collections;
@@ -67,9 +68,16 @@ private void UpdateColumnInfo()
                 prefsService.Preferences.UIState.SortDirection = sd.Direction;
             }
 
-            Properties.Settings.Default["DataGridCollumnInfo"] = json;
-            Properties.Settings.Default.Save();
-            updatingColumnInfo = false;
+            try
+            {
+                Properties.Settings.Default["DataGridCollumnInfo"] = json;
+                Properties.Settings.Default.Save();
+                updatingColumnInfo = false;
+            }
+            catch (Exception ex)
+            {
+                Logger.Error("Failed to save Preferences", ex);
+            }
         }
         protected override void OnColumnReordered(DataGridColumnEventArgs e)
         {

diff --git a/src/Services/PreferencesService.cs b/src/Services/PreferencesService.cs
@@ -27,13 +27,13 @@ private PreferencesService() { }
 
         public void SavePreferences()
         {
-            Properties.Settings.Default.Save();
-
-            if (_preferences == null) return; //don't save when they aren't loaded
-            PreferencesDto dto = _preferences.ToDto();
-
             try
             {
+                Properties.Settings.Default.Save();
+
+                if (_preferences == null) return; //don't save when they aren't loaded
+                PreferencesDto dto = _preferences.ToDto();
+
                 string tempFileName = PreferencesFilePath + ".tmp";
 
                 lock (writeLocker)

diff --git a/src/ViewModels/MainViewModel.cs b/src/ViewModels/MainViewModel.cs
@@ -47,9 +47,16 @@ private void UpgradeSettings()
         {
             if (Properties.Settings.Default.justUpdated)
             {
-                Properties.Settings.Default.Upgrade();
-                Properties.Settings.Default.justUpdated = false;
-                Properties.Settings.Default.Save();
+                try
+                {
+                    Properties.Settings.Default.Upgrade();
+                    Properties.Settings.Default.justUpdated = false;
+                    Properties.Settings.Default.Save();
+                }
+                catch (Exception ex)
+                {
+                    Logger.Error("Failed to Update & Save Preferences", ex);
+                }
             }
         }
 

diff --git a/src/ViewModels/SettingsViewModel.cs b/src/ViewModels/SettingsViewModel.cs
@@ -425,9 +425,16 @@ public static string CompassDataPath
 
             set
             {
-                Properties.Settings.Default.CompassDataPath = value;
-                EnvironmentVarsService.CompassDataPath = value;
-                Properties.Settings.Default.Save();
+                try
+                {
+                    Properties.Settings.Default.CompassDataPath = value;
+                    EnvironmentVarsService.CompassDataPath = value;
+                    Properties.Settings.Default.Save();
+                }
+                catch (Exception ex)
+                {
+                    Logger.Error("Failed to update Compass Data Path", ex);
+                }
             }
         }