Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category ID3 / meta data
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes 1
  • Private
Attached to Project: Rockbox
Opened by Charles Voelger - 2006-03-30
Last edited by Dan Everton - 2007-08-03

FS#4961 - Adds Disc Number ID Tag for Tagcache (Feature Request FS#4957)

This adds disc numbers to tag cache.

Closed by  Dan Everton
2007-08-03 10:13
Reason for closing:  Accepted
Additional comments about closing:  

Patch committed to SVN. Thanks.

Marc Cramdal commented on 2006-04-01 19:13

I think this patch would be perfect if you added the wps entry for printing the disknumber. I think %ik would be nice (k like disK)

Thanks a lot for your patch !

Charles Voelger commented on 2006-04-04 19:44

here is the patch with a %ik wps tag.

Ruben Petersen commented on 2006-04-25 13:50

Should this possibly be merged with http://www.rockbox.org/tracker/task/2974 ?

Charles Voelger commented on 2006-04-25 19:12

well as they are two different features, I would say no. Although I wonder if this every works with current cvs, I will have to look at it when I can.

Ruben Petersen commented on 2006-04-26 05:09

The discnumber tag is just a tag. So are disc title, album artist and so on. One could even just add generic support for other tags, no? %ic_’taggoeshere’ or so.

Charles Voelger commented on 2006-04-26 16:06

Well from what I saw while adding the discnumber stuff, all the tags have to have individual support in code. There is not a method where it reads them all and they can be dynamically used. Each one needs to be added to the code.

Markus Berndt commented on 2006-08-16 19:38

I would find this patch extremely useful, since I have a lot of sets of discs. However, this patch does not work anymore with the latest CVS version. Is there a newer version of the patch that is compatible with what’s in CVS? If not, I will make one an post it.

Phillip Michael commented on 2006-10-20 17:22

I would also like this patch if it were compatible with the cuurent source code.

danhibiki commented on 2006-11-24 16:36

I’ve made a patch based on this and it seems to work well here... If anyone is interested in trying it, feel free to do
tested on my 4GB Nano and it worked fine, but I haven’t tested on anything else (besides the simulator) and I don’t guarantee it will work or that it will not cause any
default behaviour is to show the tracknumber with discnumber in front of it, but when there is no discnumber tag (or when it is a string instead of a number), it shows one more 0... But at least it is now correctly sorted
worked correctly with Vorbis, MP3 and MPC. I haven’t tested anything else, but I think it will work with some other format
if you want to change the syntax of TagCache config file, you can use “discnum” to deal with it.

Markus Berndt commented on 2006-11-29 05:37

I just tried danhibiki’s patch and it works well on my iRiver H120. Thanks a bunch!

Chris commented on 2007-01-28 01:33

Working(?) with 5.5G 30GB Video iPod - displays zero hmmm... maybe the logic should be changed to something like this:

Lets assume that the disc number tag is AB, where A is the first digit and B is the second digit

1. Read Disc number from ID3 tag, store in variable
If X is greater than 9, display disc number as
If X is smaller than 10, display disc number as B

Andy Hawkins commented on 2007-02-14 10:35

I might be missing something here, but reading through the patch doesn’t appear to include anything for sorting the playlist based on the disc number.

Is this automatic, or has this been missed?

Thanks

Andy

danhibiki commented on 2007-02-14 18:15

Ok, just updated it to the current SVN.

Andy, you probably missed the change done in tagnavi.config, there you can see that it’s now sorted by disc number first. The change in other files are to make rockbox understand the disc number tag.

I’m having trouble to compile the simulator with this patch, some problem with speex it seems (probably I will have the same error without the patch...), but it compiled fine for iPod Nano.

danhibiki commented on 2007-02-14 19:02

If you run into trouble compiling the simulator, you just need to re-run ../tools/configure, it did the trick for me.

danhibiki commented on 2007-02-15 04:05

It seems that the patch is broken again, please don’t use it, I’ll take a look at it soon.

Andy Hawkins commented on 2007-02-15 10:18

Can’t come soon enough for me danhibiki :)

On an unrelated note, what is required for this patch to be accepted into the ‘real’ code (and hence provided in the daily builds)?

Alan F commented on 2007-02-15 15:13

I don’t know if I’m talking about the wrong thing here but you can get disc numbers as a filter in tag navigation. The details are in the below rockbox forum topic.

http://forums.rockbox.org/index.php?topic=8723.0

Alan F commented on 2007-02-15 15:15

as per previous comment, the comment tag can be put in the wps to display disc number.

Andy Hawkins commented on 2007-02-15 16:07

Yes, but there’s already a disk number tag. And just displaying it in the WPS screen isn’t enough it it? It needs to be used to sort items in a playlist.

danhibiki commented on 2007-02-23 19:18

Ok, it took a while because I was without access to a PC for a few days, but now it seems to be working fine with the latest SVN.

Andy, I have no idea about what it is required to this patch to be accepted, but I may join #rockbox someday and ask some developer with access to SVN. But for now you need to build your own Rockbox.

danhibiki commented on 2007-02-28 02:20

Updated to the current SVN.

David Gentzel commented on 2007-04-05 04:23

Commit of WPS tokenizer obviously broke this. Here’s an update to the current SVN.

David Gentzel commented on 2007-04-09 05:16

WPS code is still changing fairly rapidly. Another update.

Christoph Reiter commented on 2007-07-02 15:52

synced to current SVN.

Andy Hawkins commented on 2007-07-02 16:12

What chance of this getting added in to SVN? It’s been around for well over a year now, and it’s a major pain in the current operation.

Also, does it support getting disc numbers from VORBIS comments?

Cheers

Andy

David Gentzel commented on 2007-07-02 17:06

I’m certainly in favor of it getting into SVN. I doubt there’s anything unpleasant enough about the code keeping it out, so the reason is probably either lack of general interest (or lack of someone lobbying hard enough) or the fact that it’s presently ID3 only.

Christoph Reiter commented on 2007-07-02 17:42

same as above with tagviewer integration. (the “Show ID3 Info” thing)

Dan Everton commented on 2007-07-07 08:18

In rockbox/apps/gui/wps_parser.c part of the patch, what’s the WPS_TOKEN_METADATA_TRACK_TITLE token about?

danhibiki commented on 2007-07-07 18:33

I think it’s a mistake, if you take a look at the first patch from David Gentzel it uses the correct token, but in the second one there is this weird WPS_TOKEN_METADATA_TRACK_TITLE.

I created a new patch with WPS_TOKEN_METADATA_DISC instead of WPS_TOKEN_METADATA_TRACK_TITLE.

Christoph Reiter commented on 2007-07-08 17:15

@David Gentzel: It’s not ID3 only. The only not-MP3 musicfiles i have are Vorbis and they work too.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing