Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
In date.field.inc at line 229:
'granularity' => drupal_map_assoc(array('year', 'month', 'day', 'hour', 'minute'))
in my opinion there should be
'granularity' => drupal_map_assoc(array('year', 'month', 'day', 'hour', 'minute','second')),
and also in date_admin.inc in condition which is starting at 324 line
if (in_array($widget['type'], array('date_select', 'date_popup'))) ...
there should be also seconds when using 'second' granularity.
Comment | File | Size | Author |
---|---|---|---|
#2 | missing_second-1893826-2.patch | 1.98 KB | Valentine94 |
Comments
Comment #0.0
mist CreditAttribution: mist commentedadditional information
Comment #1
podarokYou should upload patch in git diff format http://drupal.org/patch with Needs review status for review
Comment #2
Valentine94Part of proposed solution with some standards fix's.
Comment #4
jmuzz CreditAttribution: jmuzz commentedThere is a seconds granularity option now, but it can not be used because the "Time increments" dropdown has "1 minute" as the most granular setting so the widget never allows anything other than "00" to be entered in the seconds part.