mirror of
https://github.com/jellyfin/jellyfin.git
synced 2026-01-23 23:20:51 +01:00
[PR #10062] [MERGED] Fix the probing of m4a metadata with missing composer #12145
Labels
No labels
area:database
awaiting-feedback
backend
blocked
breaking change: web api
bug
build
ci
confirmed
discussion needed
dotnet future
downstream
duplicate
EFjellyfin.db
enhancement
feature
future
github-actions
good first issue
hdr
help wanted
invalid
investigation
librarydb
live-tv
lyrics
media playback
music
needs testing
nuget
performance
platform
pull-request
question
regression
release critical
requires-web
roadmap
security
security
stale
support
syncplay
ui & ux
upstream
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: starred/jellyfin#12145
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
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/10062
Author: @lonebyte
Created: 7/31/2023
Status: ✅ Merged
Merged: 8/1/2023
Merged by: @Bond-009
Base:
master← Head:m4a-metadata-probing-fix📝 Commits (1)
66ff724Fix the probing of m4a metadata📊 Changes
1 file changed (+21 additions, -12 deletions)
View changed files
📝
MediaBrowser.Providers/MediaInfo/AudioFileProber.cs(+21 -12)📄 Description
Changes
The composer is not set in some of my m4a files. For some reason TagLibSharp returns the composer as an empty string in this case. This causes an exception in PeopleHelper.AddPerson, and thus probing fails.
IMHO we can simply ignore empty values.
Issues
Fixes: #10061
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.