tageditor icon indicating copy to clipboard operation
tageditor copied to clipboard

Add support for Songwriter/Web tags in M4A

Open hsn10 opened this issue 1 year ago • 12 comments

I would like to use tageditor as replacement for Studio One 5 which I use for editing ALAC M4A files.

Following fields are not implemented:

  1. Web site
  2. Songwriter

studio-one-5

hsn10 avatar Oct 16 '24 17:10 hsn10

Can you specify what field IDs are used for these fields or point me to a documentation/Wiki page? Maybe the output of tageditor get --show-unsupported -f /path/to/your/file.m4a is also sufficient. Otherwise you could also upload a sample file with those fields.

With this information this is probably implemented in 5 minutes :-)

Martchus avatar Oct 16 '24 17:10 Martchus

what all file formats you need it. looks like songwriter is written only to mp3, not m4a.

hsn10 avatar Oct 16 '24 19:10 hsn10

Well, if you don't have or can't produce a sample M4A file or can otherwise point out the field IDs you want to have support for I cannot help. Of course I could make something up but it wouldn't be useful because probably no other applications would understand it.

I can also add support for more fields in MP3 files (or rather ID3v2 tags) but I would also need to know what field IDs you mean exactly.

Note that this tag editor supports the field "Written by" and many more. Checkout the settings dialog for all supported fields. Not all are supported by all formats, though.

Martchus avatar Oct 16 '24 19:10 Martchus

https://gofile.io/d/bYuCwH

hsn10 avatar Oct 16 '24 19:10 hsn10

Good, I'll look into these in the next days.

Martchus avatar Oct 16 '24 20:10 Martchus

Its free hosting service, it will have unlikely stored files for long.

hsn10 avatar Oct 16 '24 21:10 hsn10

Yes, I've already downloaded them.

Martchus avatar Oct 16 '24 21:10 Martchus

It looks like the only file that has a field with the value "Wrote by ARP (songwriter)" present is the MP3 file. So I assume support for the "songwriter" field is only relevant for MP3 files (or rather ID3v2 tags) for you after all (despite the issue title stating M4A). The good news is that this field is already supported by Tag Editor in basically all formats it generally supports. It is called "Lyricist" in Tag Editor but maps to the same underlying field as in your MP3 example file. That means you could use Tag Editor to add this field also to your M4A, FLAC, Opus and Vorbis files as well using "Lyricist". Whether it can then be display by specific other tools such as Studio One I cannot tell.

It looks like your files that use Vorbis Comments (FLAC, Opus, Vorbis) contain the field "ARRANGER" which is not supported by Tag Editor at this point for Vorbis Comments (only for MP4 and Matroska tags). I can add support for that in Vorbis Comments as well.

It looks like your files that use Vorbis Comments (FLAC, Opus, Vorbis) contain the field "WEBSITE" (with the value https://github.com) which is not supported by Tag Editor at this point. I can add support for that in Vorbis Comments.

It looks like your MP3 file contains the field "Official artist/performer webpage" (with the value https%3A%2F%2Fgithub.com) which is not supported by Tag Editor at this point for ID3v2 tags. I can add support for such fields in MP3 tags.

You also uploaded AIFF and WAV files. These are completely unsupported by Tag Editor so they are out of scope for this ticket. Adding support for those formats would be a valid feature request but I'm not sure whether I'll take the effort of implementing that anytime soon.

Martchus avatar Oct 17 '24 22:10 Martchus

The TYER field in your MP3 file contains an invalid value which Tag Editor complains about correctly. See https://github.com/id3/ID3v2.3/blob/69c4641b049de54c97c9eb0c57b13508008a1124/id3v2.3.0.txt#L854 for allowed values of that field. I suppose it wouldn't hurt if Tag Editor would nevertheless get at least the first valid part of the value.

Martchus avatar Oct 17 '24 22:10 Martchus

AIFF is second most popular DJ format because software reads these tags perfectly. Common DJ software won't read tags from OGG or WAV.

hsn10 avatar Oct 27 '24 07:10 hsn10

But for me this format has zero relevance. So don't expect this to be implemented anytime soon. Of course this is an open source project and I'd of course accept contributions in that regard. (The implementation would mainly be in the tagparser library.)

Martchus avatar Oct 27 '24 09:10 Martchus

C/C++ projects are too difficult for contributing.

Setup to get stuff compiled is difficult and language is annoying with all that manual memory management and stuff. In javascript you just npm install stuff and you can go.

hsn10 avatar Oct 28 '24 19:10 hsn10