297721 - implement calICalendarProviders and make calCalendarManager use them

Status: NEW

(Calendar :: General, defect)

Description

[Dan Mosedale (:dmosedale, :dmose)]

Comment 1

[Dan Mosedale (:dmosedale, :dmose)]

Attachment: first cut provider patch, v1

This is a first cut of interface changes, core code changes, Lightning changes,
and implementation of calStorageProviderService.  If you guys could give this a
quick once-over before I go port various parts of this patch to Sunbird and the
other providers, I'd appreciate it.

Comment 2

[Dan Mosedale (:dmosedale, :dmose)]

Attachment: patch, v2

Whoops, forgot a file the first time around.

Comment 3

[Mike Shaver (:shaver emeritus)]

Not blocking 0.8.

Comment 4

[Simon Paquet [:sipaq]]

Dan, the patch in this bug is already three months old. Is it ready for reviews
or is this just a work in progress?

Comment 5

[Joey Minta]

*** Bug 292143 has been marked as a duplicate of this bug. ***

Comment 6

[Dan Mosedale (:dmosedale, :dmose)]

sipaq: this is a work in progress.

Comment 7

[Dan Mosedale (:dmosedale, :dmose)]

jminta has pointed out that it's not entirely clear that calICalendarProvider needs to be separate from calICalendar.  We should make a decision on that before moving forward.

Comment 8

[Dan Mosedale (:dmosedale, :dmose)]

Cleaning up incorrectly assigned bugs; search for dmosecleanup to get rid of this bugmail.

Comment 9

[Matthew (lilmatt) Willis]

mvl:
What's next for this bug?  Should it be closed, morphed, or something altogether different?

Comment 10

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

This bug should be fixed, simple as that.

For example: http://lxr.mozilla.org/seamonkey/source/calendar/base/src/calCalendarManager.js#425 does not use the calICalendarProvider interface.