Opened 17 years ago
Closed 15 years ago
#6483 closed (fixed)
Support European date format for JavaScript
| Reported by: | Owned by: | Marc Garcia | |
|---|---|---|---|
| Component: | Internationalization | Version: | dev |
| Severity: | Keywords: | i18n-rf | |
| Cc: | marcoberi@… | Triage Stage: | Accepted |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | yes |
| Easy pickings: | no | UI/UX: | no |
Description
Add support for parsing European date format "DD.MM.YYYY" in
the JavaScript date routines.
Attachments (1)
Change History (11)
by , 17 years ago
| Attachment: | 0008-Support-European-date-format-for-JavaScript.patch added |
|---|
comment:1 by , 17 years ago
| Component: | Uncategorized → Admin interface |
|---|
comment:2 by , 16 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:3 by , 16 years ago
| Keywords: | i18n-rf added |
|---|---|
| milestone: | → post-1.0 |
| Owner: | changed from to |
| Patch needs improvement: | set |
comment:4 by , 16 years ago
| Component: | Admin interface → Internationalization |
|---|
comment:5 by , 16 years ago
What approach are you talking about?
This patch only adds a new function to parse the European date format. When or how to use this new function is another question that is not asked or answered by this patch.
comment:6 by , 16 years ago
Maybe a typo error in the Regex expression:
re: /(\d{1,2})\.(\d{1,2}).(\d{4})/ --> re: /(\d{1,2}).(\d{1,2}).(\d{4})/
One extra \ character ?
comment:8 by , 15 years ago
| Cc: | added |
|---|
Note:
See TracTickets
for help on using tickets.
I think that the patch is not the right approach. As I understand DD/MM/YYYY will be used only when american format cannot be used. It'll be difficult for applications users to realize that 12/01/2008 is first of december, but 13/01/2008 is 13th of January.
I propose to use a i18n setting (stored in django(js).po) so parse the date according to current locale. ISO format could be used in all cases as fall back format.