[PR #5980] [MERGED] Fix direct play for DirectPlayProfiles without any codecs set #10761

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

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/5980
Author: @Maxr1998
Created: 5/5/2021
Status: Merged
Merged: 5/5/2021
Merged by: @Bond-009

Base: masterHead: fix-direct-play


📝 Commits (1)

  • 65a9a47 Fix direct play for DirectPlayProfiles without any codecs set

📊 Changes

1 file changed (+2 additions, -1 deletions)

View changed files

📝 MediaBrowser.Model/Dlna/ContainerProfile.cs (+2 -1)

📄 Description

70771fdcd6 broke direct play by treating empty container/codec strings as unsupported in ContainerProfile.ContainsContainer() (which is also used for video and audio codec checks). Instead, they should be treated as supported, for both the positive and negative list option.

Changes

Issues

  • None filed.

🔄 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/5980 **Author:** [@Maxr1998](https://github.com/Maxr1998) **Created:** 5/5/2021 **Status:** ✅ Merged **Merged:** 5/5/2021 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `master` ← **Head:** `fix-direct-play` --- ### 📝 Commits (1) - [`65a9a47`](https://github.com/jellyfin/jellyfin/commit/65a9a4771afc5b6b7c956f7b799586400b23af8b) Fix direct play for DirectPlayProfiles without any codecs set ### 📊 Changes **1 file changed** (+2 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `MediaBrowser.Model/Dlna/ContainerProfile.cs` (+2 -1) </details> ### 📄 Description 70771fdcd60ec5d8a9f13713662778c7e57d0633 broke direct play by treating empty container/codec strings as unsupported in `ContainerProfile.ContainsContainer()` (which is also used for video and audio codec checks). Instead, they should be treated as supported, for both the positive and negative list option. **Changes** - Reverts one of the changes in 70771fdcd60ec5d8a9f13713662778c7e57d0633. **Issues** - None filed. --- <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:26:12 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#10761