Code

Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#6035 closed (fixed)

SplitDateTimeField doesn't use SplitDateTimeWidget by default

Reported by: dtulig Owned by: dtulig
Component: Forms Version: master
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

When using SplitDateTimeField in a form, the default behaviour is to display a TextInput widget when rendered, not only is it not what is expected, but you are unable to enter a valid datetime because validation fails. Attached patch sets the widget property of SplitDateTimeField to SplitDateTimeWidget. It still allows a custom widget to be passed in as a parameter.

Attachments (1)

splitdatetimefield.diff (1.0 KB) - added by dtulig 6 years ago.

Download all attachments as: .zip

Change History (6)

Changed 6 years ago by dtulig

comment:1 Changed 6 years ago by dtulig

  • Needs documentation unset
  • Needs tests unset
  • Owner changed from nobody to dtulig
  • Patch needs improvement unset
  • Status changed from new to assigned

comment:2 Changed 6 years ago by dtulig

  • Cc david.tulig@… removed

comment:3 Changed 6 years ago by jacob

  • Triage Stage changed from Unreviewed to Accepted

comment:4 Changed 5 years ago by kmtracey

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [9385]) Fixed #6035 -- Gave the SplitDateTimeField the proper default widget. Thanks dtulig.

comment:5 Changed 5 years ago by kmtracey

(In [9386]) [1.0.X] Fixed #6035 -- Gave the SplitDateTimeField the proper default widget. Thanks dtulig.

r9385 from trunk.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.