[Kde-pim] Please test the new KDescendantsProxyModel

[Stephen Kelly]

Hi,

Earlier I checked in a rewrite of KDescendantsProxyModel, but didn't test it 
enough. Sorry for the disruption that caused. The existing version of the 
class has several bugs which really need to be fixed and are blocking other 
issues from being fixed.

To make the refactoring go more smoothly, I've checked it in again now under 
a different name. Please apply the following patches:

kde-devel at bishop:~/kde/src/KDE/kdepimlibs/akonadi{trunk}$ git diff
diff --git a/akonadi/CMakeLists.txt b/akonadi/CMakeLists.txt
index e92f0c3..7ce415f 100644
--- a/akonadi/CMakeLists.txt
+++ b/akonadi/CMakeLists.txt
@@ -113,7 +113,7 @@ set( akonadikde_LIB_SRC
   itemsync.cpp
   itemview.cpp
   job.cpp
-  kdescendantsproxymodel.cpp
+  kdescendantsproxymodel2.cpp
   kjobprivatebase.cpp
   krecursivefilterproxymodel.cpp
   linkjob.cpp


kde-devel at bishop:~/kde/src/KDE/kdepim/libkdepim{master}$ git diff .
diff --git a/libkdepim/CMakeLists.txt b/libkdepim/CMakeLists.txt
index f0ce3c4..0d8d950 100644
--- a/libkdepim/CMakeLists.txt
+++ b/libkdepim/CMakeLists.txt
@@ -17,7 +17,7 @@ set(kdepim_LIB_SRCS
    addemailaddressjob.cpp
    addcontactjob.cpp
    openemailaddressjob.cpp
-   kdescendantsproxymodel.cpp
+   kdescendantsproxymodel2.cpp
    ktimeedit.cpp
    kprefsdialog.cpp
    kdateedit.cpp


And test the applications you work on wherever they use that proxy. The 
CollectionComboBox uses it internally, so test wherever you use that too.

All the best,

Steve.



_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/