[fixed] ServerAsAServiceSettings branch: ServiceMonitor fails to start (1 Viewer)

[chefkoch]

Branch: https://github.com/MediaPortal/MediaPortal-2/tree/FEAT3823-ServerAsService_ServerSettings
Commit: https://github.com/MediaPortal/MediaPortal-2/commit/87ae55772701607c953e5eb96e0384333315c94f

ServiceMonitor fails to start with the following exception.

== Exception: MediaPortal.Common.Exceptions.ServiceNotFoundException: Could not find the MediaPortal.Common.Messaging.IMessageBroker service
  bei MediaPortal.Common.ServiceRegistration.GetService(Type type, Boolean throwIfNotFound)
  bei MediaPortal.Common.ServiceRegistration.Get[T]()
  bei MediaPortal.Common.Runtime.SystemMessaging.SendSystemStateChangeMessage(SystemState newState)
  bei MediaPortal.Common.Services.Runtime.SystemStateService.SwitchSystemState(SystemState newState, Boolean sendMessage)
  bei MediaPortal.ServiceMonitor.App.OnStartup(Object sender, StartupEventArgs args)
  Message: Could not find the MediaPortal.Common.Messaging.IMessageBroker service
  Site  : System.Object GetService(System.Type, Boolean)
  Source : MediaPortal.Common
== Stack Trace:
  bei MediaPortal.Common.ServiceRegistration.GetService(Type type, Boolean throwIfNotFound)
  bei MediaPortal.Common.ServiceRegistration.Get[T]()
  bei MediaPortal.Common.Runtime.SystemMessaging.SendSystemStateChangeMessage(SystemState newState)
  bei MediaPortal.Common.Services.Runtime.SystemStateService.SwitchSystemState(SystemState newState, Boolean sendMessage)
  bei MediaPortal.ServiceMonitor.App.OnStartup(Object sender, StartupEventArgs args)

 

[morpheus_xx]

Ah... I think I know what's the reason: It could be the change from "Multimonitor support" branch, that splitted the registration of coreservices into two parts. Probably the new call is missing.

Will check this later, thanks.

 

[chefkoch]

Will check this later, thanks.

I forgot to update the thread, because I already solved the issue. Could you please review this commit, whether I missed something?
https://github.com/MediaPortal/MediaPortal-2/commit/6027b4a3f03e2ec7e9773c7e5546d0a0f73d988c

 

[morpheus_xx]

Yes, exactly this one (but without additional line break )

 

[chefkoch]

Yes, exactly this one (but without additional line break )

Which line break ?