Fix checkbox management in component tree

Qt 5.2 introduced a behavior change (QTBUG-40060)
which broke the checkbox management in component tree.
Whenever Qt wants to set PartiallyChecked state
for any node, we toggle the state instead.

Change-Id: I4d8c8300e52b886060395cda86b238a7c1619c3f
Reviewed-by: Niels Weber <niels.weber@theqtcompany.com>

1 files changed, 6 insertions, 1 deletions

diff --git a/src/libs/installer/componentmodel.cpp b/src/libs/installer/componentmodel.cpp
index fae562516..a452829fd 100644
--- a/src/libs/installer/componentmodel.cpp
+++ b/src/libs/installer/componentmodel.cpp
@@ -194,7 +194,12 @@ bool ComponentModel::setData(const QModelIndex &index, const QVariant &value, in
 
     if (role == Qt::CheckStateRole) {
         ComponentSet nodes = component->childItems().toSet();
-        QSet<QModelIndex> changed = updateCheckedState(nodes << component, Qt::CheckState(value.toInt()));
+        Qt::CheckState newValue = Qt::CheckState(value.toInt());
+        if (newValue == Qt::PartiallyChecked) {
+            const Qt::CheckState oldValue = component->checkState();
+            newValue = (oldValue == Qt::Checked) ? Qt::Unchecked : Qt::Checked;
+        }
+        QSet<QModelIndex> changed = updateCheckedState(nodes << component, newValue);
         foreach (const QModelIndex &index, changed) {
             emit dataChanged(index, index);
             emit checkStateChanged(index);