Added Notifier to queue displaying of matching alerts.

Author: Bilge

Source/Controls/ControlExtension.cs

@@ -1,5 +1,4 @@
 using System.ComponentModel;
-using System.Drawing;
 using System.Windows.Forms;
 using Bitmap = System.Drawing.Bitmap;
 

Source/Controls/FissureList.Designer.cs

@@ -39,9 +39,9 @@ protected override CreateParams CreateParams
         /// <summary>
         /// Gets or sets the animation time in seconds.
         /// </summary>
-        private double AnimationTime { get; } = .5;
+        private double AnimationTime { get; } = .45;
 
-        private double HoldTime { get; } = 2.5;
+        private double HoldTime { get; } = 2.75;
 
         public NotificationForm(Control childControl) {
             InitializeComponent();
@@ -106,8 +106,9 @@ private async void NotificationForm_Load(object sender, EventArgs e) {
             try {
                 await Animate();
             }
+            // Owning form has been disposed.
             catch (ObjectDisposedException) {
-                // Continue closing the form.
+                // Abort animation.
             }
 
             Close();

Source/Forms/NotificationForm.cs

@@ -16,9 +16,8 @@ internal WatForm(WatApplication application) {
 
             Application = application;
 
-            var solNodes = application.SolNodesDownloader.Download();
             application.CurrentWorldState.Update +=
-                async worldState => OnWorldStateUpdate(worldState.Fissures, await solNodes);
+                async worldState => OnWorldStateUpdate(worldState.Fissures, await application.SolNodes);
 
             application.AlertsUpdate += OnAlertsUpdate;
 

Source/Forms/WatForm.Designer.cs

@@ -0,0 +1,70 @@
+using System;
+using System.Collections.Generic;
+using System.Threading.Tasks;
+using System.Windows.Forms;
+using ScriptFUSION.WarframeAlertTracker.Controls;
+using ScriptFUSION.WarframeAlertTracker.Forms;
+using ScriptFUSION.WarframeAlertTracker.Warframe;
+
+namespace ScriptFUSION.WarframeAlertTracker {
+    internal sealed class Notifier {
+        private readonly Queue<Control> queue = new Queue<Control>();
+
+        private WatApplication Application { get; }
+
+        private NotificationForm Notification { get; set; }
+
+        public Notifier(WatApplication application) {
+            Application = application;
+            application.CurrentWorldState.NewObject += OnNewWorldStateObject;
+        }
+
+        private void Notify(Control control) {
+            if (Notification != null) {
+                Queue(control);
+                return;
+            }
+
+            Notification = new NotificationForm(control);
+            Notification.Closed += delegate {
+                Notification = null;
+                Next();
+            };
+
+            Notification.Show(Application.MainForm);
+        }
+
+        private void Queue(Control control) {
+            queue.Enqueue(control);
+        }
+
+        private void Next() {
+            if (queue.Count > 0) {
+                Notify(queue.Dequeue());
+            }
+        }
+
+        private async void OnNewWorldStateObject(IWorldStateObject worldStateObject) {
+            if (!Application.AlertCollection.Matches(worldStateObject)) {
+                return;
+            }
+
+            Notify(await ControlFromWorldStateObject(worldStateObject));
+        }
+
+        private async Task<Control> ControlFromWorldStateObject(IWorldStateObject worldStateObject) {
+            switch (worldStateObject) {
+                case Fissure fissure:
+                    var fissureControl = new FissureControl {
+                        ImageRepository = Application.ImageRepository,
+                        Active = true,
+                    };
+                    fissureControl.Update(fissure, await Application.SolNodes);
+
+                    return fissureControl;
+            }
+
+            throw new ApplicationException("Unrecognized world state object type.");
+        }
+    }
+}

Source/Forms/WatForm.cs

@@ -1,4 +1,6 @@
 using System;
+using System.Collections.Generic;
+using System.Linq;
 using System.Threading.Tasks;
 
 namespace ScriptFUSION.WarframeAlertTracker.Warframe {
@@ -8,6 +10,8 @@ namespace ScriptFUSION.WarframeAlertTracker.Warframe {
     internal sealed class CurrentWorldState {
         public event Action<WorldState> Update;
 
+        public event Action<IWorldStateObject> NewObject;
+
         public WorldState CurrentState { get; private set; }
 
         /// <summary>
@@ -40,9 +44,22 @@ private async void Download() {
             var worldState = await Downloader.Download();
 
             if (CurrentState == null || worldState?.Time > CurrentState.Time) {
-                CurrentState = worldState;
-                Update?.Invoke(worldState);
+                UpdateWorldState(CurrentState, worldState);
             }
         }
+
+        private void UpdateWorldState(WorldState oldState, WorldState newState) {
+            CurrentState = newState;
+            Update?.Invoke(newState);
+
+            foreach (var worldStateObject in DetectNewObjects(oldState, newState)) {
+                NewObject?.Invoke(worldStateObject);
+            }
+        }
+
+        private static IEnumerable<IWorldStateObject> DetectNewObjects(WorldState oldState, WorldState newState) {
+            return oldState == null ? Enumerable.Empty<IWorldStateObject>() :
+                newState.WorldStateObjects.Except(oldState.WorldStateObjects, new WorldStateObjectComparer());
+        }
     }
 }

Source/Notifier.cs

@@ -3,6 +3,8 @@
 namespace ScriptFUSION.WarframeAlertTracker.Warframe {
 
     internal interface IWorldStateObject {
+        string Id { get; }
+
         bool Matches(Alert alert);
     }
 }

Source/Warframe/CurrentWorldState.cs

@@ -0,0 +1,13 @@
+using System.Collections.Generic;
+
+namespace ScriptFUSION.WarframeAlertTracker.Warframe {
+    internal class WorldStateObjectComparer : EqualityComparer<IWorldStateObject> {
+        public override bool Equals(IWorldStateObject x, IWorldStateObject y) {
+            return x.Id == y.Id;
+        }
+
+        public override int GetHashCode(IWorldStateObject obj) {
+            return obj.Id.GetHashCode();
+        }
+    }
+}

Source/Warframe/IWorldStateObject.cs

@@ -43,6 +43,7 @@
     </Reference>
     <Reference Include="System" />
     <Reference Include="System.Core" />
+    <Reference Include="System.Web" />
     <Reference Include="System.Xml.Linq" />
     <Reference Include="System.Data.DataSetExtensions" />
     <Reference Include="Microsoft.CSharp" />
@@ -72,6 +73,7 @@
     <Compile Include="Forms\NotificationOptionsForm.Designer.cs">
       <DependentUpon>NotificationOptionsForm.cs</DependentUpon>
     </Compile>
+    <Compile Include="Notifier.cs" />
     <Compile Include="Warframe\FissureTierParser.cs" />
     <Compile Include="Warframe\MissionType.cs" />
     <Compile Include="Warframe\MissionTypeFormatter.cs" />
@@ -135,6 +137,7 @@
       <DependentUpon>WatForm.cs</DependentUpon>
     </Compile>
     <Compile Include="Warframe\Fissure.cs" />
+    <Compile Include="Warframe\WorldStateObjectComparer.cs" />
     <Compile Include="WatApplication.cs" />
     <Compile Include="Downloader.cs" />
     <Compile Include="Program.cs" />