FS#7362 - Support for GROUPING tag

Attached to Project: Rockbox
Opened by Dieter (dip) - Tuesday, 26 June 2007, 23:13 GMT
Last edited by Dan Everton (safetydan) - Wednesday, 08 August 2007, 10:30 GMT
Task Type Patches
Category ID3 / meta data
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No


This patch adds support for the GROUPING tag which is usually used for grouping titles of multi part works (e.g. classical works like symphonies or sonatas). It supports id3v2.2 (TT1), id3v2.3 and id3v2.4 (TIT1), mp4 (cgrp) and vorbis comments (GROUPING as used by MusicBrainz, CONTENTGROUP as used by MP3Tag and CONTENT GROUP as used by foobar2000). In WPS you can use %iG and %IG. As tagnavi.conf tags "grouping" and "contentgroup" are supported but not "content group" due to the space in the middle.

In addition, when building the database and a track doesn't have a grouping tag, it copies the value from the title tag. This should make browsing through classcical works in the database a bit nicer since you will find all your music, not only titles having the grouping tag explicitely set (similar to the recent albumartist improvement). You need to rebuild your database to actually see any change.
This task depends upon

Closed by  Dan Everton (safetydan)
Wednesday, 08 August 2007, 10:30 GMT
Reason for closing:  Fixed
Additional comments about closing:  Patch committed. Thanks.
Comment by Dieter (dip) - Tuesday, 26 June 2007, 23:25 GMT
I didn't mean WPS-tags when mentioning "grouping" and "contentgroup" but the tags for the tagnavi.conf file. (Isn't there a way to edit my own post?)
Comment by Dieter (dip) - Monday, 23 July 2007, 22:00 GMT
LinusN already stated in this thread that this patch may be committed. Could some developer please commit it that Rockbox support for classical music is improved?
Thanks a lot.
Comment by Dieter (dip) - Tuesday, 31 July 2007, 20:34 GMT
Comment by Dan Everton (safetydan) - Tuesday, 07 August 2007, 11:23 GMT
Synced. Needs testing again though and I don't really have the setup to do that.
Comment by Dieter (dip) - Tuesday, 07 August 2007, 20:51 GMT
I tested it with the current svn (r14233M-070807) and it works great.
Comment by Dan Everton (safetydan) - Wednesday, 08 August 2007, 10:06 GMT
What's your real name for the credits? Or are you already in the credits file?
Comment by Dieter (dip) - Wednesday, 08 August 2007, 10:16 GMT
My real name is Dieter Pellkofer. I have no idea if I am already in the credits file. The only patch I created and which was committed so long was the album artist and comment patch (which was based on a patch from Robert Kukla). Should I amend my profile to include my complete name, if yes, how can I do that?