[PR #5559] [MERGED] Clean the entity name for non-words before searching #10557

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

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/5559
Author: @cvium
Created: 3/17/2021
Status: Merged
Merged: 3/19/2021
Merged by: @cvium

Base: masterHead: fix-tmdb-search-clean


📝 Commits (1)

  • 151156f Clean the entity name for non-words before searching

📊 Changes

4 files changed (+27 additions, -4 deletions)

View changed files

📝 MediaBrowser.Providers/Plugins/Tmdb/BoxSets/TmdbBoxSetProvider.cs (+9 -2)
📝 MediaBrowser.Providers/Plugins/Tmdb/Movies/TmdbMovieProvider.cs (+2 -1)
📝 MediaBrowser.Providers/Plugins/Tmdb/TV/TmdbSeriesProvider.cs (+2 -1)
📝 MediaBrowser.Providers/Plugins/Tmdb/TmdbUtils.cs (+14 -0)

📄 Description

Changes
Added non-word cleaning. It used to do a lot more though 907695dec7/MediaBrowser.Providers/Plugins/Tmdb/Movies/TmdbSearch.cs (L26-L37), but I it doesn't really belong in tmdb at all, so I'd rather we try without it for now.

I think ParseName could use some more work though, but I'm afraid to change it in 10.7 since it's used elsewhere.

Issues
Fixes #5224


🔄 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/5559 **Author:** [@cvium](https://github.com/cvium) **Created:** 3/17/2021 **Status:** ✅ Merged **Merged:** 3/19/2021 **Merged by:** [@cvium](https://github.com/cvium) **Base:** `master` ← **Head:** `fix-tmdb-search-clean` --- ### 📝 Commits (1) - [`151156f`](https://github.com/jellyfin/jellyfin/commit/151156f2271104277a575054aea9e5b0f92d6591) Clean the entity name for non-words before searching ### 📊 Changes **4 files changed** (+27 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `MediaBrowser.Providers/Plugins/Tmdb/BoxSets/TmdbBoxSetProvider.cs` (+9 -2) 📝 `MediaBrowser.Providers/Plugins/Tmdb/Movies/TmdbMovieProvider.cs` (+2 -1) 📝 `MediaBrowser.Providers/Plugins/Tmdb/TV/TmdbSeriesProvider.cs` (+2 -1) 📝 `MediaBrowser.Providers/Plugins/Tmdb/TmdbUtils.cs` (+14 -0) </details> ### 📄 Description **Changes** Added non-word cleaning. It used to do a lot more though https://github.com/jellyfin/jellyfin/blob/907695dec7fda152d0e17c1197637bc0e17c9928/MediaBrowser.Providers/Plugins/Tmdb/Movies/TmdbSearch.cs#L26-L37, but I it doesn't really belong in tmdb at all, so I'd rather we try without it for now. I think `ParseName` could use some more work though, but I'm afraid to change it in 10.7 since it's used elsewhere. **Issues** Fixes #5224 --- <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:22:28 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#10557