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

New Issue

OVERLORD · 2026-02-07T06:33:41+03:00

OVERLORD commented

2026-02-07 06:33:41 +03:00

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/6862
Author: @1337joe
Created: 11/18/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/18/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>

OVERLORD added the pull-request label 2026-02-07 06:33:41 +03:00

OVERLORD closed this issue

2026-02-07 06:33:43 +03:00

OVERLORD referenced this issue

2026-02-07 07:01:27 +03:00

[PR #11171] Validate item access #12768

OVERLORD referenced this issue

2026-02-07 07:05:50 +03:00

[PR #11876] Don't check if admin has access to library when updating #13023

Sign in to join this conversation.