Merge pull request #138 from dennis/initialize-winformui-subscription…

…-earlier

WinFormUI wouldn't always pick up creation events

CHANGELOG.md

@@ -12,6 +12,7 @@
    Use `parse_json(event.Json)` to get a LuaTable with the values as were sent 
    via the event. Also adds `generate_json(luaTable)` to generate json string from 
    a Lua table.
+ - Bugfix: WinFormUI wouldn't always pick up creation events, while it was initializing it's window.
 
 
 ## [0.9.0](https://github.com/dennis/slipstream/releases/tag/v0.9.0) (2021-08-21)

Components/WinFormUI/Forms/MainWindow.cs

@@ -56,6 +56,7 @@ public MainWindow(
             IApplicationVersionService applicationVersionService,
             IEventHandlerController eventHandlerController,
             IEventSerdeService eventSerdeService,
+            IEventBusSubscription subscription,
             bool deepView
             )
         {
@@ -71,6 +72,8 @@ bool deepView
             BroadcastEnvelope = new EventEnvelope(instanceId);
             DeepView = deepView;
 
+            EventBusSubscription = subscription;
+
             InitializeComponent();
 
             EventGridView.Columns.Add("uptime", "Uptime");
@@ -138,7 +141,6 @@ private void MainWindow_FormClosing(object? sender, FormClosingEventArgs e)
 
         private void MainWindow_Load(object? sender, EventArgs e)
         {
-            EventBusSubscription = EventBus.RegisterListener(InstanceId, fromBeginning: true, promiscuousMode: true);
             EventHandlerThreadCancellationToken = EventHandlerThreadCts.Token;
             EventHandlerThread = new Thread(new ThreadStart(EventListenerMain))
             {

Components/WinFormUI/Lua/WinFormUIInstanceThread.cs

@@ -45,6 +45,8 @@ IEventSerdeService eventSerdeService
         [STAThreadAttribute]
         protected override void Main()
         {
+            var subscription = EventBus.RegisterListener(InstanceId, fromBeginning: true, promiscuousMode: true);
+
             Application.Run(new MainWindow(
                 InstanceId,
                 this,
@@ -55,6 +57,7 @@ protected override void Main()
                 ApplicationVersionService,
                 EventHandlerController,
                 EventSerdeService,
+                subscription,
                 DeepView
            ));
         }