FS#7362 - Support for GROUPING tag
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.
2007-08-08 10:30
Reason for closing: Fixed
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
Patch committed. Thanks.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
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?)
LinusN already stated in this thread http://forums.rockbox.org/index.php?topic=11173.0 that this patch may be committed. Could some developer please commit it that Rockbox support for classical music is improved?
Thanks a lot.
synced
Synced. Needs testing again though and I don’t really have the setup to do that.
I tested it with the current svn (r14233M-070807) and it works great.
What’s your real name for the credits? Or are you already in the credits file?
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?