LMS-Community / slimserver

Server for Squeezebox and compatible players. This server is also called Lyrion Music Server.
https://lyrion.org
Other
1.19k stars 296 forks source link

Mapping of (more/all) mp3 ID3v2.x standard tags #1156

Open frank1969b opened 2 months ago

frank1969b commented 2 months ago

Today some (most) "Standard" mp3 ID3v2.x tags are mapped by LMS / Scanner, some are missing like TMED = MEDIATYPE TEXT = LYRICIST so "View tabgs" show different field names for mp3-files than on FLAC, OGG etc.

Maybe we should add TMED TEXT to the "transformation" table in https://github.com/LMS-Community/slimserver/blob/a42d7cc589de9e02fb66ac7197e429c6be5a46f5/Slim/Formats/MP3.pm#L69 or even all "standard" tags listed here: https://docs.mp3tag.de/mapping/

Maybe we could also unifiy/standardize the way, "View tags" shows them (today e.g. it shows ALBUM for FLAC/OGG... files but TALB for mp3 files.

Already discussed here: https://forums.slimdevices.com/forum/developer-forums/beta/1724218-user-defined-contributor-roles-are-only-read-from-flac-files

michaelherger commented 2 months ago

"View Tags" is, as the name suggests, to view the tags as they are. This is a raw view by design, no mapping or other applied. It would even show you tags which aren't scanned (IIRC). That won't change.

But as for the tag mapping - I'd be happy to merge a pull request. It's not highly technical, and as you've already got two nice test cases I'm sure you'll figure out how to do it 😉.

frank1969b commented 2 months ago

OK, I tried and have a mp3.pm ready, but don't know how to get it to You - tried with "new pull request" but the system only tells me "there's nothing to compare"...

michaelherger commented 2 months ago

You'll have to fork the repository on Github, add your own fork as a "remote" to your local clone. Then you push the change to your fork, rather than to this repository directly. Once that's done you can go to your Github page and create the pull request.

Or you send me the file and I'll see what I can do 😁.