[PR #6862] [MERGED] Query media streams by type instead of filtering #10992

New issue

Closed

opened 2025-12-22 08:28:09 +01:00 by backuprepo · 0 comments

backuprepo commented

2025-12-22 08:28:09 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/6862
Author: @1337joe
Created: 11/17/2021
Status: ✅ Merged
Merged: 11/20/2021
Merged by: @crobibero

Base: master ← Head: query-instead-of-filtering

📝 Commits (5)

7cf5767 Query media streams by type instead of filtering
97124f5 Add missed override
bff5ff0 Merge similar tests with Theories
9ba7bf9 Query MediaSourceManager directly in image providers
88baff5 Apply suggestions from code review

📊 Changes

6 files changed (+128 additions, -156 deletions)

View changed files

📝 MediaBrowser.Controller/Entities/Audio/Audio.cs (+0 -9)
📝 MediaBrowser.Providers/MediaInfo/AudioImageProvider.cs (+26 -14)
📝 MediaBrowser.Providers/MediaInfo/EmbeddedImageProvider.cs (+12 -4)
📝 MediaBrowser.Providers/MediaInfo/VideoImageProvider.cs (+15 -4)
📝 tests/Jellyfin.Providers.Tests/MediaInfo/EmbeddedImageProviderTests.cs (+20 -20)
📝 tests/Jellyfin.Providers.Tests/MediaInfo/VideoImageProviderTests.cs (+55 -105)

📄 Description

Changes

Query MediaSourceManager directly in image providers
Address ignored warnings/minor cleanup in AudioImageProvider

_{🔄 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/6862 **Author:** [@1337joe](https://github.com/1337joe) **Created:** 11/17/2021 **Status:** ✅ Merged **Merged:** 11/20/2021 **Merged by:** [@crobibero](https://github.com/crobibero) **Base:** `master` ← **Head:** `query-instead-of-filtering` --- ### 📝 Commits (5) - [`7cf5767`](https://github.com/jellyfin/jellyfin/commit/7cf576794940973ef31772b8524ff3c4ff82d09c) Query media streams by type instead of filtering - [`97124f5`](https://github.com/jellyfin/jellyfin/commit/97124f5fcec4b1a1bf1f81bccaaa3282029c40e1) Add missed override - [`bff5ff0`](https://github.com/jellyfin/jellyfin/commit/bff5ff0cb82ed4431031db015acb35df807ecbd7) Merge similar tests with Theories - [`9ba7bf9`](https://github.com/jellyfin/jellyfin/commit/9ba7bf96ef4fd7a8042c594bede42f4f8b7ef329) Query MediaSourceManager directly in image providers - [`88baff5`](https://github.com/jellyfin/jellyfin/commit/88baff5693c6ad749950bbf96eff9f391bcfd194) Apply suggestions from code review ### 📊 Changes **6 files changed** (+128 additions, -156 deletions) <details> <summary>View changed files</summary> 📝 `MediaBrowser.Controller/Entities/Audio/Audio.cs` (+0 -9) 📝 `MediaBrowser.Providers/MediaInfo/AudioImageProvider.cs` (+26 -14) 📝 `MediaBrowser.Providers/MediaInfo/EmbeddedImageProvider.cs` (+12 -4) 📝 `MediaBrowser.Providers/MediaInfo/VideoImageProvider.cs` (+15 -4) 📝 `tests/Jellyfin.Providers.Tests/MediaInfo/EmbeddedImageProviderTests.cs` (+20 -20) 📝 `tests/Jellyfin.Providers.Tests/MediaInfo/VideoImageProviderTests.cs` (+55 -105) </details> ### 📄 Description **Changes** - Query `MediaSourceManager` directly in image providers - Address ignored warnings/minor cleanup in `AudioImageProvider` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

backuprepo

2025-12-22 08:28:10 +01:00

closed this issue
added the
pull-request
label

backuprepo referenced this issue

2025-12-22 09:20:41 +01:00

[PR #10992] [MERGED] Escape subtitle extraction input path #12510

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#10992

No description provided.

Rows
Columns