From: pavelpa <pavelpa@localhost>
Date: Mon, 18 Jan 2010 08:49:50 +0000 (+0000)
Subject: fix: segfault
X-Git-Tag: 0.5.0~300
X-Git-Url: https://git.toastfreeware.priv.at/toast/confclerk.git/commitdiff_plain/8035cef9ea9e02e0f5db35bb8313abcba9461f20

fix: segfault
 - fixes segfault when switching days in "Day View"
 - TODO: needs to be verified, 'cause it looks like
         it shouldn't work, but it does
   - when calling 'QAbstractItemModel::removeRows()' it returns false,
     but it prevents application from crash(segfault)
     - possible explanation is that the timing has changed and so the
       conditions for the segfault
---

diff --git a/src/mvc/eventmodel.cpp b/src/mvc/eventmodel.cpp
index 41fae85..25729de 100644
--- a/src/mvc/eventmodel.cpp
+++ b/src/mvc/eventmodel.cpp
@@ -119,6 +119,16 @@ int EventModel::rowCount (const QModelIndex & parent) const
 
 void EventModel::loadEvents(const QDate &aDate, int aConferenceId)
 {
+    for(int i=0; i<mGroups.count(); i++)
+    {
+        QModelIndex idx = index(i,0);
+        Group group = mGroups[i];
+        beginRemoveRows(idx,0,group.mChildCount-1);
+        bool ok = removeRows(0,group.mChildCount,idx);
+        endRemoveRows();
+        //qDebug() << "removing " << group.mChildCount << " events from group:" << i << idx.data() << ":" << ok;
+    }
+
     mEvents.clear();
 
     // check for existence of the conference in the DB
@@ -132,6 +142,16 @@ void EventModel::loadEvents(const QDate &aDate, int aConferenceId)
 
 void EventModel::loadFavEvents(const QDate &aDate, int aConferenceId)
 {
+    for(int i=0; i<mGroups.count(); i++)
+    {
+        QModelIndex idx = index(i,0);
+        Group group = mGroups[i];
+        beginRemoveRows(idx,0,group.mChildCount-1);
+        bool ok = removeRows(0,group.mChildCount,idx);
+        endRemoveRows();
+        //qDebug() << "removing " << group.mChildCount << " events from group:" << i << idx.data() << ":" << ok;
+    }
+
     mEvents.clear();
 
     // check for existence of the conference in the DB