Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

APETAG written instead of APETAGEX sometimes #122

Closed
lazka opened this issue Jul 4, 2014 · 3 comments
Closed

APETAG written instead of APETAGEX sometimes #122

lazka opened this issue Jul 4, 2014 · 3 comments
Labels

Comments

@lazka
Copy link
Member

@lazka lazka commented Jul 4, 2014

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


From zee...@gmail.com on August 29, 2012 23:06:28

Let there is a MP3 file with the following structure:
- ID3v2
- FRAMES
- ID3v1
- APETAGEX

It is (IMHO) correctly recognized by mp3tag as having APEv2 and ID3v2 tags set.

After the following code is completed, the resulting out.mp3 has the same structure but having marker 'APETAG' instead of correct 'APETAGEX' (AFAIK). Such ape tag is not recognized not by mp3tag, nor by foobar2000.

from mutagen.easyid3 import EasyID3
audio = EasyID3('in.mp3')
audio.save('out.mp3')

Reproduction code attached.

Python 2.7, Windows 7, trunk

Attachment: reproduce.zip

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


@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Jul 4, 2014

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


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

AFAIK ID3v1 should come last after apev2 tags.

Problem: mutagen recognizes the "TAG" in "APETAGEX" as id3v1 start and updates it.

Status: Accepted

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Jul 4, 2014

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


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

Issue 120 has been merged into this issue.
@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Sep 25, 2014

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


id3: try to detect apev2 tags when looking for id3v1 tags. (Fixes issue #122)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.