On Fri, 2011-09-09 at 10:32 +0200, Patrick Ohly wrote:
> Milan, can you shed some light on why the patch solves #655253? I fail
> to see what e_cal_backend_file_modify_object() has to do with deleting
> one occurrence of a repeatable event.
> 
> If the EXDATE was really necessary to avoid having the original and the
> detached recurrence show up, then IMHO adding the EXDATE only works
> around the real problem. The real problem is more likely to be in the
> matching against RECURRENCE-ID.

        Hi,
sure, the thing why I added it there is that when you move one instance
of a recurring event to another hour, then you are asked whether you
want to change time for all instances or only this instance. Moving only
this instance should create a detached instance, and create an exception
in the master object. Which is exactly what this change does. I chose to
do this in the backed, because it seemed more correct to me, than hunt
for each single place where similar operation is done and add there two
modifications, one for the master object and another for the detached
instance.

Note the issue wasn't shown when you did it, but only when you restart
evolution and e-calendar-factory, because the backend sends you the
master object without exceptions and a detached instance.

Do you think there is a better solution for this?
        Bye,
        Milan


_______________________________________________
evolution-hackers mailing list
evolution-hackers@gnome.org
To change your list options or unsubscribe, visit ...
http://mail.gnome.org/mailman/listinfo/evolution-hackers

Reply via email to