[PR #12279] [MERGED] Move external url listing to provider for plugin use #13131

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

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/12279
Author: @crobibero
Created: 7/15/2024
Status: Merged
Merged: 7/17/2024
Merged by: @crobibero

Base: masterHead: related-url-provider


📝 Commits (2)

  • b8c5c01 Move external url listing to provider for plugin use
  • 117f653 mark UrlFormatString as obsolete

📊 Changes

13 files changed (+73 additions, -124 deletions)

View changed files

📝 Emby.Server.Implementations/ApplicationHost.cs (+2 -1)
📝 MediaBrowser.Controller/Entities/BaseItem.cs (+0 -5)
📝 MediaBrowser.Controller/Entities/Movies/Movie.cs (+0 -18)
📝 MediaBrowser.Controller/Entities/TV/Episode.cs (+0 -17)
📝 MediaBrowser.Controller/Entities/TV/Series.cs (+0 -17)
📝 MediaBrowser.Controller/Entities/Trailer.cs (+0 -17)
📝 MediaBrowser.Controller/LiveTv/LiveTvProgram.cs (+0 -20)
📝 MediaBrowser.Controller/Providers/IExternalId.cs (+2 -0)
MediaBrowser.Controller/Providers/IExternalUrlProvider.cs (+22 -0)
📝 MediaBrowser.Controller/Providers/IProviderManager.cs (+3 -1)
📝 MediaBrowser.Model/Providers/ExternalIdInfo.cs (+5 -0)
📝 MediaBrowser.Providers/Manager/ProviderManager.cs (+35 -26)
📝 tests/Jellyfin.Providers.Tests/Manager/ProviderManagerTests.cs (+4 -2)

📄 Description

Changes

Moves the GetRelatedUrls() out from BaseItem to a dedicated provider so plugins can add dynamic urls.

See linked plugin PRs for sample implementations.


🔄 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/12279 **Author:** [@crobibero](https://github.com/crobibero) **Created:** 7/15/2024 **Status:** ✅ Merged **Merged:** 7/17/2024 **Merged by:** [@crobibero](https://github.com/crobibero) **Base:** `master` ← **Head:** `related-url-provider` --- ### 📝 Commits (2) - [`b8c5c01`](https://github.com/jellyfin/jellyfin/commit/b8c5c01b41d5e5eb48587f47825470e94162b450) Move external url listing to provider for plugin use - [`117f653`](https://github.com/jellyfin/jellyfin/commit/117f653b8871347c68f589a22b2623aee393883c) mark UrlFormatString as obsolete ### 📊 Changes **13 files changed** (+73 additions, -124 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Server.Implementations/ApplicationHost.cs` (+2 -1) 📝 `MediaBrowser.Controller/Entities/BaseItem.cs` (+0 -5) 📝 `MediaBrowser.Controller/Entities/Movies/Movie.cs` (+0 -18) 📝 `MediaBrowser.Controller/Entities/TV/Episode.cs` (+0 -17) 📝 `MediaBrowser.Controller/Entities/TV/Series.cs` (+0 -17) 📝 `MediaBrowser.Controller/Entities/Trailer.cs` (+0 -17) 📝 `MediaBrowser.Controller/LiveTv/LiveTvProgram.cs` (+0 -20) 📝 `MediaBrowser.Controller/Providers/IExternalId.cs` (+2 -0) ➕ `MediaBrowser.Controller/Providers/IExternalUrlProvider.cs` (+22 -0) 📝 `MediaBrowser.Controller/Providers/IProviderManager.cs` (+3 -1) 📝 `MediaBrowser.Model/Providers/ExternalIdInfo.cs` (+5 -0) 📝 `MediaBrowser.Providers/Manager/ProviderManager.cs` (+35 -26) 📝 `tests/Jellyfin.Providers.Tests/Manager/ProviderManagerTests.cs` (+4 -2) </details> ### 📄 Description **Changes** <!-- Describe your changes here in 1-5 sentences. --> Moves the `GetRelatedUrls()` out from `BaseItem` to a dedicated provider so plugins can add dynamic urls. See linked plugin PRs for sample implementations. --- <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:07:41 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#13131