Opened 16 years ago

Closed 15 years ago

Last modified 13 years ago

#9366 closed Uncategorized (fixed)

date and time filters do not use localized strings, missing datetime filter

Reported by: Bob Thomas Owned by: Marc Garcia
Component: Internationalization Version: 1.0
Severity: Normal Keywords: i18n-fixed
Cc: Triage Stage: Fixed on a branch
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The date and time filters do not use localized/translated strings for the default format. The admin interface uses get_date_formats, which checks for a translation first, then falls back to settings.DATE_FORMAT, etc.

Also, a filter that defaults to DATETIME_FORMAT is missing, and would be a nice shortcut.

Attachments (1)

datefilters.diff (1.6 KB ) - added by Bob Thomas 16 years ago.
Use translated strings for date/time formats, add a datetime filter

Download all attachments as: .zip

Change History (14)

by Bob Thomas, 16 years ago

Attachment: datefilters.diff added

Use translated strings for date/time formats, add a datetime filter

comment:1 by Malcolm Tredinnick, 16 years ago

One item per ticket, please. You're both trying to report a bug and add a new feature here. I suspect the new feature probably won't go in, since you can already format datetimes with the date filter and we should just the right default there (whatever that is).

So please split out the new feature bit from this patch so the bugfix can be considered in isolation (it also will need a test). If you want to open a new ticket for the new feature, that's fine and it can then be looked at in due course.

comment:2 by Bob Thomas, 16 years ago

Component: UncategorizedInternationalization

I think this is somewhat related to #2203 (the bug part of it, anyway). However that ticket is fixed will affect this, since the admin interface's date display should be consistent with the date filters.

comment:3 by Daniel Pope <dan@…>, 16 years ago

@mtreddinick: This isn't really two items. This ticket is about the fact that there's no way of using the existing datetime localisation (http://code.djangoproject.com/browser/django/trunk/django/utils/translation/trans_real.py#L396) in templates.

It makes sense to have the date and time filters do this if the i18n machinery is enabled - otherwise these filters are just about useless in internationalised applications. However, it's not sufficient to just have date and time filters, as the preferred format for combined time and date in different cultures may vary independently from date, time or any combination of the two, so the solution also necessitates a datetime filter. It's a two-part solution, not two different problems.

Anyway, FWIW, this is exactly what I need for the site I'm developing at the moment.

comment:4 by Malcolm Tredinnick, 16 years ago

It is about two items. Item #1 is the first paragraph of the description. Item #2 is the second paragraph. The second one is a feature request and is independent of the first part.

comment:5 by Daniel Pope <dan@…>, 16 years ago

The second part is not independent. Imagine a culture that writes date as dd/mm/yyyy but date and time as dd/mm hh:mm yyyy. Whether any such culture exists I don't know, but an internationalisable datetime formatting system should be able to cope with this.

comment:6 by Malcolm Tredinnick, 16 years ago

You are simply saying that the second feature is necessary. Not that it is entirely and unrevocably part of the first bug fix. It will be quite possible to fix the first part without adding the second part. They are separable. This isn't a question of value.

In any case, this is going around and around. The internationalisation fix will be done as part of a larger ticket. At that point, somebody can update this with a fix to add the missing bit.

comment:7 by Jacob, 16 years ago

Triage Stage: UnreviewedAccepted

comment:9 by anonymous, 16 years ago

Cc: hv@… added

comment:10 by Marc Garcia, 15 years ago

Keywords: i18n-fixed added
Owner: changed from nobody to Marc Garcia

comment:11 by Marc Garcia, 15 years ago

Triage Stage: AcceptedFixed on a branch

comment:12 by Jannis Leidel, 15 years ago

Resolution: fixed
Status: newclosed

(In [11964]) Fixed #7980 - Improved i18n framework to support locale aware formatting (dates and numbers) and form processing.

Thanks to Marc Garcia for working on this during his Google Summer of Code 2009!

Additionally fixes #1061, #2203, #3940, #5526, #6449, #6231, #6693, #6783, #9366 and #10891.

comment:13 by Thomas Güttler, 13 years ago

Cc: hv@… removed
Easy pickings: unset
Severity: Normal
Type: Uncategorized
UI/UX: unset
Note: See TracTickets for help on using tickets.
Back to Top