Don't call invalidate when resetting the QSortFilterProxyModel.

The invalidate connection is made before connecting the source model,
which means it gets called first.

If a second proxy model is used with it, the sourceLayoutAboutToBeChanged
of the second proxy is therefore called before its sourceReset slot, but
after the persistent indexes of the first model have been invalidated.

The invalidate call is not needed because clear_mappings is called
when the source model is reset anyway.

Merge-request: 416
Reviewed-by: Olivier Goffart <ogoffart@trolltech.com>

1 files changed, 2 insertions, 1 deletions

diff --git a/tests/auto/modeltest/dynamictreemodel.cpp b/tests/auto/modeltest/dynamictreemodel.cpp
index 24d3ab3ce6..b572eb16a1 100644
--- a/tests/auto/modeltest/dynamictreemodel.cpp
+++ b/tests/auto/modeltest/dynamictreemodel.cpp
@@ -160,10 +160,11 @@ QVariant DynamicTreeModel::data(const QModelIndex &index, int role) const
 
 void DynamicTreeModel::clear()
 {
+  beginResetModel();
   m_items.clear();
   m_childItems.clear();
   nextId = 1;
-  reset();
+  endResetModel();
 }