From 7faf2be79b3e90e3fc6ff59f104d290593a042ba Mon Sep 17 00:00:00 2001 From: Luke Pulverenti Date: Tue, 11 Jun 2013 12:10:41 -0400 Subject: [PATCH] trim genre names from id3 tags --- .../MediaInfo/FFProbeAudioInfoProvider.cs | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/MediaBrowser.Providers/MediaInfo/FFProbeAudioInfoProvider.cs b/MediaBrowser.Providers/MediaInfo/FFProbeAudioInfoProvider.cs index 4cf7c1eb6..c18210901 100644 --- a/MediaBrowser.Providers/MediaInfo/FFProbeAudioInfoProvider.cs +++ b/MediaBrowser.Providers/MediaInfo/FFProbeAudioInfoProvider.cs @@ -160,15 +160,18 @@ namespace MediaBrowser.Providers.MediaInfo if (!string.IsNullOrEmpty(val)) { + // Sometimes the artist name is listed here, account for that var studios = val.Split(new[] { '/', '|' }, StringSplitOptions.RemoveEmptyEntries) - .Where(i => !string.Equals(i, audio.Artist, StringComparison.OrdinalIgnoreCase) && !string.Equals(i, audio.AlbumArtist, StringComparison.OrdinalIgnoreCase)); + .Where(i => !string.IsNullOrWhiteSpace(i)) + .Where(i => !string.Equals(i, audio.Artist, StringComparison.OrdinalIgnoreCase) && !string.Equals(i, audio.AlbumArtist, StringComparison.OrdinalIgnoreCase)); audio.Studios.Clear(); foreach (var studio in studios) { - audio.AddStudio(studio); + // Account for sloppy tags by trimming + audio.AddStudio(studio.Trim()); } } } @@ -190,7 +193,8 @@ namespace MediaBrowser.Providers.MediaInfo .Split(new[] { '/', '|' }, StringSplitOptions.RemoveEmptyEntries) .Where(i => !string.IsNullOrWhiteSpace(i))) { - audio.AddGenre(genre); + // Account for sloppy tags by trimming + audio.AddGenre(genre.Trim()); } } }