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

List:       kde-pim
Subject:    Re: [Kde-pim] [PATCH] libkcal: icalformatimpl-duration
From:       Mark Bucciarelli <mark () easymailings ! com>
Date:       2003-08-17 21:34:55
[Download RAW message or body]

On Sunday 17 August 2003 4:51 pm, Cornelius Schumacher wrote:
> On Wednesday 13 August 2003 15:09, Mark Bucciarelli wrote:
> > The attached patch makes libkcal write a duration if an event has one
> > (but no end date) or an todo has one (but no due date).
> >
> > Currently, if an event has a duration but no end date, the duration
> > info is lost; that is, neither is written.  Likewise for a todo with
> > a duration but no due date.
>
> Shouldn't this be solved in the IncideneBase class and the relevant
> subclasses, so that setting an duration unsets the end or due date and
> vice versa?

Seemed more related to how you choose to save an incidence to disk.  What if 
you want to create another implementation where you must save durations 
instead of done dates?  I think it's better to keep the Incidence as general 
as possible (and as close to the spec as possible) and implement these 
duration-less incidences in the persistence stuff.

BTW, as I read the 2445, durations can be negative!

Regards,

Mark
_______________________________________________
kde-pim mailing list
kde-pim@mail.kde.org
http://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]