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

List:       kde-commits
Subject:    branches/kdepim/enterprise/kdepim/kresources/kolab/kcal
From:       Allen Winter <winter () kde ! org>
Date:       2010-03-20 20:35:42
Message-ID: 20100320203542.18DD8AC857 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1105646 by winterz:

wow. there was an assert being hit when creating tasks because we didn't
check for valid StartDate before removing the start-date element.
this makes things work a bit nicer and no more Assert message in the console.
MERGE: none (already in trunk)


 M  +2 -2      task.cpp  


--- branches/kdepim/enterprise/kdepim/kresources/kolab/kcal/task.cpp #1105645:1105646
@@ -329,7 +329,7 @@
   QDomElement element = document.createElement( "task" );
   element.setAttribute( "version", "1.0" );
   saveAttributes( element );
-  if ( !hasStartDate() ) {
+  if ( !hasStartDate() && startDate().isValid() ) {
     // events and journals always have a start date, but tasks don't.
     // Remove the entry done by the inherited save above, because we
     // don't have one.
@@ -385,7 +385,7 @@
 
   // Only KCal priority set, use that.
   else if ( mKolabPriorityFromDom == -1 && mKCalPriorityFromDom != -1 ) {
-    kdWarning() << "decideAndSetPriority(): No Kolab priority found, only the KCal priority!" << endl; 
+    kdWarning() << "decideAndSetPriority(): No Kolab priority found, only the KCal priority!" << endl;
     setPriority( mKCalPriorityFromDom );
   }
 
[prev in list] [next in list] [prev in thread] [next in thread]