#7656 closed (fixed)
Change superclass of AdminDateWidget from TextInput to DateTimeInput
| Reported by: | jamesbeard | Owned by: | Jannis Leidel |
|---|---|---|---|
| Component: | contrib.admin | Version: | dev |
| Severity: | Keywords: | date widget text format i18n-rf | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | yes |
| Needs tests: | yes | Patch needs improvement: | yes |
| Easy pickings: | no | UI/UX: | no |
Description
Can we change the superclass of the newforms-admin contrib.admin.widgets.AdminDateWidget from TextInput to DateTimeInput?
The TextInput doesn't currently take any notice of newforms.fields.DEFAULT_DATE_INPUT_FORMATS. Changing it would cause existing date fields to render properly without having to patch Django locally or override half the admin framework. Unless there is another way that I've missed?
There is a related issue with the Javascript date helpers, but these can be easily overridden in an extraheader block.
Refer to Django developers post at http://tinyurl.com/5w3lmf for some more details.
Attachments (3)
Change History (17)
comment:1 by , 17 years ago
| Triage Stage: | Unreviewed → Design decision needed |
|---|
comment:2 by , 17 years ago
| Keywords: | i18n-rf nfa-blocker added |
|---|---|
| milestone: | → 1.0 alpha |
by , 17 years ago
| Attachment: | admin_widgets.diff added |
|---|
Updates admin widgets to subclass DateTimeInput
comment:3 by , 17 years ago
| milestone: | 1.0 alpha → post-1.0 |
|---|
This requires #7509 so it probably won't go in for a while.
comment:4 by , 17 years ago
| Keywords: | nfa-blocker removed |
|---|
comment:5 by , 17 years ago
What's the backwards incompatibility impact of this change? Ignore what it depends upon for now, since fixing a bunch of i18n date problems is quite possible before 1.0. If people's admin sites are going to become backwards incompatible because of this change, it's a pre-1.0 or never situation.
comment:6 by , 17 years ago
With the patch I created all current tests pass fine(one was failing for Brian and we haven't been able to establish why yet).
comment:7 by , 17 years ago
| Version: | newforms-admin → SVN |
|---|
by , 17 years ago
| Attachment: | patch_django_7656.20090207.diff added |
|---|
Update the patch against r9814 + a couple of fixes
comment:8 by , 17 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:9 by , 17 years ago
| Has patch: | set |
|---|---|
| Needs documentation: | set |
| Needs tests: | set |
by , 17 years ago
| Attachment: | patch_django_7656.20090209.diff added |
|---|
Patch against r9820, AdminTimeWidget inherit from TimeInput and not DateTimeInput
comment:11 by , 16 years ago
| Cc: | added |
|---|
comment:12 by , 16 years ago
| Cc: | removed |
|---|---|
| milestone: | → 1.2 |
| Owner: | changed from to |
| Patch needs improvement: | set |
| Status: | assigned → new |
| Triage Stage: | Design decision needed → Accepted |
comment:13 by , 16 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Not sure, but this should work in trunk, so marking it as nfa-blocker (loosing functionality). Modify it if I'm wrong.