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

List:       kde-commits
Subject:    KDE/kdepim/korganizer/views/todoview
From:       Volker Krause <vkrause () kde ! org>
Date:       2010-09-25 15:11:13
Message-ID: 20100925151113.BC7EAAC857 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1179464 by vkrause:

Don't assert when loading broken task relations.


 M  +2 -3      kotodomodel.cpp  


--- trunk/KDE/kdepim/korganizer/views/todoview/kotodomodel.cpp #1179463:1179464
@@ -478,10 +478,9 @@
     Akonadi::Item parentItem = mCalendar->itemForIncidenceUid( parentUid );
     Incidence::Ptr incidence = CalendarSupport::incidence( parentItem );
     Todo::Ptr relatedTodo = incidence.dynamicCast<Todo>();
-    Q_ASSERT( relatedTodo );
 
-    // check if there are recursively linked todos
-    if ( isInHierarchyLoop( todo ) ) {
+    // check if there are recursively linked todos, or if we got broken input data
+    if ( isInHierarchyLoop( todo ) || !relatedTodo ) {
       // recursion detected, break recursion
       return insertTodo( todoItem, false );
     }
[prev in list] [next in list] [prev in thread] [next in thread]