quodlibet / mutagen

Python module for handling audio metadata
https://mutagen.readthedocs.io
GNU General Public License v2.0
1.49k stars 159 forks source link

ID3v1 tag is hidden by terminating APEv2 #120

Open lazka opened 10 years ago

lazka commented 10 years ago

Originally reported by: Christoph Reiter (Bitbucket: lazka, GitHub: lazka)


From zee...@gmail.com on August 29, 2012 07:10:43

If I add APE tag to a MP3 having ID3v1 presented already, then tags are written in the following order.

1) mp3-data
2) id3v1
3) apetagex
4) apetagex

This ordering breaks many algorithms (including mp3tag.de) which recognize ID3v1 as a 128 bytes block at the end of a file.

Example for the trunk code:

import mutagen
from mutagen.apev2 import APEv2
tags = APEv2()
tags['hashhh'] = 'abcd'
tags.save('a-file-with-id3v1.mp3')

Attachment: short.mp3

Original issue: http://code.google.com/p/mutagen/issues/detail?id=120


lazka commented 10 years ago

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


From reiter.christoph@gmail.com on October 20, 2012 05:20:36

OK, sorry, that's a different problem:

APEv2.save doesn't do any checking and just appends the tag if not loaded from the same file.

Status: Accepted
Mergedinto:

lazka commented 10 years ago

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


From reiter.christoph@gmail.com on October 20, 2012 04:45:25

Status: Duplicate
Mergedinto: 122