From 8a3ede5ba1daf2d9314d71db972175b2791456c0 Mon Sep 17 00:00:00 2001
From: Maximilian Goldstein <max.goldstein@qt.io>
Date: Fri, 8 Jan 2021 11:58:55 +0100
Subject: Fix warning and assert when removing items from groups

When removing items from groups actively shown by a model filter a warning
and assert was triggered in the past. This change fixes this behavior.

Fixes: QTBUG-86017
Change-Id: I49b7498a3d03141b654e453a3c35a43fc7ba804a
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
---
 .../qml/qqmldelegatemodel/data/qtbug_86017.qml     | 32 ++++++++++++++++++++++
 .../qqmldelegatemodel/tst_qqmldelegatemodel.cpp    | 16 +++++++++++
 2 files changed, 48 insertions(+)
 create mode 100644 tests/auto/qml/qqmldelegatemodel/data/qtbug_86017.qml

(limited to 'tests')

diff --git a/tests/auto/qml/qqmldelegatemodel/data/qtbug_86017.qml b/tests/auto/qml/qqmldelegatemodel/data/qtbug_86017.qml
new file mode 100644
index 0000000000..02d737e37f
--- /dev/null
+++ b/tests/auto/qml/qqmldelegatemodel/data/qtbug_86017.qml
@@ -0,0 +1,32 @@
+import QtQuick 2.8
+import QtQml.Models 2.1
+
+DelegateModel {
+    id: visualModel
+    model: ListModel {
+        id: myLM
+        ListElement {
+            name: "Apple"
+        }
+        ListElement {
+            name: "Orange"
+        }
+    }
+
+    filterOnGroup: "selected"
+
+    groups: [
+        DelegateModelGroup {
+            name: "selected"
+            includeByDefault: true
+        }
+    ]
+
+    delegate: Text {
+        Component.onCompleted: {
+            DelegateModel.inPersistedItems = true
+            DelegateModel.inSelected = false
+        }
+        text: "item " + index
+    }
+}
diff --git a/tests/auto/qml/qqmldelegatemodel/tst_qqmldelegatemodel.cpp b/tests/auto/qml/qqmldelegatemodel/tst_qqmldelegatemodel.cpp
index 87f42c0c8a..8e6d147028 100644
--- a/tests/auto/qml/qqmldelegatemodel/tst_qqmldelegatemodel.cpp
+++ b/tests/auto/qml/qqmldelegatemodel/tst_qqmldelegatemodel.cpp
@@ -42,6 +42,7 @@ public:
 private slots:
     void valueWithoutCallingObjectFirst_data();
     void valueWithoutCallingObjectFirst();
+    void qtbug_86017();
 };
 
 class AbstractItemModel : public QAbstractItemModel
@@ -134,6 +135,21 @@ void tst_QQmlDelegateModel::valueWithoutCallingObjectFirst()
     QCOMPARE(model->variantValue(index, role), expectedValue);
 }
 
+void tst_QQmlDelegateModel::qtbug_86017()
+{
+    QQmlEngine engine;
+    QQmlComponent component(&engine);
+    component.loadUrl(testFileUrl("qtbug_86017.qml"));
+    QScopedPointer<QObject> root(component.create());
+    QVERIFY2(root, qPrintable(component.errorString()));
+    QTRY_VERIFY(component.isReady());
+    QQmlDelegateModel *model = qobject_cast<QQmlDelegateModel*>(root.data());
+
+    QVERIFY(model);
+    QCOMPARE(model->count(), 2);
+    QCOMPARE(model->filterGroup(), "selected");
+}
+
 QTEST_MAIN(tst_QQmlDelegateModel)
 
 #include "tst_qqmldelegatemodel.moc"
-- 
cgit v1.2.3