Merge pull request #224 from dacky179/PrepareNotificationFacadeEarlier

Fix problem while publishing notifications during OnStart
PHOENIXCONTACT · May 6, 2024 · db3eeb4 · db3eeb4
2 parents 38d8975 + 053fdc8
commit db3eeb4
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/Moryx.Resources.Management/ModuleController/ModuleController.cs b/src/Moryx.Resources.Management/ModuleController/ModuleController.cs
@@ -95,6 +95,8 @@ protected override void OnInitialize()
         /// </summary>
         protected override void OnStart()
         {
+            ActivateFacade(_notificationSourceFacade);
+
             // Start type controller for resource and proxy creation
             Container.Resolve<IResourceTypeController>().Start();
 
@@ -106,7 +108,6 @@ protected override void OnStart()
             resourceManager.Start();
 
             // Activate external facade to register events
-            ActivateFacade(_notificationSourceFacade);
             ActivateFacade(_resourceManagementFacade);
         }
 
@@ -116,11 +117,12 @@ protected override void OnStart()
         protected override void OnStop()
         {
             // Tear down facades
-            DeactivateFacade(_notificationSourceFacade);
             DeactivateFacade(_resourceManagementFacade);
 
             var resourceManager = Container.Resolve<IResourceManager>();
             resourceManager.Stop();
+
+            DeactivateFacade(_notificationSourceFacade);
         }
 
         private readonly ResourceManagementFacade _resourceManagementFacade = new ResourceManagementFacade();