285019 - import calendar doesn't work

Status: RESOLVED FIXED

(Calendar :: Sunbird Only, defect)

Description

[Michiel van Leeuwen (email: mvl+moz@)]

file->import:

JavaScript strict warning: chrome://calendar/content/importExport.js, line 2029:
reference to undefined property gICalLib.addObserver
JavaScript error: chrome://calendar/content/importExport.js, line 2029:
gICalLib.addObserver is not a function

Comment 1

[Michiel van Leeuwen (email: mvl+moz@)]

import should import an cvs file, or xml, or whatever (importExport.js) into a
calendar.
So this bug is mainly to make importExport.js work with the new interfaces.

Comment 2

[Michiel van Leeuwen (email: mvl+moz@)]

Attachment: patch part 1 (checked in)

This patch should be the beginning of a fix. It uses the ics service to parse a
ics file, instead of doing it in regexps. It might need some work on the
front-end, like the ability to select a calendar to import into.

Comment 3

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 185046 [details] [diff] [review]
patch part 1 (checked in)

patch checked in. Leaving bug open for further issues

Comment 4

[Michiel van Leeuwen (email: mvl+moz@)]

Attachment: misc fixes

It turns out there are only a few small problems left. This patch should fix
things.

Comment 5

[Joey Minta]

Comment on attachment 190747 [details] [diff] [review]
misc fixes

+	    alert(getStringBundle().GetStringFromName("unableToRead") +
filePath + "\n"+ex );

promptService, someday in the future.

r=jminta

Comment 6

[Michiel van Leeuwen (email: mvl+moz@)]

patch checked in. I believe import works now.

Comment 7

[Reed Loden [:reed]]

The bugspam monkeys have been set free and are feeding on Calendar :: Sunbird Only. Be afraid for your sanity!