diff options
| author | TheMelmacian <76712303+TheMelmacian@users.noreply.github.com> | 2026-05-15 11:12:08 +0200 |
|---|---|---|
| committer | TheMelmacian <76712303+TheMelmacian@users.noreply.github.com> | 2026-05-15 12:00:00 +0200 |
| commit | fae4950ac2b5918081198ee5f876dd82ca81ae5d (patch) | |
| tree | 8a62b0409b51afc15c1dc09e26e53c3757a043c0 | |
| parent | 068b3fd58d23a25c26b81b9511647c9abbfd798f (diff) | |
Apply suggestions from code review
Co-authored-by: Bond-009 <bond.009@outlook.com>
| -rw-r--r-- | Jellyfin.Api/Controllers/FilterController.cs | 4 | ||||
| -rw-r--r-- | MediaBrowser.Model/Querying/QueryFilters.cs | 9 |
2 files changed, 7 insertions, 6 deletions
diff --git a/Jellyfin.Api/Controllers/FilterController.cs b/Jellyfin.Api/Controllers/FilterController.cs index 740423ef04..cfc8be28ae 100644 --- a/Jellyfin.Api/Controllers/FilterController.cs +++ b/Jellyfin.Api/Controllers/FilterController.cs @@ -197,7 +197,7 @@ public class FilterController : BaseJellyfinApiController var culture = _localization.FindLanguageInfo(language); return new NameValuePair { - Name = culture != null ? $"{culture.DisplayName} ({language})" : language, + Name = culture is null ? language : $"{culture.DisplayName} ({language})", Value = language }; }) @@ -210,7 +210,7 @@ public class FilterController : BaseJellyfinApiController var culture = _localization.FindLanguageInfo(language); return new NameValuePair { - Name = culture != null ? $"{culture.DisplayName} ({language})" : language, + Name = culture is null ? language : $"{culture.DisplayName} ({language})", Value = language }; }) diff --git a/MediaBrowser.Model/Querying/QueryFilters.cs b/MediaBrowser.Model/Querying/QueryFilters.cs index b877af71c6..095f460923 100644 --- a/MediaBrowser.Model/Querying/QueryFilters.cs +++ b/MediaBrowser.Model/Querying/QueryFilters.cs @@ -2,6 +2,7 @@ #pragma warning disable CS1591 using System; +using System.Collections.Generic; using MediaBrowser.Model.Dto; namespace MediaBrowser.Model.Querying @@ -16,12 +17,12 @@ namespace MediaBrowser.Model.Querying SubtitleLanguages = Array.Empty<NameValuePair>(); } - public NameGuidPair[] Genres { get; set; } + public IReadOnlyList<NameGuidPair> Genres { get; set; } - public string[] Tags { get; set; } + public IReadOnlyList<string> Tags { get; set; } - public NameValuePair[] AudioLanguages { get; set; } + public IReadOnlyList<NameValuePair> AudioLanguages { get; set; } - public NameValuePair[] SubtitleLanguages { get; set; } + public IReadOnlyList<NameValuePair> SubtitleLanguages { get; set; } } } |