• Status Closed
  • Percent Complete
  • Task Type Patches
  • Category User Interface → Language
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Version 3.2
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by bertrik - 2009-06-07
Last edited by funman - 2009-06-30

FS#10290 - Sansa AMS language fixes

Attached file fixes some language/key issues for the AMS Sansas:
* any references to c200 are changed into c200* so they apply to the c200v2 too
* references to the SELECT button (instead of PLAY) now also apply to fuze* and clip* (LANG_CONFIRM_WITH_BUTTON and LANG_TIME_SET_BUTTON)

Open issues:
* the sansa m200v4 (or m200) is not mentioned at all yet in the language file. Should it be added?

Closed by  funman
2009-06-30 21:15
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

closing since there were no m200 patch attached - committed in r21579

that would be a work for domonoky or another m200v4 owner (m200 support still lacks storage support from what I read, but I didn’t verify)

BTW, these changes should be made to all languages (not just english), tools/ can probably help with that.

rasher just committed the c200 → c200* part of this in svn r21490.

hillshum spoke about LANG_CONFIRM_WITH_BUTTON on irc

after this is changed it would become a m200 only bug i think

Please do these changes for all languages. tools/ can automate this process. Sorry for messing up the work by committing the c200 > c200* change.

rasher, it was already my intention to use langtool, see my comment from Wednesday, 10 June 2009, 13:37 GMT+1
And don’t worry about messing up, it was already messed up by svn r21346 (which *did* do a change to english.lang only).

Please note that the original patch uses clip* instead of just clip (to include the clipv2) and fuze* instead of just fuze (to include the fuzev2)

Yeah, I saw that you’d already posted that, also. Sorry again.

Maybe it’s smarter to attach a series of langtool commandlines here, rather than a patch?

here’s the langtool commands for changing LANG_CONFIRM_WITH_BUTTON and LANG_TIME_SET_BUTTON, and changing all “fuze” and “clip” to “fuze*” and “clip*”

This version uses a better way of replacing “fuze” and “clip” with “fuze*” and “clip*”

Because there’s no need to deal with changing the clip, it’s taken out (thank to rasher for pointing that out).

Okay, here’s the ones I found after reading through english.lang. LANG_DISK_NAME_MMC needs the Fuze added, LANG_TIME_SET_REVERT and LANG_BUILDING_DATABASE may need to be changed as well (depending on what keys are actually uses). LANG_DISK_FULL and LANG_OFF_ABORT are recording-only so any players that can record (currently none) need that.

This version adds the aforementioned lang entries, excepting the recording dependent ones. I’m pretty sure this is all of the entries we need to change

Got a string wrong. Fixed


Available keyboard shortcuts


Task Details

Task Editing