Opened 15 years ago
Closed 15 years ago
#7921 closed defect (fixed)
Invalid ISO8601 datetimes are used in the timeline and html
| Reported by: | Owned by: | Christian Boos | |
|---|---|---|---|
| Priority: | normal | Milestone: | 0.11.3 |
| Component: | timeline | Version: | 0.11 |
| Severity: | normal | Keywords: | iso8601 datetimes |
| Cc: | Branch: | ||
| Release Notes: | |||
| API Changes: | |||
| Internal Changes: | |||
Description
In the "from" query argument to the timeline view and in the title attribute of links to the timeline, datetimes are used in an invalid ISO8601 format. The title attribute in particular is useful for parsing, which seems to be why it is there (and hence validity matters).
Datetimes in these locations are currently formatted like so: YYYY-MM-DDTHH:MM:SSZ±HHMM
The ISO8601 spec specifies such a datetime should be formatted instead like: YYYY-MM-DDTHH:MM:SS±HH:MM
The "Z" after the time is used to denote a UTC timezone (offset of Zero). Combining it with an offset of another timezone does not make sense. Additionally, the spec requires a colon between the hours and minutes of the offset.
Attachments (1)
Change History (4)
comment:1 by , 15 years ago
| Milestone: | → 0.11.3 |
|---|---|
| Owner: | set to |
| Version: | none → 0.11 |
by , 15 years ago
| Attachment: | t7921-iso-8601-fixes-r7795.diff added |
|---|
Fix formatting of dates when using the iso:8601 format.
comment:2 by , 15 years ago
| Milestone: | 0.11.4 → 0.11.3 |
|---|
comment:3 by , 15 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Patch applied in r7823. Thanks for raising the issue!
Sorry, my misunderstanding. I'll fix that.