From 1b21f4a2d7d9c4ab5f3e5e96c0928bdf65278b52 Mon Sep 17 00:00:00 2001
From: Mark McDowall <markus.mcd5@gmail.com>
Date: Wed, 12 Feb 2014 00:59:22 -0800
Subject: [PATCH] Fixed: Double periods before extensions when episode title
 ends in a period

---
 .../OrganizerTests/GetNewFilenameFixture.cs     | 17 +++++++++++++++++
 src/NzbDrone.Core/Organizer/FileNameBuilder.cs  |  6 ++++--
 2 files changed, 21 insertions(+), 2 deletions(-)

diff --git a/src/NzbDrone.Core.Test/OrganizerTests/GetNewFilenameFixture.cs b/src/NzbDrone.Core.Test/OrganizerTests/GetNewFilenameFixture.cs
index 338e3059c..45043235a 100644
--- a/src/NzbDrone.Core.Test/OrganizerTests/GetNewFilenameFixture.cs
+++ b/src/NzbDrone.Core.Test/OrganizerTests/GetNewFilenameFixture.cs
@@ -371,5 +371,22 @@ namespace NzbDrone.Core.Test.OrganizerTests
             Subject.BuildFilename(new List<Episode> { _episode1 }, _series, _episodeFile)
                    .Should().Be("30 Rock - 30.Rock.S01E01.xvid-LOL");
         }
+
+        [Test]
+        public void should_trim_periods_from_end_of_episode_title()
+        {
+            _namingConfig.StandardEpisodeFormat = "{Series Title} - S{season:00}E{episode:00} - {Episode Title}";
+            _namingConfig.MultiEpisodeStyle = 3;
+
+            var episode = Builder<Episode>.CreateNew()
+                            .With(e => e.Title = "Part 1.")
+                            .With(e => e.SeasonNumber = 6)
+                            .With(e => e.EpisodeNumber = 6)
+                            .Build();
+
+
+            Subject.BuildFilename(new List<Episode> { episode }, new Series { Title = "30 Rock" }, _episodeFile)
+                   .Should().Be("30 Rock - S06E06 - Part 1");
+        }
     }
 }
\ No newline at end of file
diff --git a/src/NzbDrone.Core/Organizer/FileNameBuilder.cs b/src/NzbDrone.Core/Organizer/FileNameBuilder.cs
index 2997be7d5..c158cb514 100644
--- a/src/NzbDrone.Core/Organizer/FileNameBuilder.cs
+++ b/src/NzbDrone.Core/Organizer/FileNameBuilder.cs
@@ -44,6 +44,8 @@ namespace NzbDrone.Core.Organizer
         public static readonly Regex SeriesTitleRegex = new Regex(@"(?<token>\{(?:Series)(?<separator>\s|\.|-|_)Title\})",
                                                                             RegexOptions.Compiled | RegexOptions.IgnoreCase);
 
+        private static readonly char[] EpisodeTitleTrimCharaters = new[] { ' ', '.' };
+
         public FileNameBuilder(INamingConfigService namingConfigService,
                                IQualityDefinitionService qualityDefinitionService,
                                ICacheManger cacheManger,
@@ -88,7 +90,7 @@ namespace NzbDrone.Core.Organizer
             var pattern = namingConfig.StandardEpisodeFormat;
             var episodeTitles = new List<string>
             {
-                sortedEpisodes.First().Title
+                sortedEpisodes.First().Title.TrimEnd(EpisodeTitleTrimCharaters)
             };
 
             var tokenValues = new Dictionary<string, string>(FilenameBuilderTokenEqualityComparer.Instance);
@@ -140,7 +142,7 @@ namespace NzbDrone.Core.Organizer
                             break;
                     }
 
-                    episodeTitles.Add(episode.Title);
+                    episodeTitles.Add(episode.Title.TrimEnd(EpisodeTitleTrimCharaters));
                 }
 
                 seasonEpisodePattern = ReplaceNumberTokens(seasonEpisodePattern, sortedEpisodes);