Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Problem/Motivation
When a date-only field is rendered out with the default field formatter, an incorrect ISO timestamp attribute is attached to the <time> element. Basically, the time portion is taken from the current time, instead of using the default time (noon UTC).
Proposed resolution
Move the ISO date creation to after the default time has been set.
Remaining tasks
Fix problem.
Add test coverage.
User interface changes
None.
API changes
None.
Data model changes
None.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | interdiff-02-03.txt | 2.29 KB | mpdonadio |
| #3 | 2716991-03.patch | 3.07 KB | mpdonadio |
| #3 | 2716991-test-only.patch | 1.98 KB | mpdonadio |
| #2 | 2716991-02.patch | 2.59 KB | mpdonadio |
Comments
Comment #2
mpdonadioStill need to add the test to demonstrate the problem, but XPath isn't my strong point for a quick fix on this.
Comment #3
mpdonadioComment #7
mpdonadioI'm happy with this. Review away. Putting back to 8.1.x b/c this is a bug w/ a demonstrable test, and would qualify getting into a minor.
Comment #8
tstoecklerYup, thanks for the test coverage. Looks great!
Comment #9
alexpottCommitted 93447ea and pushed to 8.1.x and 8.2.x. Thanks!