The default bug view has changed. See this FAQ.

rationalize calICalendar deleteItem and modifyItem w.r.t. occurrences

RESOLVED FIXED

Status

Calendar
Provider: CalDAV
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: dmose, Unassigned)

Tracking

Details

(Reporter)

Description

11 years ago
It seems that the views currently expect to be able to, in some situations, call deleteItem() and modifyItem() with proxy items.  The storage provider currently handles this case, ICS, CalDAV, and memory do not.  

The first thing that needs to be done is that we need to decide on the exact semantics for all such cases.  An example of this is that in the storage provider, if you pass in a proxy to deleteItem, it assumes that this item must be an existing exception, and attempts to delete that exception.  However, the semantics for what the providers should do if this proxy is simply an existing, non-exception occurrence are not documented.
please note that the patch for bug #327890 provides the functionality for ICS and memory provider to be able to handle proxies passed to deleteItem() and modifyItem(). I discussed with dmose in IRC today that we'll leave the calDAV provider as it is for the lightning 0.1 release, so this is basically out of the way.
(Reporter)

Comment 2

11 years ago
OK, the thing to be release noted here is the as-yet fixed CalDAV piece.
No longer blocks: 327889
Whiteboard: [cal relnote]
(Reporter)

Comment 3

11 years ago
OK, the thing to be release noted here is the as-yet fixed CalDAV piece.
(Reporter)

Comment 4

11 years ago
Er, "as-yet unfixed CalDAV piece".
The bugspam monkeys have struck again. They are currently chewing on default assignees for Calendar. Be afraid for your sanity!
Assignee: base → nobody
(Reporter)

Updated

11 years ago
Component: Internal Components → Provider: CalDav

Updated

11 years ago
QA Contact: base → caldav-provider
CalDAV was likely fixed in bug 328756.
Adding qawanted to verify that this is in fact fixed.
Keywords: qawanted
Whiteboard: [cal relnote]

Comment 7

11 years ago
(In reply to comment #6)
> CalDAV was likely fixed in bug 328756.
> Adding qawanted to verify that this is in fact fixed.
> 
Could you add a comment explaining this? I don't see how bug 328756 fixed CalDAV. Further, it's not really clear what you want QA to verify in this situation.

Could you include some details about how to test this, or some lxr links to where this issue is represented in the code? That would help the QA team quite a bit. Thanks!

Comment 8

11 years ago
(In reply to comment #7)
> Could you add a comment explaining this? I don't see how bug 328756 fixed
> CalDAV. Further, it's not really clear what you want QA to verify in this
> situation.
The real bug is bug 328576. (inversion of 5 and 7).  To test this, check the modified single occurrences of repeating events (Edit this occurrence) are properly round-tripped from a caldav server.

Comment 9

11 years ago
I tested modified recurrences with Cosmo 0.5 and the events are displayed correctly (also after restart). 
However, importing an ics file with modified occurrences does not yet work!

Comment 10

11 years ago
(In reply to comment #9)
> However, importing an ics file with modified occurrences does not yet work!

But this is probably bug 354578.

Comment 11

11 years ago
Resolving it fixed based on comment 6 and comment 9.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Keywords: qawanted
You need to log in before you can comment on or make changes to this bug.