[PR #15436] [CLOSED] Fix - Music artists with the same name are merged while MBIDs are ignored #14307

Closed
opened 2026-02-07 07:27:44 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/15436
Author: @MateoTomaszeuski
Created: 11/7/2025
Status: Closed

Base: masterHead: music-artist-fix


📝 Commits (4)

  • b8d15ac Fix Music artists with the same name are merged while MBIDs are ignored #9623
  • 2879aeb Update MediaBrowser.Controller/Entities/Audio/MusicArtist.cs
  • 8efbf6a roll back to this.TryGetProviderId() for succesful build
  • e8a5f18 Merge remote-tracking branch 'upstream/master' into master-old

📊 Changes

2 files changed (+155 additions, -1 deletions)

View changed files

📝 MediaBrowser.Controller/Entities/Audio/MusicArtist.cs (+10 -1)
tests/Jellyfin.Api.Tests/Controllers/ArtistMergingTests.cs (+145 -0)

📄 Description

Changes

Implemented logic to include mbIDs when its included in metadata. If not present, logic stays the same as before.
Also implemented a test suite to make sure everything works as before, and the new logic is correct

Issues
Fixes: #9623


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/15436 **Author:** [@MateoTomaszeuski](https://github.com/MateoTomaszeuski) **Created:** 11/7/2025 **Status:** ❌ Closed **Base:** `master` ← **Head:** `music-artist-fix` --- ### 📝 Commits (4) - [`b8d15ac`](https://github.com/jellyfin/jellyfin/commit/b8d15ac40ffaa8678cd915ce608c1c3f7ff4c367) Fix Music artists with the same name are merged while MBIDs are ignored #9623 - [`2879aeb`](https://github.com/jellyfin/jellyfin/commit/2879aeb6970925e1a5d4d365f7c34edff72c510a) Update MediaBrowser.Controller/Entities/Audio/MusicArtist.cs - [`8efbf6a`](https://github.com/jellyfin/jellyfin/commit/8efbf6adbf38d38ea76cadb09cbcafc82a434971) roll back to this.TryGetProviderId() for succesful build - [`e8a5f18`](https://github.com/jellyfin/jellyfin/commit/e8a5f18add4941eb2aa1114ae4b72649758b99af) Merge remote-tracking branch 'upstream/master' into master-old ### 📊 Changes **2 files changed** (+155 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `MediaBrowser.Controller/Entities/Audio/MusicArtist.cs` (+10 -1) ➕ `tests/Jellyfin.Api.Tests/Controllers/ArtistMergingTests.cs` (+145 -0) </details> ### 📄 Description **Changes** Implemented logic to include mbIDs when its included in metadata. If not present, logic stays the same as before. Also implemented a test suite to make sure everything works as before, and the new logic is correct **Issues** Fixes: #9623 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 07:27:44 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#14307