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

List:       kde-pim
Subject:    Re: [Kde-pim] Re: PATCH: addIncidence() DCOP call for KOrganizer
From:       Reinhold Kainhofer <reinhold () kainhofer ! com>
Date:       2005-06-27 8:03:36
Message-ID: 200506271003.37440.reinhold () kainhofer ! com
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


Am Montag, 27. Juni 2005 09:54 schrieb David Jarvie:
>  +bool CalendarView::addIncidence( const QString& ical )
> +{
> +  kdDebug(5850) << "CalendarView::addIncidence:\n" << ical << endl;
> +  ICalFormat format;
> +  Incidence* incidence = format.fromString(ical);

Uh oh, this is wrong, since it completely ignores the time zone. You need to 
pass the time zone (of the active calendar?) to the ICalFormat constructor.

You also need to check the code whether fromString correctly uses a VTIMEZONE, 
if given in the ical string.

Reinhold
-- 
------------------------------------------------------------------
Reinhold Kainhofer, Vienna, Austria
email: reinhold@kainhofer.com, http://reinhold.kainhofer.com/
 * Financial and Actuarial Mathematics, TU Wien, http://www.fam.tuwien.ac.at
 * K Desktop Environment, http://www.kde.org/, KOrganizer / KPilot maintainer

[Attachment #5 (application/pgp-signature)]

_______________________________________________
kde-pim mailing list
kde-pim@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]