Fixed: Allow text in front of cleaned Album/Track tag (#498)

* Fixed: Allow text in front of cleaned tag

* fixup! Add another test case that shouldnt be matched.
pull/485/head
Qstick 6 years ago committed by GitHub
parent d45d0ab1f6
commit 56e1f53834
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -53,10 +53,13 @@ namespace NzbDrone.Core.Test.ParserTests
}
[TestCase("Songs of Experience (Deluxe Edition)", "Songs of Experience")]
[TestCase("Songs of Experience (iTunes Deluxe Edition)", "Songs of Experience")]
[TestCase("Songs of Experience [Super Special Edition]", "Songs of Experience")]
[TestCase("Mr. Bad Guy [Special Edition]", "Mr. Bad Guy")]
[TestCase("Sweet Dreams (Album)", "Sweet Dreams")]
[TestCase("Now What?! (Limited Edition)", "Now What?!")]
[TestCase("Random Album Title (Promo CD)", "Random Album Title")]
[TestCase("Limited Edition", "Limited Edition")]
public void should_remove_common_tags_from_album_title(string title, string correct)
{
var result = Parser.Parser.CleanAlbumTitle(title);

@ -207,7 +207,7 @@ namespace NzbDrone.Core.Parser
private static readonly Regex[] CommonTagRegex = new Regex[] {
new Regex(@"(\[|\()*\b((featuring|feat.|feat|ft|ft.)\s{1}){1}\s*.*(\]|\))*", RegexOptions.IgnoreCase | RegexOptions.Compiled),
new Regex(@"(\[|\(){1}(version|limited|deluxe|single|clean|album|special|bonus|promo)+\s*.*(\]|\)){1}", RegexOptions.IgnoreCase | RegexOptions.Compiled),
new Regex(@"(?:\(|\[)(?:[^\(\[]*)(?:version|limited|deluxe|single|clean|album|special|bonus|promo)(?:[^\)\]]*)(?:\)|\])", RegexOptions.IgnoreCase | RegexOptions.Compiled)
};
public static ParsedTrackInfo ParseMusicPath(string path)

Loading…
Cancel
Save