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

mid3v2 should support POPM tag #71

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

mid3v2 should support POPM tag #71

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

Comments

@lazka
Copy link
Member

@lazka lazka commented Jul 4, 2014

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


From snurgle@gmail.com on September 17, 2010 22:23:54

It would be nice if you could set the POPM tag from the command line (would be useful in shell scripts, etc).  From what I can tell, the mutagen library supports POPM, but there's no syntax from the mid3v2 front end (and unfortunately my Python is not up to snuff).

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


@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 roy.rico on February 17, 2011 00:00:03

So, The support for POPM is there in the mid3v2 script, however, there's an error.   You'll have to edit your python script. 
Mine was in /usr/share/pyshared/mutagen
edit id3.py
Find "Class POPM(Frame)" line, on approximately line 1539

change the following 2 lines

_framespec = [ Latin1TextSpec('email'), IntegerSpec('rating'), IntegerSpec('count') ]
_framespec = [ Latin1TextSpec('email'), IntegerSpec('rating') ]

    def _pprint(self): return "%s=%s %s/255" % (
        self.email, self.rating, self.count )
    def _pprint(self): return "%s, %s" % (
        self.rating, self.email )

I know i edited it a little funky, but this works better for me, until it's fixed for real by the team.
@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 joe.wreschnig@gmail.com on April 17, 2011 05:05:17

This issue was closed by revision r104 .

Status: Fixed

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.