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.Controller/Providers/BaseImageEnhancer.cs | 9 +++++++-- MediaBrowser.Controller/Providers/BaseItemXmlParser.cs | 15 +++++++++++++++ .../Providers/FolderProviderFromXml.cs | 4 ++-- .../Providers/Movies/MovieProviderFromXml.cs | 4 ++-- .../Providers/TV/EpisodeProviderFromXml.cs | 2 +- MediaBrowser.Controller/Providers/TV/EpisodeXmlParser.cs | 10 ++++++++++ .../Providers/TV/SeriesProviderFromXml.cs | 2 +- MediaBrowser.Controller/Providers/TV/SeriesXmlParser.cs | 15 ++++++++++++--- 8 files changed, 50 insertions(+), 11 deletions(-) (limited to 'MediaBrowser.Controller/Providers') diff --git a/MediaBrowser.Controller/Providers/BaseImageEnhancer.cs b/MediaBrowser.Controller/Providers/BaseImageEnhancer.cs index bd60003beb..0bcfff183d 100644 --- a/MediaBrowser.Controller/Providers/BaseImageEnhancer.cs +++ b/MediaBrowser.Controller/Providers/BaseImageEnhancer.cs @@ -2,6 +2,7 @@ using MediaBrowser.Controller.Entities; using MediaBrowser.Model.Drawing; using MediaBrowser.Model.Entities; +using MediaBrowser.Model.Logging; using System; using System.Drawing; using System.Threading.Tasks; @@ -13,6 +14,10 @@ namespace MediaBrowser.Controller.Providers /// public abstract class BaseImageEnhancer : IDisposable { + /// + /// The logger + /// + private static readonly ILogger Logger = LogManager.GetLogger("ImageEnhancer"); /// /// Return true only if the given image for the given item will be enhanced by this enhancer. /// @@ -96,7 +101,7 @@ namespace MediaBrowser.Controller.Providers var typeName = GetType().Name; - Logger.LogDebugInfo("Running {0} for {1}", typeName, item.Path ?? item.Name ?? "--Unknown--"); + Logger.Debug("Running {0} for {1}", typeName, item.Path ?? item.Name ?? "--Unknown--"); try { @@ -104,7 +109,7 @@ namespace MediaBrowser.Controller.Providers } catch (Exception ex) { - Logger.LogException("{0} failed enhancing {1}", ex, typeName, item.Name); + Logger.ErrorException("{0} failed enhancing {1}", ex, typeName, item.Name); throw; } diff --git a/MediaBrowser.Controller/Providers/BaseItemXmlParser.cs b/MediaBrowser.Controller/Providers/BaseItemXmlParser.cs index 0869b25bc4..e54e6bfc2d 100644 --- a/MediaBrowser.Controller/Providers/BaseItemXmlParser.cs +++ b/MediaBrowser.Controller/Providers/BaseItemXmlParser.cs @@ -1,6 +1,7 @@ using MediaBrowser.Common.Extensions; using MediaBrowser.Controller.Entities; using MediaBrowser.Model.Entities; +using MediaBrowser.Model.Logging; using System; using System.Collections.Generic; using System.Linq; @@ -16,6 +17,20 @@ namespace MediaBrowser.Controller.Providers public class BaseItemXmlParser where T : BaseItem, new() { + /// + /// The logger + /// + protected ILogger Logger { get; private set; } + + /// + /// Initializes a new instance of the class. + /// + /// The logger. + public BaseItemXmlParser(ILogger logger) + { + Logger = logger; + } + /// /// Fetches metadata for an item from one xml file /// diff --git a/MediaBrowser.Controller/Providers/FolderProviderFromXml.cs b/MediaBrowser.Controller/Providers/FolderProviderFromXml.cs index 110502bc1f..ab0f37b96e 100644 --- a/MediaBrowser.Controller/Providers/FolderProviderFromXml.cs +++ b/MediaBrowser.Controller/Providers/FolderProviderFromXml.cs @@ -66,13 +66,13 @@ namespace MediaBrowser.Controller.Providers private bool Fetch(BaseItem item, CancellationToken cancellationToken) { cancellationToken.ThrowIfCancellationRequested(); - + var metadataFile = item.ResolveArgs.GetMetaFileByPath(Path.Combine(item.MetaLocation, "folder.xml")); if (metadataFile.HasValue) { var path = metadataFile.Value.Path; - new BaseItemXmlParser().Fetch((Folder)item, path, cancellationToken); + new BaseItemXmlParser(Logger).Fetch((Folder)item, path, cancellationToken); SetLastRefreshed(item, DateTime.UtcNow); return true; } diff --git a/MediaBrowser.Controller/Providers/Movies/MovieProviderFromXml.cs b/MediaBrowser.Controller/Providers/Movies/MovieProviderFromXml.cs index b87c71df3d..b180fa580f 100644 --- a/MediaBrowser.Controller/Providers/Movies/MovieProviderFromXml.cs +++ b/MediaBrowser.Controller/Providers/Movies/MovieProviderFromXml.cs @@ -75,11 +75,11 @@ namespace MediaBrowser.Controller.Providers.Movies var boxset = item as BoxSet; if (boxset != null) { - new BaseItemXmlParser().Fetch(boxset, path, cancellationToken); + new BaseItemXmlParser(Logger).Fetch(boxset, path, cancellationToken); } else { - new BaseItemXmlParser().Fetch((Movie)item, path, cancellationToken); + new BaseItemXmlParser(Logger).Fetch((Movie)item, path, cancellationToken); } SetLastRefreshed(item, DateTime.UtcNow); return true; diff --git a/MediaBrowser.Controller/Providers/TV/EpisodeProviderFromXml.cs b/MediaBrowser.Controller/Providers/TV/EpisodeProviderFromXml.cs index df46b7167f..08d0988eb9 100644 --- a/MediaBrowser.Controller/Providers/TV/EpisodeProviderFromXml.cs +++ b/MediaBrowser.Controller/Providers/TV/EpisodeProviderFromXml.cs @@ -115,7 +115,7 @@ namespace MediaBrowser.Controller.Providers.TV return false; } - new EpisodeXmlParser().Fetch(item, metadataFile, cancellationToken); + new EpisodeXmlParser(Logger).Fetch(item, metadataFile, cancellationToken); return true; } } diff --git a/MediaBrowser.Controller/Providers/TV/EpisodeXmlParser.cs b/MediaBrowser.Controller/Providers/TV/EpisodeXmlParser.cs index 7133d87451..4b8219e004 100644 --- a/MediaBrowser.Controller/Providers/TV/EpisodeXmlParser.cs +++ b/MediaBrowser.Controller/Providers/TV/EpisodeXmlParser.cs @@ -1,4 +1,5 @@ using MediaBrowser.Controller.Entities.TV; +using MediaBrowser.Model.Logging; using System.IO; using System.Xml; @@ -9,6 +10,15 @@ namespace MediaBrowser.Controller.Providers.TV /// public class EpisodeXmlParser : BaseItemXmlParser { + /// + /// Initializes a new instance of the class. + /// + /// The logger. + public EpisodeXmlParser(ILogger logger) + : base(logger) + { + } + /// /// Fetches the data from XML node. /// diff --git a/MediaBrowser.Controller/Providers/TV/SeriesProviderFromXml.cs b/MediaBrowser.Controller/Providers/TV/SeriesProviderFromXml.cs index 728ac0549c..c92841fb65 100644 --- a/MediaBrowser.Controller/Providers/TV/SeriesProviderFromXml.cs +++ b/MediaBrowser.Controller/Providers/TV/SeriesProviderFromXml.cs @@ -74,7 +74,7 @@ namespace MediaBrowser.Controller.Providers.TV { var path = metadataFile.Value.Path; - new SeriesXmlParser().Fetch((Series)item, path, cancellationToken); + new SeriesXmlParser(Logger).Fetch((Series)item, path, cancellationToken); SetLastRefreshed(item, DateTime.UtcNow); return true; diff --git a/MediaBrowser.Controller/Providers/TV/SeriesXmlParser.cs b/MediaBrowser.Controller/Providers/TV/SeriesXmlParser.cs index 7516904255..2822a43b29 100644 --- a/MediaBrowser.Controller/Providers/TV/SeriesXmlParser.cs +++ b/MediaBrowser.Controller/Providers/TV/SeriesXmlParser.cs @@ -1,7 +1,7 @@ -using MediaBrowser.Common.Logging; -using MediaBrowser.Controller.Entities.TV; +using MediaBrowser.Controller.Entities.TV; using MediaBrowser.Controller.Resolvers.TV; using MediaBrowser.Model.Entities; +using MediaBrowser.Model.Logging; using System; using System.Xml; @@ -12,6 +12,15 @@ namespace MediaBrowser.Controller.Providers.TV /// public class SeriesXmlParser : BaseItemXmlParser { + /// + /// Initializes a new instance of the class. + /// + /// The logger. + public SeriesXmlParser(ILogger logger) + : base(logger) + { + } + /// /// Fetches the data from XML node. /// @@ -74,7 +83,7 @@ namespace MediaBrowser.Controller.Providers.TV } else { - Logger.LogInfo("Unrecognized series status: " + status); + Logger.Info("Unrecognized series status: " + status); } } -- cgit v1.2.3