• Status Closed
  • Percent Complete
  • Task Type Patches
  • Category Playlists
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by rasher - 2006-11-08
Last edited by bluebrother - 2006-11-08

FS#6306 - Rename Tagcache to Database

As discussed on IRC, this patch renames tag cache to simply Database. Includes changes to manual and english.lang as well as changing the names of the files tagcache writes to disk. Pretty much untested, so give it a whirl.

Closed by  rasher
2006-11-25 09:53
Reason for closing:  Accepted
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

Committed to cvs

I think the attached patch qualifies this as a patch, not a bug ;-) Feel free to change it back if you disagree.

I think it would be better to rename it to “Tag Database” instead of simply “Database” – this doesn’t give a hint what is used to build the database, users might expect it to contain everything, including e.g. images, wave files etc. Additionally, I don’t think renaming the tagcache files itself is necessary – we can always argue that this is still the internal name when pointing a user to remove the tagcache files (and, in most cases this will only be “remove all *.tcd files, so the basename itself doesn’t matter that much)

Thanks for fixing the wrong fields, I simply forgot to change them when submitting.

If we were to call it more than just database, I’d think “Music Database” was better. The reason why I moved the files were because they were mentioned in the manual (tagcache_changelog), but I agree it isn’t strictly necessary. But on the other hand, there are no real ill effects other than people having to rebuild.

Project Manager

I think “database” is the best, nice and simple.

nls commented on 2006-11-08 20:00

I vote for simply ‘database’ too.

rasher, I think you missed one here:
- \\item[ID3 Database:] This shows the files based on the \\setting{Tag Cache}

+ \\item[ID3 Database:] This shows the files based on the \\setting{Database}

Should probably be: (as the string changed in the .lang file)
(maybe it could be called something like “Database view”?)
- \\item[ID3 Database:] This shows the files based on the \\setting{Tag Cache}

+ \\item[Database:] This shows the files based on the \\setting{Database}

Also the change to (the last part of the patch) was not supposed to
be there, right?
-prefix=”/usr/local” +prefix=”/home/rasher/rockbox/build-env/”

nls: Right, right and right. Fixed.

I also fixed some occurrences of “The Database database” (caused by recklessly replacing “Tag Cache” with “Database”) and stopped treating database like a proper noun in most places (except when referring to the setting).

The only user-visible left is the configfile options, which are still called tagcache_*, but I’ll leave those.


Available keyboard shortcuts


Task Details

Task Editing