From ab1065a567151fd45fcf4698cd7d18708b94e35f Mon Sep 17 00:00:00 2001 From: LukePulverenti Date: Thu, 21 Feb 2013 15:26:35 -0500 Subject: removed static logger --- MediaBrowser.Common/Kernel/BaseKernel.cs | 6 ++--- .../Kernel/BasePeriodicWebSocketListener.cs | 26 +++++++++++++++++++--- 2 files changed, 25 insertions(+), 7 deletions(-) (limited to 'MediaBrowser.Common/Kernel') diff --git a/MediaBrowser.Common/Kernel/BaseKernel.cs b/MediaBrowser.Common/Kernel/BaseKernel.cs index a4a1d5b4d..fe0d619c4 100644 --- a/MediaBrowser.Common/Kernel/BaseKernel.cs +++ b/MediaBrowser.Common/Kernel/BaseKernel.cs @@ -143,7 +143,7 @@ namespace MediaBrowser.Common.Kernel get { // Lazy load - LazyInitializer.EnsureInitialized(ref _configuration, ref _configurationLoaded, ref _configurationSyncLock, () => XmlSerializer.GetXmlConfiguration(ApplicationPaths.SystemConfigurationFilePath)); + LazyInitializer.EnsureInitialized(ref _configuration, ref _configurationLoaded, ref _configurationSyncLock, () => XmlSerializer.GetXmlConfiguration(ApplicationPaths.SystemConfigurationFilePath, Logger)); return _configuration; } protected set @@ -441,8 +441,6 @@ namespace MediaBrowser.Common.Kernel AddLogTarget(logFile, "ApplicationLogFile"); - Logging.Logger.LoggerInstance = Logging.LogManager.GetLogger("App"); - OnLoggerLoaded(); } @@ -590,7 +588,7 @@ namespace MediaBrowser.Common.Kernel try { - plugin.Initialize(this); + plugin.Initialize(this, Logging.LogManager.GetLogger(plugin.GetType().Name)); Logger.Info("{0} {1} initialized.", plugin.Name, plugin.Version); } diff --git a/MediaBrowser.Common/Kernel/BasePeriodicWebSocketListener.cs b/MediaBrowser.Common/Kernel/BasePeriodicWebSocketListener.cs index 7db1ca36b..f431c5463 100644 --- a/MediaBrowser.Common/Kernel/BasePeriodicWebSocketListener.cs +++ b/MediaBrowser.Common/Kernel/BasePeriodicWebSocketListener.cs @@ -6,6 +6,7 @@ using System.Linq; using System.Net.WebSockets; using System.Threading; using System.Threading.Tasks; +using MediaBrowser.Model.Logging; namespace MediaBrowser.Common.Kernel { @@ -38,6 +39,25 @@ namespace MediaBrowser.Common.Kernel /// Task{`1}. protected abstract Task GetDataToSend(TStateType state); + /// + /// The logger + /// + protected ILogger Logger; + + /// + /// Initializes a new instance of the class. + /// + /// The logger. + protected BasePeriodicWebSocketListener(ILogger logger) + { + if (logger == null) + { + throw new ArgumentNullException("logger"); + } + + Logger = logger; + } + /// /// Processes the message internal. /// @@ -71,7 +91,7 @@ namespace MediaBrowser.Common.Kernel var cancellationTokenSource = new CancellationTokenSource(); - Logger.LogInfo("{1} Begin transmitting over websocket to {0}", message.Connection.RemoteEndPoint, GetType().Name); + Logger.Info("{1} Begin transmitting over websocket to {0}", message.Connection.RemoteEndPoint, GetType().Name); var timer = new Timer(TimerCallback, message.Connection, Timeout.Infinite, Timeout.Infinite); @@ -135,7 +155,7 @@ namespace MediaBrowser.Common.Kernel } catch (Exception ex) { - Logger.LogException("Error sending web socket message {0}", ex, Name); + Logger.ErrorException("Error sending web socket message {0}", ex, Name); DisposeConnection(tuple); } finally @@ -167,7 +187,7 @@ namespace MediaBrowser.Common.Kernel /// The connection. private void DisposeConnection(Tuple connection) { - Logger.LogInfo("{1} stop transmitting over websocket to {0}", connection.Item1.RemoteEndPoint, GetType().Name); + Logger.Info("{1} stop transmitting over websocket to {0}", connection.Item1.RemoteEndPoint, GetType().Name); try { -- cgit v1.2.3