[PR #4302] [MERGED] Convert field string to enum. #10009

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

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/4302
Author: @crobibero
Created: 10/10/2020
Status: Merged
Merged: 11/11/2020
Merged by: @cvium

Base: masterHead: query-fields


📝 Commits (10+)

  • 69360b7 Convert field string to enum.
  • 4f7c13e Merge remote-tracking branch 'upstream/master' into query-fields
  • 36bae6f Use JsonConverter
  • b9406e5 Merge remote-tracking branch 'upstream/master' into query-fields
  • 372f0eb Remove AddItemFields
  • 11d69fd Make MrLinter happy
  • f8e6edb Merge remote-tracking branch 'upstream/master' into query-fields
  • 6748ba2 Merge remote-tracking branch 'upstream/master' into query-fields
  • 0dd2b16 Add ModelBinder to ItemFilter
  • 7fe8ca2 Merge remote-tracking branch 'upstream/master' into query-fields

📊 Changes

18 files changed (+97 additions, -159 deletions)

View changed files

📝 Jellyfin.Api/Controllers/ArtistsController.cs (+6 -8)
📝 Jellyfin.Api/Controllers/ChannelsController.cs (+6 -8)
📝 Jellyfin.Api/Controllers/GenresController.cs (+3 -4)
📝 Jellyfin.Api/Controllers/InstantMixController.cs (+21 -28)
📝 Jellyfin.Api/Controllers/ItemsController.cs (+4 -6)
📝 Jellyfin.Api/Controllers/LibraryController.cs (+4 -4)
📝 Jellyfin.Api/Controllers/LiveTvController.cs (+16 -21)
📝 Jellyfin.Api/Controllers/MoviesController.cs (+3 -3)
📝 Jellyfin.Api/Controllers/MusicGenresController.cs (+3 -4)
📝 Jellyfin.Api/Controllers/PersonsController.cs (+4 -5)
📝 Jellyfin.Api/Controllers/PlaylistsController.cs (+3 -4)
📝 Jellyfin.Api/Controllers/StudiosController.cs (+3 -4)
📝 Jellyfin.Api/Controllers/TrailersController.cs (+1 -1)
📝 Jellyfin.Api/Controllers/TvShowsController.cs (+10 -14)
📝 Jellyfin.Api/Controllers/UserLibraryController.cs (+3 -4)
📝 Jellyfin.Api/Controllers/YearsController.cs (+3 -4)
📝 Jellyfin.Api/Extensions/DtoExtensions.cs (+0 -36)
📝 Jellyfin.Api/Models/LiveTvDtos/GetProgramsDto.cs (+4 -1)

📄 Description

Part of #4214

Note: GetProgramsDto.Fields cannot be converted


🔄 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/4302 **Author:** [@crobibero](https://github.com/crobibero) **Created:** 10/10/2020 **Status:** ✅ Merged **Merged:** 11/11/2020 **Merged by:** [@cvium](https://github.com/cvium) **Base:** `master` ← **Head:** `query-fields` --- ### 📝 Commits (10+) - [`69360b7`](https://github.com/jellyfin/jellyfin/commit/69360b749a53bd41087530f7fbe2e0c7798f704b) Convert field string to enum. - [`4f7c13e`](https://github.com/jellyfin/jellyfin/commit/4f7c13ecf431004060d0d9eb6cda9bd2593394d0) Merge remote-tracking branch 'upstream/master' into query-fields - [`36bae6f`](https://github.com/jellyfin/jellyfin/commit/36bae6f030f2bcb2879e8463909a806d6b327569) Use JsonConverter - [`b9406e5`](https://github.com/jellyfin/jellyfin/commit/b9406e5444ce8f71ab0ec4e40a9d189a881d232e) Merge remote-tracking branch 'upstream/master' into query-fields - [`372f0eb`](https://github.com/jellyfin/jellyfin/commit/372f0eb38a9c0803d69d5dc3c4f0fdedd948cd0c) Remove AddItemFields - [`11d69fd`](https://github.com/jellyfin/jellyfin/commit/11d69fd3b1c25927c1046c4a095a3bd9f29fa722) Make MrLinter happy - [`f8e6edb`](https://github.com/jellyfin/jellyfin/commit/f8e6edbfeb8c437c2ed0a9c9b44bed0fc105b2c2) Merge remote-tracking branch 'upstream/master' into query-fields - [`6748ba2`](https://github.com/jellyfin/jellyfin/commit/6748ba287d8cd9054072ce1ffc51c7aa8690e108) Merge remote-tracking branch 'upstream/master' into query-fields - [`0dd2b16`](https://github.com/jellyfin/jellyfin/commit/0dd2b169a3d9b0c1b31cd83b4022c729e28f0d3b) Add ModelBinder to ItemFilter - [`7fe8ca2`](https://github.com/jellyfin/jellyfin/commit/7fe8ca2b775b2f89e84706fa32dedd7a5db65f77) Merge remote-tracking branch 'upstream/master' into query-fields ### 📊 Changes **18 files changed** (+97 additions, -159 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Api/Controllers/ArtistsController.cs` (+6 -8) 📝 `Jellyfin.Api/Controllers/ChannelsController.cs` (+6 -8) 📝 `Jellyfin.Api/Controllers/GenresController.cs` (+3 -4) 📝 `Jellyfin.Api/Controllers/InstantMixController.cs` (+21 -28) 📝 `Jellyfin.Api/Controllers/ItemsController.cs` (+4 -6) 📝 `Jellyfin.Api/Controllers/LibraryController.cs` (+4 -4) 📝 `Jellyfin.Api/Controllers/LiveTvController.cs` (+16 -21) 📝 `Jellyfin.Api/Controllers/MoviesController.cs` (+3 -3) 📝 `Jellyfin.Api/Controllers/MusicGenresController.cs` (+3 -4) 📝 `Jellyfin.Api/Controllers/PersonsController.cs` (+4 -5) 📝 `Jellyfin.Api/Controllers/PlaylistsController.cs` (+3 -4) 📝 `Jellyfin.Api/Controllers/StudiosController.cs` (+3 -4) 📝 `Jellyfin.Api/Controllers/TrailersController.cs` (+1 -1) 📝 `Jellyfin.Api/Controllers/TvShowsController.cs` (+10 -14) 📝 `Jellyfin.Api/Controllers/UserLibraryController.cs` (+3 -4) 📝 `Jellyfin.Api/Controllers/YearsController.cs` (+3 -4) 📝 `Jellyfin.Api/Extensions/DtoExtensions.cs` (+0 -36) 📝 `Jellyfin.Api/Models/LiveTvDtos/GetProgramsDto.cs` (+4 -1) </details> ### 📄 Description Part of #4214 ~~Note: `GetProgramsDto.Fields` cannot be converted~~ --- <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:12:29 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#10009