Edgewall Software
Modify

Ticket #7921 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

Invalid ISO8601 datetimes are used in the timeline and html

Reported by: trs@… Owned by: cboos
Priority: normal Milestone: 0.11.3
Component: timeline Version: 0.11
Severity: normal Keywords: iso8601 datetimes
Cc:
Release Notes:
API 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

t7921-iso-8601-fixes-r7795.diff (2.2 KB) - added by cboos 3 years ago.
Fix formatting of dates when using the iso:8601 format.

Download all attachments as: .zip

Change History

comment:1 Changed 3 years ago by cboos

  • Milestone set to 0.11.3
  • Owner set to cboos
  • Version changed from none to 0.11

Sorry, my misunderstanding. I'll fix that.

Changed 3 years ago by cboos

Fix formatting of dates when using the iso:8601 format.

comment:2 Changed 3 years ago by cboos

  • Milestone changed from 0.11.4 to 0.11.3

comment:3 Changed 3 years ago by cboos

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

Patch applied in r7823. Thanks for raising the issue!

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
to The owner will be changed from cboos. Next status will be 'closed'
Author


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

 
Note: See TracTickets for help on using tickets.