mirror of
https://github.com/koel/koel.git
synced 2026-04-26 01:06:00 +03:00
[GH-ISSUE #660] Separate multiple artists #472
Labels
No labels
Authentication
Dependencies
Documentation
Feature Request
Flac
Help Wanted
Installation/Setup
Integration
Mobile
PR Welcome
Pending Release
Performance
Playlist
S3
Search
Sync
[Pri] Low
[Pri] Normal
[Status] Keep Open
[Status] Needs Author Reply
[Status] Needs Review
[Status] Stale
[Status] Will Implement
[Type] Blessed
[Type] Bug
[Type] Duplicate
[Type] Enhancement
[Type] Help Request
[Type] Question
[Type] Task
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/koel-koel#472
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?
Originally created by @duhow on GitHub (Sep 30, 2017).
Original GitHub issue: https://github.com/koel/koel/issues/660
A note on Wiki says:
TCOM, TEXT, TOLY, TOPE, and TPE1 can contain multiple values separated by a foreslash ("/").NOTE: ID3 v2.3 uses
/as multiple artist separator, while v2.4 uses\\.Having this song for example, tagged with MusicBrainz:
When I'm searching for
Virtual Riotartist songs, this one does not appear. Instead, it appears asVirtual Riot feat. Variaartist, which is not just one, but two.The TPE1 field is not separated with foreslash, but it exists a custom field Artists which contains all the artists tagged in this file.
In order to process Artists correctly and finding them, is it posible to priorize the Artists field first if exists, separate all the artists, and add them to the same song?
@X-Ryl669 commented on GitHub (Oct 13, 2017):
It's a huge can of worms you're currently trying to open.
ID3v2.3 says:
Since your song does not follow this scheme, it's not being parsed correctly, so: you should fix your song. That being said, I doubt the PHP's GetID3 library is parsing the "/" anyway as separate entities, so even if the file was good, Koel would not extract more than one artists from this song.
That's exactly related to a topic of another issue I've started, to change a song's model to support multiple artists. So for now, I doubt this could be fixed.