In 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 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.

#2 missing_second-1893826-2.patch1.98 KBValentine94
FAILED: [[SimpleTest]]: [MySQL] 3,983 pass(es), 1,259 fail(s), and 258 exception(s). View
Members fund testing for the Drupal project. Drupal Association Learn more


mist’s picture

Issue summary: View changes

additional information

podarok’s picture

Version: 7.x-2.6 » 7.x-2.x-dev
Issue summary: View changes
Status: Needs review » Patch (to be ported)

You should upload patch in git diff format with Needs review status for review

Valentine94’s picture

Status: Patch (to be ported) » Needs review
1.98 KB
FAILED: [[SimpleTest]]: [MySQL] 3,983 pass(es), 1,259 fail(s), and 258 exception(s). View

Part of proposed solution with some standards fix's.

Status: Needs review » Needs work

The last submitted patch, 2: missing_second-1893826-2.patch, failed testing.

jmuzz’s picture

Status: Needs work » Active

There 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.