[PR #5870] [MERGED] Fix TMDb Person Provider #10697

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

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/5870
Author: @cvium
Created: 4/20/2021
Status: Merged
Merged: 4/21/2021
Merged by: @Bond-009

Base: masterHead: fix-tmdbpersonprovider


📝 Commits (2)

📊 Changes

2 files changed (+29 additions, -36 deletions)

View changed files

📝 MediaBrowser.Providers/Plugins/Tmdb/People/TmdbPersonImageProvider.cs (+23 -24)
📝 MediaBrowser.Providers/Plugins/Tmdb/People/TmdbPersonProvider.cs (+6 -12)

📄 Description

Changes
Inverted logic and search by name was disabled for some reason (probably because of the old api limit)

Issues
Fixes #4810


🔄 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/5870 **Author:** [@cvium](https://github.com/cvium) **Created:** 4/20/2021 **Status:** ✅ Merged **Merged:** 4/21/2021 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `master` ← **Head:** `fix-tmdbpersonprovider` --- ### 📝 Commits (2) - [`06b8cf4`](https://github.com/jellyfin/jellyfin/commit/06b8cf42d1d1edc424eaccb487590bc6a695ccba) Fix TMDb Person Provider - [`c0ea56a`](https://github.com/jellyfin/jellyfin/commit/c0ea56a10dd085eaa363ec02122a119b07ecc0f8) use int.Parse ### 📊 Changes **2 files changed** (+29 additions, -36 deletions) <details> <summary>View changed files</summary> 📝 `MediaBrowser.Providers/Plugins/Tmdb/People/TmdbPersonImageProvider.cs` (+23 -24) 📝 `MediaBrowser.Providers/Plugins/Tmdb/People/TmdbPersonProvider.cs` (+6 -12) </details> ### 📄 Description <!-- Ensure your title is short, descriptive, and in the imperative mood (Fix X, Change Y, instead of Fixed X, Changed Y). For a good inspiration of what to write in commit messages and PRs please review https://chris.beams.io/posts/git-commit/ and our documentation. --> **Changes** Inverted logic and search by name was disabled for some reason (probably because of the old api limit) **Issues** Fixes #4810 --- <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:25:09 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#10697