diff options
| author | Joe Rogers <1337joe@gmail.com> | 2022-05-17 21:03:51 +0200 |
|---|---|---|
| committer | Joe Rogers <1337joe@gmail.com> | 2022-05-17 21:03:51 +0200 |
| commit | 644e7740406bf7bac37a9eb6ca66c024a7b2cc6a (patch) | |
| tree | 2ade12ad3ab59ea0fa5459ec126baa9ff5c8c403 | |
| parent | e89d1a3be5f91f868864db0d0655de0efab98e10 (diff) | |
Fix default audio selection ignoring type
| -rw-r--r-- | Emby.Server.Implementations/Library/MediaStreamSelector.cs | 4 | ||||
| -rw-r--r-- | tests/Jellyfin.Server.Implementations.Tests/Library/MediaStreamSelectorTests.cs | 8 |
2 files changed, 10 insertions, 2 deletions
diff --git a/Emby.Server.Implementations/Library/MediaStreamSelector.cs b/Emby.Server.Implementations/Library/MediaStreamSelector.cs index c5abb9a0a..20a2edb05 100644 --- a/Emby.Server.Implementations/Library/MediaStreamSelector.cs +++ b/Emby.Server.Implementations/Library/MediaStreamSelector.cs @@ -13,11 +13,11 @@ namespace Emby.Server.Implementations.Library { public static int? GetDefaultAudioStreamIndex(IReadOnlyList<MediaStream> streams, IReadOnlyList<string> preferredLanguages, bool preferDefaultTrack) { - var sortedStreams = GetSortedStreams(streams, MediaStreamType.Audio, preferredLanguages); + var sortedStreams = GetSortedStreams(streams, MediaStreamType.Audio, preferredLanguages).ToList(); if (preferDefaultTrack) { - var defaultStream = streams.FirstOrDefault(i => i.IsDefault); + var defaultStream = sortedStreams.FirstOrDefault(i => i.IsDefault); if (defaultStream != null) { diff --git a/tests/Jellyfin.Server.Implementations.Tests/Library/MediaStreamSelectorTests.cs b/tests/Jellyfin.Server.Implementations.Tests/Library/MediaStreamSelectorTests.cs index 2b1c6e937..538010f6c 100644 --- a/tests/Jellyfin.Server.Implementations.Tests/Library/MediaStreamSelectorTests.cs +++ b/tests/Jellyfin.Server.Implementations.Tests/Library/MediaStreamSelectorTests.cs @@ -29,13 +29,21 @@ public class MediaStreamSelectorTests { new() { + Index = 0, + Type = MediaStreamType.Video, + IsDefault = true + }, + new() + { Index = 1, + Type = MediaStreamType.Audio, Language = "fre", IsDefault = true }, new() { Index = 2, + Type = MediaStreamType.Audio, Language = "eng", IsDefault = false } |
