1 files changed, 15 insertions, 6 deletions
diff --git a/MediaBrowser.Providers/Music/AlbumInfoExtensions.cs b/MediaBrowser.Providers/Music/AlbumInfoExtensions.cs
index d3fce37c71..2923dd3290 100644
--- a/MediaBrowser.Providers/Music/AlbumInfoExtensions.cs
+++ b/MediaBrowser.Providers/Music/AlbumInfoExtensions.cs
@@ -23,11 +23,11 @@ namespace MediaBrowser.Providers.Music
 
         public static string? GetReleaseGroupId(this AlbumInfo info)
         {
-            var id = info.GetProviderId(MetadataProvider.MusicBrainzReleaseGroup);
+            var id = MusicBrainzId(MetadataProvider.MusicBrainzReleaseGroup, info.GetProviderId(MetadataProvider.MusicBrainzReleaseGroup));
 
             if (string.IsNullOrEmpty(id))
             {
-                return info.SongInfos.Select(i => i.GetProviderId(MetadataProvider.MusicBrainzReleaseGroup))
+                return info.SongInfos.Select(i => MusicBrainzId(MetadataProvider.MusicBrainzReleaseGroup, i.GetProviderId(MetadataProvider.MusicBrainzReleaseGroup)))
                     .FirstOrDefault(i => !string.IsNullOrEmpty(i));
             }
 
@@ -36,11 +36,11 @@ namespace MediaBrowser.Providers.Music
 
         public static string? GetReleaseId(this AlbumInfo info)
         {
-            var id = info.GetProviderId(MetadataProvider.MusicBrainzAlbum);
+            var id = MusicBrainzId(MetadataProvider.MusicBrainzAlbum, info.GetProviderId(MetadataProvider.MusicBrainzAlbum));
 
             if (string.IsNullOrEmpty(id))
             {
-                return info.SongInfos.Select(i => i.GetProviderId(MetadataProvider.MusicBrainzAlbum))
+                return info.SongInfos.Select(i => MusicBrainzId(MetadataProvider.MusicBrainzAlbum, i.GetProviderId(MetadataProvider.MusicBrainzAlbum)))
                     .FirstOrDefault(i => !string.IsNullOrEmpty(i));
             }
 
@@ -50,15 +50,17 @@ namespace MediaBrowser.Providers.Music
         public static string? GetMusicBrainzArtistId(this AlbumInfo info)
         {
             info.ProviderIds.TryGetValue(MetadataProvider.MusicBrainzAlbumArtist.ToString(), out string? id);
+            id = MusicBrainzId(MetadataProvider.MusicBrainzAlbumArtist, id);
 
             if (string.IsNullOrEmpty(id))
             {
                 info.ArtistProviderIds.TryGetValue(MetadataProvider.MusicBrainzArtist.ToString(), out id);
+                id = MusicBrainzId(MetadataProvider.MusicBrainzArtist, id);
             }
 
             if (string.IsNullOrEmpty(id))
             {
-                return info.SongInfos.Select(i => i.GetProviderId(MetadataProvider.MusicBrainzAlbumArtist))
+                return info.SongInfos.Select(i => MusicBrainzId(MetadataProvider.MusicBrainzAlbumArtist, i.GetProviderId(MetadataProvider.MusicBrainzAlbumArtist)))
                     .FirstOrDefault(i => !string.IsNullOrEmpty(i));
             }
 
@@ -68,14 +70,21 @@ namespace MediaBrowser.Providers.Music
         public static string? GetMusicBrainzArtistId(this ArtistInfo info)
         {
             info.ProviderIds.TryGetValue(MetadataProvider.MusicBrainzArtist.ToString(), out var id);
+            id = MusicBrainzId(MetadataProvider.MusicBrainzArtist, id);
 
             if (string.IsNullOrEmpty(id))
             {
-                return info.SongInfos.Select(i => i.GetProviderId(MetadataProvider.MusicBrainzAlbumArtist))
+                return info.SongInfos.Select(i => MusicBrainzId(MetadataProvider.MusicBrainzAlbumArtist, i.GetProviderId(MetadataProvider.MusicBrainzAlbumArtist)))
                     .FirstOrDefault(i => !string.IsNullOrEmpty(i));
             }
 
             return id;
         }
+
+        /// <summary>
+        /// Returns the id if it can be an id of the given provider, otherwise <c>null</c>.
+        /// </summary>
+        private static string? MusicBrainzId(MetadataProvider provider, string? id)
+            => ProviderIdsExtensions.IsValidProviderId(provider.ToString(), id) ? id : null;
     }
 }