Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Database
  • Assigned To
    Robert Kukla
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes 2
  • Private
Attached to Project: Rockbox
Opened by Robert Kukla - 2007-10-30
Last edited by Paul Louden - 2009-03-15

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

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.

Closed by  Paul Louden
2009-03-15 03:50
Reason for closing:  Rejected
Additional comments about closing:  

Needs to be reimplemented in a better way to be considered.

Mark Fawcus commented on 2007-11-01 23:55

This works well for me - a very nice feature.

Travis Tooke commented on 2007-11-04 22:45

Sync, sorta.. I just manually edited the patch for that slight change in tagnavi.config and got rid of those pesky trailing CRs.

Shiloh Hawley commented on 2007-11-05 05:24

Thanks for the cant-do-without patch, I love it!

Travis Tooke commented on 2008-01-08 20:17

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.

Shiloh Hawley commented on 2009-01-23 16:23

A few minor changes caused this not to build, here is a synced version

Sadurní commented on 2009-02-21 16:03

I'm using this patch for a while and it works perfectly, with my personal tagnavi_custom.config, and it works very nice.
Why it can't be submitted to svn? It's a shame to have to rebuild the database if I want to try some other patch cleanly or if I must switch to a daily build.
Is there some bug or something that prevents it to be submitted?
Anyway, thanks for this patch.

Christian Engels commented on 2009-03-12 13:19

Just another update for the latest head.

Paul Louden commented on 2009-03-12 13:46

I don't think this patch is a good idea for SVN inclusion. It increases database size in RAM for people who don't use it, and at most saves you a single keypress during navigation over simply having a Year filter before having an Album filter. I don't think its RAM cost is worth it for a single keypress, especially since the sort tags patch could allow you to have your albums in year order anyway.

Thomas Martitz commented on 2009-03-12 15:41

This should be done solely using the tagnavi imo.
Currently, it seems that there's a limitation so that only the track views can be formatted. This limitation should be resolved, so that a fomatted album name like
"%format "fmt_album" "(%4d) %s" year album" works too. This would be a nice implementation of year album, which also adds the possiblity to combine everything with everything.

Robert Kukla commented on 2009-03-12 17:03

This can't be done with the current implementation of the database as it would require the equivalent of an Sql group-by clause. There is some discussion in the forum post (linked to in the FS entry) about it.

Paul Louden commented on 2009-03-12 18:29

The problem with this patch is it passes the cost of adding this feature entirely on to users who don't use it by inflating the database size. Some other solution needs to be found, because otherwise its cost is a good deal larger than what it offers, in a practical sense. People can already organize their filetree this way, and their database can *nearly* be accessed this way (by filtering year, then album, meaning this only really simplifies the display). It's just really not a good "cost vs gain" at all.

Robert Kukla commented on 2009-03-12 19:09

I am not arguing for svn inclusion (I don't use it myself and only did it as a favour to someone at the forums). I am only pointing out that the idea to do it via the tagnavi, as appealing as it might seem to the user, requires some redesigning of how the database works internally and is not just a matter of adding a format string.

Paul Louden commented on 2009-03-12 19:16

Ah, well if this patch isn't a candidate for SVN inclusion, it should be closed as per tracker policy.

Thomas Martitz commented on 2009-03-12 22:04

Hence I said the limitation should be resolved.

Sadurní commented on 2009-03-14 14:04

Thanks Paul for your clarification about svn inclusion for this patch.

Shiloh Hawley commented on 2009-03-14 17:28

I wonder if another tracker could be set up for patches like this one that people use, but are not candidates for SVN (the "ignore the" patch also comes to mind).

Paul Louden commented on 2009-03-15 03:49

You don't need to post the same message more than once.

If you want to start your own unofficial tracker, nobody's stopping you.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing