Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#8051 - "year album" tag for tag cache

Attached to Project: Rockbox
Opened by Robert Kukla (roolku) - Tuesday, 30 October 2007, 15:40 GMT+1
Task Type Patches
Category Database
Status New
Assigned To Robert Kukla (roolku)
Player Type All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Private No

Details

This patch addresses the often heard desire to display albums as "year album" in the Database (see for example: http://forums.rockbox.org/index.php?topic=13191.0)

It implements option 3b) from the discussion here:

http://forums.rockbox.org/index.php?topic=13191.msg100433#msg100433

The new tag is called yearalbum and can be used as a filter in the database. It can not be used in the WPS and use in a database conditional clause is discouraged (but possible).

I have edited the tagnavi.config to use the new tag where appropriate.

Warning: a database rebuild is required, so export your runtime data.
   yearalbum.patch (12.1 KiB)
 tagcache.c     |   21 +++++++++++--
 tagcache.h     |    9 +++--
 tagnavi.config |   86 ++++++++++++++++++++++++++++-----------------------------
 tagtree.c      |    1 
 4 files changed, 66 insertions(+), 51 deletions(-)

This task depends upon

Comment by Mark Fawcus (yapper) - Friday, 02 November 2007, 00:55 GMT+1
This works well for me - a very nice feature.
Comment by Travis Tooke (tdtooke) - Sunday, 04 November 2007, 23:45 GMT+1
Sync, sorta.. I just manually edited the patch for that slight change in tagnavi.config and got rid of those pesky trailing CRs.
   yearalbum-20071104.patch (12 KiB)
 tagcache.c     |   21 +++++++++++--
 tagcache.h     |    9 +++--
 tagnavi.config |   86 ++++++++++++++++++++++++++++-----------------------------
 tagtree.c      |    1 
 4 files changed, 66 insertions(+), 51 deletions(-)

Comment by shiloh (gree665) - Monday, 05 November 2007, 06:24 GMT+1
Thanks for the cant-do-without patch, I love it!
Comment by Travis Tooke (tdtooke) - Tuesday, 08 January 2008, 21:17 GMT+1
I went ahead and synced this, hope you don't mind me butting in. I love this patch as do many of the people that use my build.
   yearalbum-20080108.patch (12.1 KiB)
 tagcache.c     |   20 ++++++++++---
 tagcache.h     |    8 +----
 tagnavi.config |   86 ++++++++++++++++++++++++++++-----------------------------
 tagtree.c      |    1 
 4 files changed, 63 insertions(+), 52 deletions(-)

Loading...