[PR #2549] [MERGED] Refactor some provider classes #8973

New issue

Closed

opened 2025-12-22 07:13:56 +01:00 by backuprepo · 0 comments

backuprepo commented

2025-12-22 07:13:56 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/2549
Author: @dkanada
Created: 3/9/2020
Status: ✅ Merged
Merged: 3/18/2020
Merged by: @dkanada

Base: master ← Head: refactor

📝 Commits (4)

52fde64 remove unused files and fix some future warnings
d16f68b move omdb providers
d7c34b4 start tvdb migration for plugin interface
4f195f2 remove useless interface

📊 Changes

55 files changed (+146 additions, -205 deletions)

View changed files

📝 Emby.Server.Implementations/ApplicationHost.cs (+2 -2)
📝 MediaBrowser.Controller/Channels/IChannel.cs (+1 -1)
📝 MediaBrowser.Controller/Entities/BaseItem.cs (+16 -9)
📝 MediaBrowser.Controller/Entities/Book.cs (+4 -0)
📝 MediaBrowser.Controller/Providers/AlbumInfo.cs (+1 -0)
📝 MediaBrowser.Controller/Providers/BoxSetInfo.cs (+0 -1)
📝 MediaBrowser.Controller/Providers/DirectoryService.cs (+0 -3)
➖ MediaBrowser.Controller/Providers/DynamicImageInfo.cs (+0 -10)
📝 MediaBrowser.Controller/Providers/DynamicImageResponse.cs (+4 -0)
📝 MediaBrowser.Controller/Providers/EpisodeInfo.cs (+1 -0)
➖ MediaBrowser.Controller/Providers/ExtraInfo.cs (+0 -15)
➖ MediaBrowser.Controller/Providers/ExtraSource.cs (+0 -9)
📝 MediaBrowser.Controller/Providers/ICustomMetadataProvider.cs (+1 -1)
📝 MediaBrowser.Controller/Providers/IDirectoryService.cs (+3 -0)
➖ MediaBrowser.Controller/Providers/IExtrasProvider.cs (+0 -20)
📝 MediaBrowser.Controller/Providers/IForcedProvider.cs (+1 -1)
📝 MediaBrowser.Controller/Providers/IImageProvider.cs (+3 -3)
➖ MediaBrowser.Controller/Providers/ILocalImageFileProvider.cs (+0 -10)
📝 MediaBrowser.Controller/Providers/ILocalImageProvider.cs (+5 -1)
📝 MediaBrowser.Controller/Providers/ILocalMetadataProvider.cs (+3 -2)

...and 35 more files

📄 Description

I decided to make a refactoring pull request before migrating the video providers to plugins.

_{🔄 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/2549 **Author:** [@dkanada](https://github.com/dkanada) **Created:** 3/9/2020 **Status:** ✅ Merged **Merged:** 3/18/2020 **Merged by:** [@dkanada](https://github.com/dkanada) **Base:** `master` ← **Head:** `refactor` --- ### 📝 Commits (4) - [`52fde64`](https://github.com/jellyfin/jellyfin/commit/52fde64f103b85eb05aabe7c6fb07d7e26db6d48) remove unused files and fix some future warnings - [`d16f68b`](https://github.com/jellyfin/jellyfin/commit/d16f68bb14588ba9869a5a74e8f71dfc4af2856a) move omdb providers - [`d7c34b4`](https://github.com/jellyfin/jellyfin/commit/d7c34b461134d591062416da7d5c8639efb98b9a) start tvdb migration for plugin interface - [`4f195f2`](https://github.com/jellyfin/jellyfin/commit/4f195f289cb4c63d067b50a34d0f63848e7911d5) remove useless interface ### 📊 Changes **55 files changed** (+146 additions, -205 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Server.Implementations/ApplicationHost.cs` (+2 -2) 📝 `MediaBrowser.Controller/Channels/IChannel.cs` (+1 -1) 📝 `MediaBrowser.Controller/Entities/BaseItem.cs` (+16 -9) 📝 `MediaBrowser.Controller/Entities/Book.cs` (+4 -0) 📝 `MediaBrowser.Controller/Providers/AlbumInfo.cs` (+1 -0) 📝 `MediaBrowser.Controller/Providers/BoxSetInfo.cs` (+0 -1) 📝 `MediaBrowser.Controller/Providers/DirectoryService.cs` (+0 -3) ➖ `MediaBrowser.Controller/Providers/DynamicImageInfo.cs` (+0 -10) 📝 `MediaBrowser.Controller/Providers/DynamicImageResponse.cs` (+4 -0) 📝 `MediaBrowser.Controller/Providers/EpisodeInfo.cs` (+1 -0) ➖ `MediaBrowser.Controller/Providers/ExtraInfo.cs` (+0 -15) ➖ `MediaBrowser.Controller/Providers/ExtraSource.cs` (+0 -9) 📝 `MediaBrowser.Controller/Providers/ICustomMetadataProvider.cs` (+1 -1) 📝 `MediaBrowser.Controller/Providers/IDirectoryService.cs` (+3 -0) ➖ `MediaBrowser.Controller/Providers/IExtrasProvider.cs` (+0 -20) 📝 `MediaBrowser.Controller/Providers/IForcedProvider.cs` (+1 -1) 📝 `MediaBrowser.Controller/Providers/IImageProvider.cs` (+3 -3) ➖ `MediaBrowser.Controller/Providers/ILocalImageFileProvider.cs` (+0 -10) 📝 `MediaBrowser.Controller/Providers/ILocalImageProvider.cs` (+5 -1) 📝 `MediaBrowser.Controller/Providers/ILocalMetadataProvider.cs` (+3 -2) _...and 35 more files_ </details> ### 📄 Description I decided to make a refactoring pull request before migrating the video providers to plugins. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

backuprepo

2025-12-22 07:13:56 +01:00

closed this issue
added the
pull-request
label

backuprepo referenced this issue

2025-12-22 08:55:31 +01:00

[PR #8973] [MERGED] Use custom database healthcheck #11750

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: starred/jellyfin#8973

No description provided.

Rows
Columns