Fix QQuickContainerPrivate::itemSiblingOrderChanged()

Repeaters are not part of the content model. Exclude them when
calculating the target index when the order of items changes.

Task-number: QTBUG-61310
Change-Id: Iaedd59288ed38e985a34ed8e1f515fdfb50d74e6
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>

1 files changed, 27 insertions, 0 deletions

diff --git a/tests/auto/controls/data/tst_container.qml b/tests/auto/controls/data/tst_container.qml
index 049982fb..b8de2b08 100644
--- a/tests/auto/controls/data/tst_container.qml
+++ b/tests/auto/controls/data/tst_container.qml
@@ -126,4 +126,31 @@ TestCase {
         compare(control1.currentIndex, 1)
         compare(control2.currentIndex, 1)
     }
+
+    Component {
+        id: repeaterContainer
+        Container {
+            id: container
+            Item { objectName: "0" }
+            Item { objectName: "1" }
+            Item { objectName: "2" }
+            Item { objectName: "3" }
+            contentItem: Row {
+                Repeater {
+                    model: container.contentModel
+                }
+            }
+        }
+    }
+
+    // don't crash (QTBUG-61310)
+    function test_repeater() {
+        var control = createTemporaryObject(repeaterContainer)
+        verify(control)
+
+        compare(control.itemAt(0).objectName, "0")
+        compare(control.itemAt(1).objectName, "1")
+        compare(control.itemAt(2).objectName, "2")
+        compare(control.itemAt(3).objectName, "3")
+    }
 }