[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-pim
Subject:    Re: [Kde-pim] [PATCH] Server-based resources and editing events
From:       Cornelius Schumacher <schumacher () kde ! org>
Date:       2003-12-11 23:22:14
[Download RAW message or body]

On Thursday 11 December 2003 17:55, Best, Jan-Pascal van wrote:
>
> > > It would mean changing the signature of
> > > ResourceCalendar::save() to add an
> > > (optional) QPtrList<KCal::Incidence> parameter. Resource
> > > implementations based
> > > on files (local or remote) can ignore this parameter and just
> > > save the whole lot,
> > > but ldap/database/exchange resources can just save the
> > > modified incidences.
>
> I realised that would be a bit awkward: For new events, the event is
> added to the calendar by the koEventEditor by a mCalendar->addEvent()
> call. I'd think changed events should be notified to the
> calendar in same way. Attached patch adds a method eventChanged()
> to KCal::Calendar, which is implemented in CalendarResources by
> notifying the appropriate resource. ResourceCalendar also gets an
> eventChanged() method. The Exchange resource implements this by
> writing the modified event back to the server, which works
> wonderfully.
>
> What do you think of this?

Have a look at the beginChange() and endChange() methods in Calendar. 
They are intended to do the same, but with proper locking of the 
resources or whatever else is needed. Would that also fit your needs?

One problem is that currently KOrganizer doesn't always calls these 
methods when events are changed. This has to be improved.

-- 
Cornelius Schumacher <schumacher@kde.org>
_______________________________________________
kde-pim mailing list
kde-pim@mail.kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
kde-pim home page at http://pim.kde.org/
[prev in list] [next in list] [prev in thread] [next in thread]