#14924: I18N looks for translations in the reverse order of the apps
------------------------------------+---------------------------------------
Reporter: vanschelven | Owner: nobody
Status: new | Milestone:
Component: Uncategorized | Version: 1.2
Resolution: | Keywords:
Stage: Accepted | Has_patch: 0
Needs_docs: 0 | Needs_tests: 0
Needs_better_patch: 0 |
------------------------------------+---------------------------------------
Comment (by vanschelven):
I'd like to have some discussion on my own comment below:
> Having looked at the original tests the most desirable order appears to
be:
>
> 1. Project path locale
> 2. Apps (in correct order)
> 3. Locale-path
> 4. Django's own translations
I've got the above patch running in some production code now, and I'm
feeling this is not the most logical order. (The order was originally
established by looking at the existing tests and code, and matching that
order as closely as possible while making sure the apps are evaluated in
the correct, not reverse, order).
In my mind, locale path should be the first to be evaluated. The reason
someone would put up a specific path for locales pretty much has to be to
override the existing behavior (which would be found in project and apps).
Input (discussion) on this is much appreciated. I'll gladly put in a new
patch once a decision is reached. If this is a separate issue, I'm also
fine on opening a separate ticket. (it could be argued that we can move
forward on the original bug report as a separate issue more easily, since
it's "obviously broken").
--
Ticket URL: <http://code.djangoproject.com/ticket/14924#comment:2>
Django <http://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to django-upda...@googlegroups.com.
To unsubscribe from this group, send email to
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en.
