From 6ef7e71caa9f7b72eb30714e5426d563712ba511 Mon Sep 17 00:00:00 2001 From: Thomas Gillen Date: Thu, 19 Jun 2014 21:35:35 +0100 Subject: [PATCH] Fixed absolute episodes being being resolved and the tvdb provider not identifying them Episode resolver will only consider absolute episodes (those without a season) if the media type is TV Series for the collection. --- .../Entities/TV/Episode.cs | 2 +- MediaBrowser.Controller/Library/TVUtils.cs | 37 +++++++++---------- .../TV/TvdbEpisodeProvider.cs | 20 +++++----- .../Library/Resolvers/TV/SeriesResolver.cs | 2 +- 4 files changed, 30 insertions(+), 31 deletions(-) diff --git a/MediaBrowser.Controller/Entities/TV/Episode.cs b/MediaBrowser.Controller/Entities/TV/Episode.cs index be761ef662..b9630a66f4 100644 --- a/MediaBrowser.Controller/Entities/TV/Episode.cs +++ b/MediaBrowser.Controller/Entities/TV/Episode.cs @@ -273,7 +273,7 @@ namespace MediaBrowser.Controller.Entities.TV { if (!IndexNumber.HasValue && !string.IsNullOrEmpty(Path)) { - IndexNumber = TVUtils.GetEpisodeNumberFromFile(Path, Parent is Season); + IndexNumber = TVUtils.GetEpisodeNumberFromFile(Path, true); // If a change was made record it if (IndexNumber.HasValue) diff --git a/MediaBrowser.Controller/Library/TVUtils.cs b/MediaBrowser.Controller/Library/TVUtils.cs index 7841a32aed..64f3a3b4b7 100644 --- a/MediaBrowser.Controller/Library/TVUtils.cs +++ b/MediaBrowser.Controller/Library/TVUtils.cs @@ -93,27 +93,26 @@ namespace MediaBrowser.Controller.Library }; /// - /// To avoid the following matching movies they are only valid when contained in a folder which has been matched as a being season + /// To avoid the following matching movies they are only valid when contained in a folder which has been matched as a being season, or the media type is TV series /// - private static readonly Regex[] EpisodeExpressionsInASeasonFolder = + private static readonly Regex[] EpisodeExpressionsWithoutSeason = { + new Regex( + @".*[\\\/](?\d{1,3})\.\w+$", + RegexOptions.Compiled), + // "01.avi" new Regex( @".*(\\|\/)(?\d{1,2})\s?-\s?[^\\\/]*$", RegexOptions.Compiled), - // 01 - blah.avi, 01-blah.avi - new Regex( - @".*(\\|\/)(?\d{1,2})[^\d\\]*[^\\\/]*$", + // "01 - blah.avi", "01-blah.avi" + new Regex( + @".*(\\|\/)(?\d{1,2})\.[^\\\/]+$", RegexOptions.Compiled), - // 01.avi, 01.blah.avi "01 - 22 blah.avi" + // "01.blah.avi" new Regex( - @".*(\\|\/)(?\d)(?\d{1,2})[^\d\\]+[^\\\/]*$", + @".*[\\\/][^\\\/]* - (?\d{1,3})[^\\\/]*$", RegexOptions.Compiled), - // 01.avi, 01.blah.avi - new Regex( - @".*(\\|\/)\D*\d+(?\d{2})", - RegexOptions.Compiled) - // hell0 - 101 - hello.avi - + // "blah - 01.avi", "blah 2 - 01.avi", "blah - 01 blah.avi", "blah 2 - 01 blah", "blah - 01 - blah.avi", "blah 2 - 01 - blah" }; /// @@ -197,7 +196,7 @@ namespace MediaBrowser.Controller.Library /// The path. /// The file system children. /// true if [is series folder] [the specified path]; otherwise, false. - public static bool IsSeriesFolder(string path, IEnumerable fileSystemChildren, IDirectoryService directoryService) + public static bool IsSeriesFolder(string path, bool considerSeasonlessSeries, IEnumerable fileSystemChildren, IDirectoryService directoryService) { // A folder with more than 3 non-season folders in will not becounted as a series var nonSeriesFolders = 0; @@ -236,7 +235,7 @@ namespace MediaBrowser.Controller.Library if (EntityResolutionHelper.IsVideoFile(fullName) || EntityResolutionHelper.IsVideoPlaceHolder(fullName)) { - if (GetEpisodeNumberFromFile(fullName, false).HasValue) + if (GetEpisodeNumberFromFile(fullName, considerSeasonlessSeries).HasValue) { return true; } @@ -251,9 +250,9 @@ namespace MediaBrowser.Controller.Library /// Episodes the number from file. /// /// The full path. - /// if set to true [is in season]. + /// if set to true [is in season]. /// System.String. - public static int? GetEpisodeNumberFromFile(string fullPath, bool isInSeason) + public static int? GetEpisodeNumberFromFile(string fullPath, bool considerSeasonlessNames) { string fl = fullPath.ToLower(); foreach (var r in EpisodeExpressions) @@ -262,9 +261,9 @@ namespace MediaBrowser.Controller.Library if (m.Success) return ParseEpisodeNumber(m.Groups["epnumber"].Value); } - if (isInSeason) + if (considerSeasonlessNames) { - var match = EpisodeExpressionsInASeasonFolder.Select(r => r.Match(fl)) + var match = EpisodeExpressionsWithoutSeason.Select(r => r.Match(fl)) .FirstOrDefault(m => m.Success); if (match != null) diff --git a/MediaBrowser.Providers/TV/TvdbEpisodeProvider.cs b/MediaBrowser.Providers/TV/TvdbEpisodeProvider.cs index dd9f3af8b5..ef9f5427c4 100644 --- a/MediaBrowser.Providers/TV/TvdbEpisodeProvider.cs +++ b/MediaBrowser.Providers/TV/TvdbEpisodeProvider.cs @@ -217,13 +217,8 @@ namespace MediaBrowser.Providers.TV var episodeNumber = identity.IndexNumber; var seasonOffset = TvdbSeriesProvider.GetSeriesOffset(seriesProviderIds) ?? 0; var seasonNumber = identity.SeasonIndex + seasonOffset; - - if (seasonNumber == null) - { - return null; - } - - var file = Path.Combine(seriesDataPath, string.Format("episode-{0}-{1}.xml", seasonNumber.Value, episodeNumber)); + + string file; var success = false; var usingAbsoluteData = false; @@ -236,14 +231,18 @@ namespace MediaBrowser.Providers.TV try { - FetchMainEpisodeInfo(episode, file, cancellationToken); + if (seasonNumber != null) + { + file = Path.Combine(seriesDataPath, string.Format("episode-{0}-{1}.xml", seasonNumber.Value, episodeNumber)); + FetchMainEpisodeInfo(episode, file, cancellationToken); - success = true; + success = true; + } } catch (FileNotFoundException) { // Could be using absolute numbering - if (seasonNumber.Value != 1) + if (seasonNumber.HasValue && seasonNumber.Value != 1) { throw; } @@ -255,6 +254,7 @@ namespace MediaBrowser.Providers.TV FetchMainEpisodeInfo(episode, file, cancellationToken); usingAbsoluteData = true; + success = true; } var end = identity.IndexNumberEnd ?? episodeNumber; diff --git a/MediaBrowser.Server.Implementations/Library/Resolvers/TV/SeriesResolver.cs b/MediaBrowser.Server.Implementations/Library/Resolvers/TV/SeriesResolver.cs index e97bc97f85..6b376d3b46 100644 --- a/MediaBrowser.Server.Implementations/Library/Resolvers/TV/SeriesResolver.cs +++ b/MediaBrowser.Server.Implementations/Library/Resolvers/TV/SeriesResolver.cs @@ -74,7 +74,7 @@ namespace MediaBrowser.Server.Implementations.Library.Resolvers.TV return null; } - if (args.ContainsMetaFileByName("series.xml") || filename.IndexOf("[tvdbid=", StringComparison.OrdinalIgnoreCase) != -1 || TVUtils.IsSeriesFolder(args.Path, args.FileSystemChildren, args.DirectoryService)) + if (args.ContainsMetaFileByName("series.xml") || filename.IndexOf("[tvdbid=", StringComparison.OrdinalIgnoreCase) != -1 || TVUtils.IsSeriesFolder(args.Path, collectionType == CollectionType.TvShows, args.FileSystemChildren, args.DirectoryService)) { return new Series(); }