From 662184ac68803aac553921520a1b4b7b23a0633c Mon Sep 17 00:00:00 2001
From: Ivan Solovev <ivan.solovev@qt.io>
Date: Wed, 15 Jun 2022 13:10:10 +0200
Subject: QComboBox: emit currentIndexChanged() and currentTextChanged() when
 the model is cleared

QComboBox uses QPersistentModelIndex to store the current index of the
underlying model. When the model is cleared, that index is automatically
invalidated, so calling QComboBoxPrivate::setCurrentIndex(QModelIndex())
does not result in signals being emitted, because we do not detect the
index change.

This patch uses indexBeforeChange to detect such situation and emit all
necessary signals.

Fixes: QTBUG-103007
Pick-to: 6.4 6.3 6.2
Change-Id: I29326830a30a17900839e8d1737a08bd940081ea
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
---
 .../widgets/widgets/qcombobox/tst_qcombobox.cpp    | 33 ++++++++++++++++++++++
 1 file changed, 33 insertions(+)

(limited to 'tests')

diff --git a/tests/auto/widgets/widgets/qcombobox/tst_qcombobox.cpp b/tests/auto/widgets/widgets/qcombobox/tst_qcombobox.cpp
index 987db3e399..4ca3a1b5a5 100644
--- a/tests/auto/widgets/widgets/qcombobox/tst_qcombobox.cpp
+++ b/tests/auto/widgets/widgets/qcombobox/tst_qcombobox.cpp
@@ -146,6 +146,7 @@ private slots:
     void checkEmbeddedLineEditWhenStyleSheetIsSet();
     void propagateStyleChanges();
     void buttonPressKeys();
+    void clearModel();
 
 private:
     PlatformInputContext m_platformInputContext;
@@ -3593,5 +3594,37 @@ void tst_QComboBox::buttonPressKeys()
     }
 }
 
+void tst_QComboBox::clearModel()
+{
+    using namespace Qt::StringLiterals;
+    QStringListModel model({ "one"_L1, "two"_L1, "three"_L1 });
+
+    QComboBox combo;
+    combo.setModel(&model);
+    combo.setCurrentIndex(1);
+
+    QCOMPARE(combo.currentIndex(), 1);
+    QCOMPARE(combo.currentText(), model.index(1).data().toString());
+
+    QSignalSpy indexSpy(&combo, &QComboBox::currentIndexChanged);
+    QSignalSpy textSpy(&combo, &QComboBox::currentTextChanged);
+
+    QVERIFY(indexSpy.isEmpty());
+    QVERIFY(textSpy.isEmpty());
+
+    model.setStringList({});
+
+    QCOMPARE(indexSpy.count(), 1);
+    const int index = indexSpy.takeFirst().at(0).toInt();
+    QCOMPARE(index, -1);
+
+    QCOMPARE(textSpy.count(), 1);
+    const QString text = textSpy.takeFirst().at(0).toString();
+    QCOMPARE(text, QString());
+
+    QCOMPARE(combo.currentIndex(), -1);
+    QCOMPARE(combo.currentText(), QString());
+}
+
 QTEST_MAIN(tst_QComboBox)
 #include "tst_qcombobox.moc"
-- 
cgit v1.2.3