871199 - All day events are corrupted by ical.js backend

Status: RESOLVED FIXED

(Calendar :: ICAL.js Integration, defect)

Description

[Matthew Mecca [:mmecca]]

All day events and other properties of type VALUE=DATE are incorrectly formatted by the ical.js backend, causing data corruption on ics calendars containing these events.

STR:
1) Switch to the ical.js backend
2) Create and open a new ics calendar
3) Create an all day event

Results:
On restarting Thunderbird, calendar parsing fails with

Error: 'Error: invalid date-time value: "2013--0-5-T1::"' when calling method: [calIIcalComponent::getNextProperty] = NS_ERROR_XPC_JS_THREW_JS_OBJECT
Source file: resource://calendar/modules/calIteratorUtils.jsm
Line: 186


Date properties are written in the format:
DTSTART:2013-05-11

but should be in the format:
DTSTART;VALUE=DATE:20130511

Comment 1

[Philipp Kewisch [:Fallen][☁️📆]]

Attachment: Fix - v1

Turns out this is https://github.com/mozilla-comm/ical.js/pull/84 and I just forgot to include the patch in Lightning.

I'd appreciate if you could review the PR on github for the tests it adds. The patch uploaded here just contains the changes in Lightning.

Comment 2

[Philipp Kewisch [:Fallen][☁️📆]]

Comment on attachment 749897 [details] [diff] [review]
Fix - v1

r=lightsofapollo at github. Some slight changes I'll add before commit.

Comment 3

[Philipp Kewisch [:Fallen][☁️📆]]

Pushed to comm-central changeset 65fd9a64e428

Comment 4

[Philipp Kewisch [:Fallen][☁️📆]]

Backported to releases/comm-aurora changeset 37c0e4310473