Opened 14 years ago

Closed 14 years ago

Last modified 12 years ago

#12848 closed (duplicate)

language names should be kept native

Reported by: Benny Daon Owned by: nobody
Component: Internationalization Version: dev
Severity: Keywords:
Cc: Jannis Leidel Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Ramiro Morales)

the LANGUAGES settings array holds tuples of language code and language name. This array is used in templates for language selection. By default, language names are not translated. The docs show a sample of how to translate the names into the current locale:

gettext = lambda s: s

LANGUAGES = (
    ('de', gettext('German')),
    ('en', gettext('English')),
)

Which I think is wrong. If the locale is Chinese, I won't be able to choose between Hebrew and Russian.
The attached patch chages the i18n context process so it translats every language name to its native language. I couldn't think of a way to test it so I added a new view to the example site.


Attachments (2)

native_languages_V1.diff (6.0 KB ) - added by Benny Daon 14 years ago.
native_languages_v2.diff (9.0 KB ) - added by Benny Daon 14 years ago.
A better fix - includes docs and tests

Download all attachments as: .zip

Change History (8)

by Benny Daon, 14 years ago

Attachment: native_languages_V1.diff added

comment:1 by Benny Daon, 14 years ago

seems like all the new translation files came up as /dev/null in the diff file, sorry. I'm using 'hg diff' and trying to find out what I'm doing wrong.

comment:2 by Alex Gaynor, 14 years ago

That's just what trac does, it's very special.

comment:3 by Ramiro Morales, 14 years ago

Description: modified (diff)
Resolution: duplicate
Status: newclosed

Duplicate of #4030, the patch attached there implements a more holistic solution.

by Benny Daon, 14 years ago

Attachment: native_languages_v2.diff added

A better fix - includes docs and tests

comment:4 by Benny Daon, 14 years ago

Cc: Jannis Leidel added
Has patch: set
milestone: 1.21.3
Resolution: duplicate
Status: closedreopened
Version: 1.2-alphaSVN

Following discussions with Jezdez, I reopen the ticket and attach a new patch. The patch includes tests that pass when run alone:

python run_tests.py templates

when the entire suite is ran, the test fails. Haven't a clue.

comment:5 by Łukasz Rekucki, 14 years ago

Resolution: duplicate
Status: reopenedclosed

As already noted, this is a duplicate of #4030. Please comment and submit any patches there.

comment:6 by Jacob, 12 years ago

milestone: 1.3

Milestone 1.3 deleted

Note: See TracTickets for help on using tickets.
Back to Top