[Korganizer-devel] [Bug 189711] libkcal stores illegal negative DTSTART in ics / ical files

[Stephan Diestelhorst]

https://bugs.kde.org/show_bug.cgi?id=189711





--- Comment #2 from Stephan Diestelhorst <stephan diestelhorst gmail com>  2009-04-15 14:07:41 ---
...whoops... accidential send pressed...

As per RFC 2445 4.6.2 the dtstart entry for todoc is optional and according to
4.8.2.4 and 4.3.4 has to be in YYYYMMDD encoding.

The comment suggests that the DTSTART field should be ignored anyway.
The following patch fixes this for my by dropping the bogus and optional
DTSTART field if not used.

diff -r afa0e6dcdc6a kcal/icalformat_p.cpp
--- a/kcal/icalformat_p.cpp     Wed Apr 15 14:06:13 2009 +0200
+++ b/kcal/icalformat_p.cpp     Wed Apr 15 14:06:25 2009 +0200
@@ -200,7 +200,7 @@
   }

   // start time
-  if ( todo->hasStartDate() || todo->recurs() ) {
+  if ( todo->hasStartDate() ) {
     icaltimetype start;
     if ( todo->allDay() ) {
       start = writeICalDate( todo->dtStart( true ).date() );

Please apply upstream!

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.