mirror of
https://github.com/jellyfin/jellyfin.git
synced 2026-05-04 18:09:12 +03:00
[PR #5559] [MERGED] Clean the entity name for non-words before searching #10557
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 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:
master← Head:fix-tmdb-search-clean📝 Commits (1)
151156fClean 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
ParseNamecould 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.