Closed Bug 427617 Opened 12 years ago Closed 11 years ago
ics import fails if event has RDATE property with value type PERIOD (worked in 0
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:188.8.131.52) Gecko/20080311 Firefox/184.108.40.206 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:220.127.116.11) Gecko/20080311 Firefox/18.104.22.168 Import of certain .ics files produces this error: Unable to read from file:C:\Documents and Settings\Thosp\My Documents \Tennessee_Trails_Hike_04-12-2008.ics [Exception... "Component returned failure code: 0x8000ffff (NS_ERROR_UNEXPECTED) [calIRecurrenceItem.icalProperty]" nsresult: "0x8000ffff (NS_ERROR_UNEXPECTED)" location: "JS frame :: file:///C:/Documents%20and%20Settings/Thosp/Application%20Data/Thunderbird/Profiles/sgbdalrp.default/extensions/%7Be2fda1a4-762b-4020-b5ad-a41df1933103%7D/js/calItemBase.js :: anonymous :: line 604" data: no] Reproducible: Always Steps to Reproduce: 1. Go to theleafchronicle.com click on more events, click on event click on icon that says MS Outlook. Save as .ics in My Documents 2. Go to Thunderbird Lightning and click on IMPORT. 3. BINGO! Actual Results: Under 0.8 nothing happens. Expected Results: Event should have been add to calendar and visible, just as it did with 0.7...
This file, and several others from the same source have passed ICal validation from http://severinghaus.org/projects/icv/. I copied the text from the QuickView screen and pasted it there. However, events from this source worked with Lightning 0.7
Attachment #314159 - Attachment mime type: text/calendar → text/plain
Confirmed. Sunbird 0.5 (20070614) and Sunbird 0.7 (20071023): File can be imported. Event is shown in view and unifinder. Sunbird 0.8 fails during import: Error: [Exception... "Component returned failure code: 0x8000ffff (NS_ERROR_UNEXPECTED) [calIRecurrenceItem.icalProperty]" nsresult: "0x8000ffff (NS_ERROR_UNEXPECTED)" location: "JS frame :: file:///[...]/js/calItemBase.js :: anonymous :: line 604" data: no] Regression range: Works in Sunbird 0.8pre (2007-12-03-04) Fails in Sunbird 0.8pre (2007-12-04-05) Checkins during regression range: <http://tinyurl.com/5am6qr> Bug 400950 seems to be the only related checkin. Question: It is an error caused by the changes in Bug 400950? Or do we now fail on some invalid .ics that was silently ignored before? Note: The import succeeds if I remove the following line from the .ics file: RDATE;VALUE=PERIOD:20080412T080000/20080412T080000
I'd say the ics is valid: <http://tools.ietf.org/rfcmarkup?doc=draft-ietf-calsify-rfc2445bis-08#section-22.214.171.124>. We've never supported PERIODs, and I suspect we've filtered those RDATEs silently out in the past, while we don't do that any more. But I need to investigate if and how we could fix this.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: .ics file import that worked with Lightning 0.7 doesn't work with 0.8 → ics import fails if event has RDATE property with value type PERIOD (worked in 0.7)
Here is another sample with repeating dates e.g. birthdays - #1 causes error(uses "rdate") - #2 is fine(uses "rrule"): #1 (calendar file with repeating date for next 10 years exported from Lotus Notes 8): BEGIN:VEVENT DTSTART;TZID="W. Europe":20040428T170000 DTEND;TZID="W. Europe":20040428T170000 TRANSP:OPAQUE RDATE;VALUE=PERIOD:20040428T150000Z/20040428T150000Z ,20050428T150000Z/20050428T150000Z,20060428T150000Z/20060428T150000Z ,20070428T150000Z/20070428T150000Z,20080428T150000Z/20080428T150000Z ,20090428T150000Z/20090428T150000Z,20100428T150000Z/20100428T150000Z ,20110428T150000Z/20110428T150000Z,20120428T150000Z/20120428T150000Z ,20130428T150000Z/20130428T150000Z,20140428T150000Z/20140428T150000Z ,20150428T150000Z/20150428T150000Z,20160428T150000Z/20160428T150000Z ,20170428T150000Z/20170428T150000Z,20180428T150000Z/20180428T150000Z ,20190428T150000Z/20190428T150000Z,20200428T150000Z/20200428T150000Z ,20210428T150000Z/20210428T150000Z,20220428T150000Z/20220428T150000Z ,20230428T150000Z/20230428T150000Z,20240428T150000Z/20240428T150000Z ,20250428T150000Z/20250428T150000Z,20260428T150000Z/20260428T150000Z ,20270428T150000Z/20270428T150000Z,20280428T150000Z/20280428T150000Z DTSTAMP:20080808T134701Z CLASS:PRIVATE SUMMARY:Hochzeitstag UID:33157D9E2E2AF518C1256E7F00545F07-Lotus_Notes_Generated X-LOTUS-PARITAL-REPEAT:TRUE X-LOTUS-UPDATE-SEQ:1 X-LOTUS-UPDATE-WISL:$S:1;$L:1;$B:1;$R:1;$E:1 X-LOTUS-START;TZID="W. Europe":20040428T170000 X-LOTUS-END;TZID="W. Europe":20040428T170000 X-LOTUS-NOTESVERSION:2 X-LOTUS-APPTTYPE:4 X-LOTUS-CHILD_UID:CD6A826620A35A2EC1256E7F005473DF END:VEVENT #2 (repeating date generated with Sunbird): BEGIN:VEVENT CREATED:20080819T153936Z LAST-MODIFIED:20080819T154053Z DTSTAMP:20080819T153936Z UID:8f2103b9-389f-47b1-bcf6-46f179f586ad SUMMARY:Geb. Test RRULE:FREQ=YEARLY;INTERVAL=1;BYMONTHDAY=25;BYMONTH=8 DTSTART;VALUE=DATE:20080825 DTEND;VALUE=DATE:20080826 CATEGORIES:Birthday TRANSP:TRANSPARENT END:VEVENT
Lotus Notes 8 uses RDATE on ALL anniversaries. There is no way to manually remove those RDATE records from the exported file before importing it in e.g. Sunbird. Same problem with Lightning.
Only the release driver can grant blocking-calendar0.9.
OS: Windows XP → All
Hardware: PC → All
I think this is not a regression, because those RDATEs have been filtered out in previous versions, but of course dataloss. We could ignore the error, but this would silently cause dataloss on writing (e.g. subscribing to such a file) which is IMO not acceptable. As far as I see, this requires major changes to the core code and thus won't makes it for 0.9. Sorry.
I was using Sunbird to view calendars exported from Lotus Notes so this is a regression for me. I didn't know that entries were being missed out, so I was already suffering silent data loss. Would it be possible to load the incomplete calendar, display a warning and mark it as read-only to prevent overwriting? Is this a bug that can be pushed back to upstream libical?
(In reply to comment #9) > already suffering silent data loss. Would it be possible to load the incomplete > calendar, display a warning and mark it as read-only to prevent overwriting? We're already in string freeze for 0.9. > Is this a bug that can be pushed back to upstream libical? AFAIK it's not a problem of libical.
In some cases, this bug causes pretty major confusion. Currently, I use Thunderbird/Lightning in a Lotus Notes 8 environment. When someone reschedules an instance of a recurring meeting from Notes, the e-mail I receive appears to be blank. It looks like Lightning parses the ICS, and when it fails on the RDATE/PERIOD, all parsing of the message stops, so nothing shows in the message pane. The ICS does not even appear in the attachment list, so I can't detach it to munge it by hand. Is there any hope for this to be fixed in a future Lightning?
This is a patch is taking over the PERIODs' start dates, so such calendars are at least viewable then, although the end date of such RDATEs may not be correct. On roundtrip the PERIODs are exchanged by DATE-TIMES, thus dataloss is still immanent.
Attachment #371727 - Flags: review?(philipp)
Comment on attachment 371727 [details] [diff] [review] patch - v1 Looks good, r=philipp
Attachment #371727 - Flags: review?(philipp) → review+
Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/aef2a3c5c542> -> FIXED Set up follow-up bug 489747 for proper support of end date.
Assignee: nobody → dbo.moz
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.0
These bugs are likely targeted at Lightning 1.0b1, not Lightning 1.0. If this change was done in error, please adjust the target milestone to its correct value. To filter on this bugspam, you can use "lightning-10-target-move".
Target Milestone: 1.0 → 1.0b1
You need to log in before you can comment on or make changes to this bug.