radec94 / stagger

Automatically exported from code.google.com/p/stagger
BSD 2-Clause "Simplified" License
0 stars 0 forks source link

stagger --print fails with AttributeError #37

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. download attached id3 file
2. run stagger --print dump2.id3
3. File "/home/michael/omg/trunk/stagger/tags.py", line 324, in getter
    (track, sep, total) = frame.text[0].partition("/")
AttributeError: 'list' object has no attribute 'text'

For some reason, the 'TRCK' attribute of the stagger object is a list of
two equal frames. This might be due to a broken/nonstandard tag.
Interestingly, mutagen shows only one TRCK tag, while exfalso (which uses
mutagen) shows twice the same TRCK, as stagger does. So, this tag is
certainly broken, however the exception that occurs is not quite the right
thing to happen here.

Original issue reported on code.google.com by superm...@googlemail.com on 30 Oct 2009 at 11:36

Attachments:

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r78.

Original comment by Karoly.Lorentey on 2 Nov 2009 at 10:13

GoogleCodeExporter commented 9 years ago
Good catch!  Stagger's friendly frame API did not handle duplicate frames well.

Unfortunately the attached dump file was empty, so I couldn't test my solution 
with your original tag.  I was able 
to reproduce the problem with a synthetic tag, but could you please try 
reattaching to help me extend stagger's 
testsuite? Thanks!

Original comment by Karoly.Lorentey on 2 Nov 2009 at 10:19