Re: Re: ID3 writeback fails (Invalid Language in COMM)

I'll take a look at this tonight. I had a few funky mp3's with empty language codes that I ran into when I was originally implementing this feature. It's quite annoying.

I'll work with your patch tonight and get in touch with the getid3() guy and see if we can come up with a clean solution to this problem. I'll be in touch soon.

Blake

On Mon, 17 Mar 2003 02:50:26 -0500
Matt McClure <[email protected]> wrote:

> Matt McClure wrote:
> > One more bug to report though... after editing track info, the "Refresh"
> > button only reads the first 30 characters of the track name tag.
> > Looking at the MP3 itself, I can see the whole string is present in the
> > tag (40 characters).
> 
> Seems that getid3 isn't overriding the id3v1 tags with the id3v2 tags as
> it claims to do in the changelog:
> 
> >      Comments are now available in the root 2-dimensional array
> >      ['comments'] - each entry in this array will contain one or more
> >      strings. For example, if there are two artists then
> >      ['comments']['artist'][0] will contain the first one and
> >      ['comments']['artist'][1] the other. All keys are forced
> >      lowercase. Comments will be stored in the ['comments'] array in
> >      this order of precedence:
> >      1) Native format tags (ASF, VQF, NSV, RIFF, Quicktime, Vorbis)
> >      2) APE tags
> >      3) ID3v2
> >      4) Lyrics3
> >      5) ID3v1
> >      Lower-priority tags will not overwrite or append existing values
> >      of higher-priority tags (for example, 'artist' in ID3v1 will be
> >      ignored if already specified in APE), but missing values will be
> >      filled in (for example, if 'album' is specified in ID3v2 but not
> >      in APE, it will be included in the ['comments'] array).
> 
> I applied the attached patch, and it works for me.  However, I'm only
> using MP3 with ID3v2 and ID3v1 tags, so YMMV if you're using other file
> types.
> 
> -- 
> Matt
> http://www.faradic.net/~mmcclure/
> 
>     The whole genre of the one-bedroom apartment on the Lower East
>     Side has escaped the [MTV] Cribs radar.
>         -Mike Doughty
> 
>

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)

iD8DBQE+dja9hEyptefxjgkRAhigAJ9umlCkvkT2Cp+WLQCuEeZjqh73ygCfXokn
l0EseIpOTctl7ga/a2PuKmI=
=ICPJ
-----END PGP SIGNATURE-----