AnewtCalendar uses incorrect date/datetime syntax
Bug #717882 reported by
Jasper Looije
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Anewt |
Confirmed
|
Undecided
|
wouter bolsterlee | ||
Bug Description
AnewtCalendar renders date components in VEVENT incorrectly. Acording to the RFC5545 by default a date-time stamp (separeted by a T, iso8601) is expected. In the case of an all-day event (e.g. anniversary) only the date is suplied.
Currently all date components (DTSTAMP DTSTART DTEND) are rendered as if there is no time-part:
DTSTAMP; VALUE=DATE:
The presens of the time part is wrong, because the VALUE=DATE implies only a date.
Correct behavior should be
- in case of a full timestamp:
DTSTART:
- in case of an all-day event
DTSTART; VALUE=DATE:20110212
| Changed in anewt: | |
| assignee: | nobody → Jasper Looije (jasper-jamu) |
Revision history for this message
| Jasper Looije (jasper-jamu) wrote | #1 |
Revision history for this message
| Jasper Looije (jasper-jamu) wrote | #2 |
Revision history for this message
| Jasper Looije (jasper-jamu) wrote | #3 |
| Changed in anewt: | |
| assignee: | Jasper Looije (jasper-jamu) → Wouter Bolsterlee (uws) (uws) |
| status: | New → Confirmed |
To post a comment you must log in.
Attached patch generates correct ical output. In my test script following warning is generated:
PHP Warning: assert(): Assertion "is_int(
I am not sure wether my patch triggers this or it is my test script.