298772 - Pluggable interfaces for import and export

Status: RESOLVED FIXED

(Calendar :: Internal Components, defect)

Description

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

import and export is currently done in one big (chrome) js file. This isn't
extendable. Also, the file isn't updated for the new api's.

Comment 1

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

Attachment: initial patch

This patch defines new interfaces and implements them. I didn't port all of the
importers/exporters. I doubt they are needed, but if they are, we can always
add them after we agree on an interface etc.

The importers/exporters are pluggable. nsICategoryManager is used to register
them
I don't like the way the extensions for the filepicker are stored in the
im/exporters. For example, *.htm and *.html will be two lines. That's not
really nice. But i do want to be able to have an extension that also exports to
html, but in some other cool layout, not the default boring list.

Comments on the interface or the impl?

Comment 2

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

Attachment: patch v2

Updated patch. Moved the files to better locations, remove the gCalendarWindow
stuff etc.

Comment 3

[Mike Shaver (:shaver emeritus)]

Comment on attachment 188229 [details] [diff] [review]
patch v2

It's a shame we can't do arrays of AString, to avoid needing the calIFileType
interface...

I think we want the methods to be called importFromStream and exportToStream,
since we're not really exporting the stream.  (We sort of are importing the
stream, but I think the symmetry here will win.)

We should probably call the CSV importer "OutlookCSVImporter", since it's not
really a general CSV-handler, right?

spelling: "Comma Separated Values" (also  an l10n hazard, but that can be a
follow-up)

In parseTextField, you have an else after a return, which is a non-sequitur you
should avoid. =)

I don't see the calendar/Makefile.in change that would be needed to descend
into calendar/import-export and actually build all these goodies, which I
presume you want to do.

r=shaver with the above fixes made.  Sorry for the delay!

Comment 4

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

Patch checked in. I will file new bugs for the l10n issues.