diff --git a/NzbDrone.Core.Test/MediaFileTests/EpisodeImportTests/ImportDecisionMakerFixture.cs b/NzbDrone.Core.Test/MediaFileTests/EpisodeImportTests/ImportDecisionMakerFixture.cs index da2e0355f8..2f0f06320f 100644 --- a/NzbDrone.Core.Test/MediaFileTests/EpisodeImportTests/ImportDecisionMakerFixture.cs +++ b/NzbDrone.Core.Test/MediaFileTests/EpisodeImportTests/ImportDecisionMakerFixture.cs @@ -150,18 +150,5 @@ public void failed_parse_shouldnt_blowup_the_process() ExceptionVerification.ExpectedErrors(3); } - - [Test] - public void should_use_filename_without_extension_when_getting_episodes_from_ParsingService() - { - var expectedFilename = Path.GetFileNameWithoutExtension(_videoFiles.First()); - - GivenSpecifications(_pass1, _pass2, _pass3); - - Subject.GetImportDecisions(_videoFiles, _series); - - Mocker.GetMock().Verify(v => v.GetEpisodes(expectedFilename, _series), Times.Once()); - } } - } \ No newline at end of file diff --git a/NzbDrone.Core/MediaFiles/EpisodeImport/ImportApprovedEpisodes.cs b/NzbDrone.Core/MediaFiles/EpisodeImport/ImportApprovedEpisodes.cs index 798e686dd0..e4557d2e39 100644 --- a/NzbDrone.Core/MediaFiles/EpisodeImport/ImportApprovedEpisodes.cs +++ b/NzbDrone.Core/MediaFiles/EpisodeImport/ImportApprovedEpisodes.cs @@ -77,7 +77,7 @@ public List Import(List decisions, bool newDownl } catch (Exception e) { - _logger.WarnException("Couldn't add report to download queue. " + localEpisode, e); + _logger.WarnException("Couldn't import episode " + localEpisode, e); } } diff --git a/NzbDrone.Core/MediaFiles/EpisodeImport/ImportDecisionMaker.cs b/NzbDrone.Core/MediaFiles/EpisodeImport/ImportDecisionMaker.cs index 8b4d8bd1bf..f1f16a577c 100644 --- a/NzbDrone.Core/MediaFiles/EpisodeImport/ImportDecisionMaker.cs +++ b/NzbDrone.Core/MediaFiles/EpisodeImport/ImportDecisionMaker.cs @@ -43,8 +43,7 @@ private IEnumerable GetDecisions(IEnumerable videoFiles, try { - var fileWithoutExtension = Path.GetFileNameWithoutExtension(file); - var parsedEpisode = _parsingService.GetEpisodes(Path.GetFileNameWithoutExtension(file), series); + var parsedEpisode = _parsingService.GetEpisodes(file, series); if (parsedEpisode != null) { diff --git a/NzbDrone.Core/Parser/Model/LocalEpisode.cs b/NzbDrone.Core/Parser/Model/LocalEpisode.cs index cff8d01286..f78013a80e 100644 --- a/NzbDrone.Core/Parser/Model/LocalEpisode.cs +++ b/NzbDrone.Core/Parser/Model/LocalEpisode.cs @@ -17,5 +17,10 @@ public class LocalEpisode public QualityModel Quality { get; set; } public int SeasonNumber { get { return Episodes.Select(c => c.SeasonNumber).Distinct().Single(); } } + + public override string ToString() + { + return Path; + } } } \ No newline at end of file diff --git a/NzbDrone.Core/Parser/ParsingService.cs b/NzbDrone.Core/Parser/ParsingService.cs index b2e39af830..4cd41a8f17 100644 --- a/NzbDrone.Core/Parser/ParsingService.cs +++ b/NzbDrone.Core/Parser/ParsingService.cs @@ -1,4 +1,5 @@ using System.Collections.Generic; +using System.IO; using System.Linq; using NLog; using NzbDrone.Core.Parser.Model; @@ -8,7 +9,7 @@ namespace NzbDrone.Core.Parser { public interface IParsingService { - LocalEpisode GetEpisodes(string fileName, Series series); + LocalEpisode GetEpisodes(string filename, Series series); Series GetSeries(string title); RemoteEpisode Map(ParsedEpisodeInfo parsedEpisodeInfo); } @@ -26,9 +27,9 @@ public ParsingService(IEpisodeService episodeService, ISeriesService seriesServi _logger = logger; } - public LocalEpisode GetEpisodes(string fileName, Series series) + public LocalEpisode GetEpisodes(string filename, Series series) { - var parsedEpisodeInfo = Parser.ParseTitle(fileName); + var parsedEpisodeInfo = Parser.ParsePath(filename); if (parsedEpisodeInfo == null) { @@ -47,7 +48,7 @@ public LocalEpisode GetEpisodes(string fileName, Series series) Series = series, Quality = parsedEpisodeInfo.Quality, Episodes = episodes, - Path = fileName, + Path = filename, ParsedEpisodeInfo = parsedEpisodeInfo }; }