Added load on system startup functionality via Registry.

Author: Bilge

Source/Forms/OptionsForm.cs

@@ -1,5 +1,6 @@
 using System.Windows.Forms;
 using ScriptFUSION.WarframeAlertTracker.Controls;
+using ScriptFUSION.WarframeAlertTracker.Registry;
 using ScriptFUSION.WarframeAlertTracker.Properties;
 using ScriptFUSION.WarframeAlertTracker.Resource;
 
@@ -9,20 +10,24 @@ public partial class OptionsForm : Form {
 
         private Settings Settings { get; }
 
-        internal OptionsForm(Settings settings, FissureControl dummyFissureControl) {
+        private RegistrySettings RegistrySettings { get; }
+
+        internal OptionsForm(Settings settings, RegistrySettings registrySettings, FissureControl dummyFissureControl) {
             InitializeComponent();
-            ReadSettings(Settings = settings);
+            ReadSettings(Settings = settings, RegistrySettings = registrySettings);
 
             imageRepository = dummyFissureControl.ImageRepository;
             dummyFissureControl.Size = sample.Size;
             sample.Image = dummyFissureControl.Snapshot();
         }
 
-        private void ReadSettings(Settings settings) {
+        private void ReadSettings(Settings settings, RegistrySettings registrySettings) {
+            loadSystem.Checked = registrySettings.LoadAtStartUp.Enabled;
             loadHidden.Checked = settings.LoadHidden;
         }
 
         private void WriteSettings() {
+            RegistrySettings.LoadAtStartUp.Enabled = loadSystem.Checked;
             Settings.LoadHidden = loadHidden.Checked;
         }
 

Source/Forms/WatForm.cs

@@ -44,7 +44,7 @@ internal WatForm(WatApplication application) {
             trayIcon.Icon = Icon;
         }
 
-        public void ToggleWindowVisibility() {
+        private void ToggleWindowVisibility() {
             if (Visible = !Visible) {
                 // Restore previous state.
                 WindowState = LastWindowState;
@@ -87,9 +87,14 @@ private void alerts_Click(object sender, EventArgs e) {
         }
 
         private async void options_Click(object sender, EventArgs e) {
-            using (var form = new OptionsForm(Application.Settings, await FissureControl.CreateTestControl(Application.ImageRepository))) {
+            using (var form = new OptionsForm(
+                Application.Settings,
+                Application.RegistrySettings,
+                await FissureControl.CreateTestControl(Application.ImageRepository)
+            )) {
                 if (form.ShowDialog(this) == DialogResult.OK) {
                     Application.Settings.Save();
+                    Application.RegistrySettings.Save();
                 }
             }
         }
@@ -105,7 +110,7 @@ private void showMenuItem_Click(object sender, EventArgs e) {
         }
 
         private void homePageMenuItem_Click(object sender, EventArgs e) {
-            System.Diagnostics.Process.Start("https://github.com/ScriptFUSION/WAT");
+            Process.Start("https://github.com/ScriptFUSION/WAT");
         }
 
         private void exitMenuItem_Click(object sender, EventArgs e) {

Source/Properties/Settings.cs

@@ -0,0 +1,21 @@
+using System;
+using System.ComponentModel;
+using ScriptFUSION.WarframeAlertTracker.Alerts;
+
+namespace ScriptFUSION.WarframeAlertTracker.Properties {
+    internal sealed partial class Settings {
+        public event Action<AlertCollection> AlertsUpdate;
+
+        public Settings() {
+            if (Alerts == null) Alerts = new AlertCollection();
+
+            PropertyChanged += Settings_PropertyChanged;
+        }
+
+        private void Settings_PropertyChanged(object sender, PropertyChangedEventArgs e) {
+            if (e.PropertyName == "Alerts") {
+                AlertsUpdate?.Invoke(Alerts);
+            }
+        }
+    }
+}

Source/Registry/LoadAtStartupRegistryEntry.cs

@@ -0,0 +1,47 @@
+using System.Windows.Forms;
+
+namespace ScriptFUSION.WarframeAlertTracker.Registry {
+    internal sealed class LoadAtStartupRegistryEntry : RegistryEntry {
+        private readonly string value = $"\"{Application.ExecutablePath}\"";
+
+        private bool dirty;
+        private bool enabled;
+
+        public bool Enabled
+        {
+            get => enabled;
+            set
+            {
+                enabled = value;
+
+                dirty = true;
+            }
+        }
+
+        public LoadAtStartupRegistryEntry()
+            : base(Microsoft.Win32.Registry.CurrentUser, @"Software\Microsoft\Windows\CurrentVersion\Run", Application.ProductName) {
+            enabled = IsSynchronised();
+        }
+
+        public override bool IsSynchronised() {
+            using (var key = Hive.OpenSubKey(Key)) {
+                return key?.GetValue(Name)?.ToString().StartsWith(value) ?? false;
+            }
+        }
+
+        public override void Synchronise() {
+            if (!dirty) return;
+
+            using (var key = Hive.OpenSubKey(Key, true)) {
+                if (Enabled) {
+                    key.SetValue(Name, value);
+                }
+                else {
+                    key.DeleteValue(Name, false);
+                }
+            }
+
+            dirty = false;
+        }
+    }
+}

Source/Registry/RegistryEntry.cs

@@ -0,0 +1,21 @@
+using Microsoft.Win32;
+
+namespace ScriptFUSION.WarframeAlertTracker.Registry {
+    public abstract class RegistryEntry {
+        protected RegistryKey Hive { get; }
+
+        protected string Key { get; }
+
+        protected string Name { get; }
+
+        protected RegistryEntry(RegistryKey hive, string key, string name) {
+            Hive = hive;
+            Key = key;
+            Name = name;
+        }
+
+        public abstract bool IsSynchronised();
+
+        public abstract void Synchronise();
+    }
+}

Source/Registry/RegistrySettings.cs

@@ -0,0 +1,9 @@
+namespace ScriptFUSION.WarframeAlertTracker.Registry {
+    internal sealed class RegistrySettings {
+        public LoadAtStartupRegistryEntry LoadAtStartUp { get; set; } = new LoadAtStartupRegistryEntry();
+
+        public void Save() {
+            LoadAtStartUp.Synchronise();
+        }
+    }
+}

Source/WarframeAlertTracker.csproj

@@ -75,6 +75,9 @@
       <DependentUpon>OptionsForm.cs</DependentUpon>
     </Compile>
     <Compile Include="Notifier.cs" />
+    <Compile Include="Registry\LoadAtStartupRegistryEntry.cs" />
+    <Compile Include="Registry\RegistryEntry.cs" />
+    <Compile Include="Registry\RegistrySettings.cs" />
     <Compile Include="Properties\Settings.cs" />
     <Compile Include="Warframe\FissureTierParser.cs" />
     <Compile Include="Warframe\MissionType.cs" />

Source/WatApplication.cs

@@ -4,18 +4,19 @@
 using ScriptFUSION.WarframeAlertTracker.Warframe;
 using System.Windows.Forms;
 using Newtonsoft.Json.Linq;
+using ScriptFUSION.WarframeAlertTracker.Registry;
 using ScriptFUSION.WarframeAlertTracker.Properties;
 
 namespace ScriptFUSION.WarframeAlertTracker {
     internal sealed class WatApplication : ApplicationContext {
-        private Form mainForm;
+        private WatForm mainForm;
 
         /// <remarks>
         /// Hides the base implementation so the base Form is always null. This is needed to stop
         /// Application.RunMessageLoopInner forcing the form to be visible, which is undesirable for our purposes since
         /// we provide an option to start the application hidden.
         /// </remarks>
-        public new Form MainForm
+        public new WatForm MainForm
         {
             get => mainForm;
             set
@@ -30,6 +31,8 @@ internal sealed class WatApplication : ApplicationContext {
 
         public Settings Settings { get; } = Settings.Default;
 
+        public RegistrySettings RegistrySettings { get; } = new RegistrySettings();
+
         public ImageRepository ImageRepository { get; } = new ImageRepository(new ResourceDownloader(Downloader));
 
         public Task<JObject> SolNodes { get; }