From beb6dd6345810ac1eeba38e22fae606e1bff4061 Mon Sep 17 00:00:00 2001 From: Bogdan Date: Sat, 6 Jan 2024 18:41:07 +0200 Subject: [PATCH] Fix possible enumerations in TrackGroupingService --- .../Identification/TrackGroupingService.cs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/NzbDrone.Core/MediaFiles/TrackImport/Identification/TrackGroupingService.cs b/src/NzbDrone.Core/MediaFiles/TrackImport/Identification/TrackGroupingService.cs index 6eabc393c..c434055b0 100644 --- a/src/NzbDrone.Core/MediaFiles/TrackImport/Identification/TrackGroupingService.cs +++ b/src/NzbDrone.Core/MediaFiles/TrackImport/Identification/TrackGroupingService.cs @@ -22,8 +22,8 @@ namespace NzbDrone.Core.MediaFiles.TrackImport.Identification private const string MultiDiscPatternFormat = @"^(?.*%s[\W_]*)\d"; private static readonly Logger _logger = NzbDroneLogger.GetLogger(typeof(TrackGroupingService)); - private static readonly List MultiDiscMarkers = new List { @"dis[ck]", @"cd" }; - private static readonly List VariousArtistTitles = new List { "", "various artists", "various", "va", "unknown" }; + private static readonly List MultiDiscMarkers = new () { @"dis[ck]", @"cd" }; + private static readonly List VariousArtistTitles = new () { "", "various artists", "various", "va", "unknown" }; public List GroupTracks(List localTracks) { @@ -115,8 +115,8 @@ namespace NzbDrone.Core.MediaFiles.TrackImport.Identification const double tagFuzz = 0.9; // check that any Album/Release MBID is unique - if (tracks.Select(x => x.FileTrackInfo.AlbumMBId).Distinct().Where(x => x.IsNotNullOrWhiteSpace()).Count() > 1 || - tracks.Select(x => x.FileTrackInfo.ReleaseMBId).Distinct().Where(x => x.IsNotNullOrWhiteSpace()).Count() > 1) + if (tracks.Select(x => x.FileTrackInfo.AlbumMBId).Distinct().Count(x => x.IsNotNullOrWhiteSpace()) > 1 || + tracks.Select(x => x.FileTrackInfo.ReleaseMBId).Distinct().Count(x => x.IsNotNullOrWhiteSpace()) > 1) { _logger.Trace("LooksLikeSingleRelease: MBIDs are not unique"); return false; @@ -152,7 +152,7 @@ namespace NzbDrone.Core.MediaFiles.TrackImport.Identification const double artistTagThreshold = 0.75; const double tagFuzz = 0.9; - var artistTags = tracks.Select(x => x.FileTrackInfo.ArtistTitle); + var artistTags = tracks.Select(x => x.FileTrackInfo.ArtistTitle).ToList(); if (!HasCommonEntry(artistTags, artistTagThreshold, tagFuzz)) { @@ -178,7 +178,7 @@ namespace NzbDrone.Core.MediaFiles.TrackImport.Identification // and group them. // we only bother doing this for the immediate parent directory. - var trackFolders = tracks.Select(x => Tuple.Create(x, Path.GetDirectoryName(x.Path))); + var trackFolders = tracks.Select(x => Tuple.Create(x, Path.GetDirectoryName(x.Path))).ToList(); var distinctFolders = trackFolders.Select(x => x.Item2).Distinct().ToList(); distinctFolders.Sort();