From 8c0dab650dae8a88712313525cccf01abd3bf24f Mon Sep 17 00:00:00 2001 From: Ivan Solovev Date: Fri, 23 Apr 2021 10:24:13 +0200 Subject: [PATCH] QSFPM: fix filterCaseSensitivityChanged signal emission logic This patch amends bcbbbdb2d640c059c19e9337c7418b83b1b7e4ea. It fixes the logic of filterCaseSensitivityChanged signal emission. The call to QRegularExpression overload of setFilterRegularExpression could change the filterCaseSensitivity, but the signal was never emitted. [ChangeLog][QtCore][QSortFilterProxyModel] A call to QRegularExpression overload of setFilterRegularExpression now emits a filterCaseSensitivityChanged signal, if required. Change-Id: Id4ef04227c1f8ed98153fa5107ec3fbe4c0c77fb Reviewed-by: Samuel Gaist Reviewed-by: Fabian Kosmale (cherry picked from commit 0d76a5cd2cdc37f9fb4cbd62af4ab6102a0acd7d) Reviewed-by: Qt Cherry-pick Bot --- .../itemmodels/qsortfilterproxymodel.cpp | 4 ++++ ...sortfilterproxymodel_regularexpression.cpp | 19 +++++++++++++++++++ 2 files changed, 23 insertions(+) diff --git a/src/corelib/itemmodels/qsortfilterproxymodel.cpp b/src/corelib/itemmodels/qsortfilterproxymodel.cpp index 23ade517c7c..a232adbe45a 100644 --- a/src/corelib/itemmodels/qsortfilterproxymodel.cpp +++ b/src/corelib/itemmodels/qsortfilterproxymodel.cpp @@ -2524,9 +2524,13 @@ QRegularExpression QSortFilterProxyModel::filterRegularExpression() const void QSortFilterProxyModel::setFilterRegularExpression(const QRegularExpression ®ularExpression) { Q_D(QSortFilterProxyModel); + const Qt::CaseSensitivity cs = filterCaseSensitivity(); d->filter_about_to_be_changed(); d->filter_regularexpression = regularExpression; d->filter_changed(QSortFilterProxyModelPrivate::Direction::Rows); + const Qt::CaseSensitivity updatedCs = filterCaseSensitivity(); + if (cs != updatedCs) + emit filterCaseSensitivityChanged(updatedCs); } #endif diff --git a/tests/auto/corelib/itemmodels/qsortfilterproxymodel_regularexpression/tst_qsortfilterproxymodel_regularexpression.cpp b/tests/auto/corelib/itemmodels/qsortfilterproxymodel_regularexpression/tst_qsortfilterproxymodel_regularexpression.cpp index 9d7ba3ca84e..8411d40a721 100644 --- a/tests/auto/corelib/itemmodels/qsortfilterproxymodel_regularexpression/tst_qsortfilterproxymodel_regularexpression.cpp +++ b/tests/auto/corelib/itemmodels/qsortfilterproxymodel_regularexpression/tst_qsortfilterproxymodel_regularexpression.cpp @@ -27,6 +27,7 @@ ****************************************************************************/ #include +#include #include #include "tst_qsortfilterproxymodel.h" @@ -41,6 +42,7 @@ private slots: void tst_caseSensitivity(); void tst_keepCaseSensitivity_QTBUG_92260(); void tst_keepPatternOptions_QTBUG_92260(); + void tst_regexCaseSensitivityNotification(); }; tst_QSortFilterProxyModelRegularExpression::tst_QSortFilterProxyModelRegularExpression() : @@ -140,5 +142,22 @@ void tst_QSortFilterProxyModelRegularExpression::tst_keepPatternOptions_QTBUG_92 QRegularExpression::MultilineOption); } +/*! + This test ensures that if the case sensitivity is changed during a call to + setFilterRegularExpression, the notification signal will be emitted +*/ +void tst_QSortFilterProxyModelRegularExpression::tst_regexCaseSensitivityNotification() +{ + QSortFilterProxyModel proxy; + QSignalSpy spy(&proxy, &QSortFilterProxyModel::filterCaseSensitivityChanged); + proxy.setFilterCaseSensitivity(Qt::CaseInsensitive); + QCOMPARE(spy.count(), 1); + QRegularExpression re("regex"); + QVERIFY(!re.patternOptions().testFlag(QRegularExpression::CaseInsensitiveOption)); + proxy.setFilterRegularExpression(re); + QCOMPARE(proxy.filterCaseSensitivity(), Qt::CaseSensitive); + QCOMPARE(spy.count(), 2); +} + QTEST_MAIN(tst_QSortFilterProxyModelRegularExpression) #include "tst_qsortfilterproxymodel_regularexpression.moc"