Do not lose settings after updating the app

dziemborowicz · Aug 24, 2016 · 5eb2cc3 · 5eb2cc3
1 parent 294cb36
commit 5eb2cc3
Show file tree

Hide file tree

Showing 5 changed files with 37 additions and 4 deletions.
diff --git a/Hourglass/App.config b/Hourglass/App.config
@@ -19,6 +19,9 @@
             <setting name="UniqueId" serializeAs="String">
                 <value>00000000-0000-0000-0000-000000000000</value>
             </setting>
+            <setting name="UpgradeRequired" serializeAs="String">
+                <value>True</value>
+            </setting>
         </Hourglass.Properties.Settings>
     </userSettings>
 </configuration>
diff --git a/Hourglass/Managers/AppManager.cs b/Hourglass/Managers/AppManager.cs
@@ -6,6 +6,8 @@
 
 namespace Hourglass.Managers
 {
+    using System.Linq;
+
     /// <summary>
     /// Manages the app.
     /// </summary>
@@ -22,6 +24,7 @@ public class AppManager : Manager
         private static readonly Manager[] Managers = 
         {
             ErrorManager.Instance,
+            SettingsManager.Instance,
             UpdateManager.Instance,
             KeepAwakeManager.Instance,
             WakeUpManager.Instance,
@@ -30,8 +33,7 @@ public class AppManager : Manager
             SoundManager.Instance,
             TimerStartManager.Instance,
             TimerOptionsManager.Instance,
-            TimerManager.Instance,
-            SettingsManager.Instance
+            TimerManager.Instance
         };
 
         /// <summary>
@@ -57,7 +59,7 @@ public override void Initialize()
         /// </summary>
         public override void Persist()
         {
-            foreach (Manager manager in Managers)
+            foreach (Manager manager in Managers.Reverse())
             {
                 manager.Persist();
             }
@@ -77,7 +79,7 @@ protected override void Dispose(bool disposing)
 
             if (disposing)
             {
-                foreach (Manager manager in Managers)
+                foreach (Manager manager in Managers.Reverse())
                 {
                     manager.Dispose();
                 }

diff --git a/Hourglass/Managers/SettingsManager.cs b/Hourglass/Managers/SettingsManager.cs
@@ -25,6 +25,19 @@ private SettingsManager()
         {
         }
 
+        /// <summary>
+        /// Initializes the class.
+        /// </summary>
+        public override void Initialize()
+        {
+            if (Settings.Default.UpgradeRequired)
+            {
+                Settings.Default.Upgrade();
+                Settings.Default.UpgradeRequired = false;
+                Settings.Default.Save();
+            }
+        }
+
         /// <summary>
         /// Persists the state of the class.
         /// </summary>

diff --git a/Hourglass/Properties/Settings.Designer.cs b/Hourglass/Properties/Settings.Designer.cs
diff --git a/Hourglass/Properties/Settings.settings b/Hourglass/Properties/Settings.settings
@@ -26,5 +26,8 @@
     <Setting Name="WindowSizeInfo" Type="Hourglass.Serialization.WindowSizeInfo" Scope="User">
       <Value Profile="(Default)" />
     </Setting>
+    <Setting Name="UpgradeRequired" Type="System.Boolean" Scope="User">
+      <Value Profile="(Default)">True</Value>
+    </Setting>
   </Settings>
 </SettingsFile>