QML engine: Fix binding setup

We set QProperty bindings up in the wrong way: Parent components would
overwrite their child component's binding. This patch reverses the
order, fixing the bug.

Task-number: QTBUG-87153
Pick-to: 6.0
Change-Id: I3e90d1d14a41a7c5c337745f1453484d360a3979
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>

1 files changed, 14 insertions, 0 deletions

diff --git a/tests/auto/qml/qqmlecmascript/tst_qqmlecmascript.cpp b/tests/auto/qml/qqmlecmascript/tst_qqmlecmascript.cpp
index d7cb85a75d..37e3a33442 100644
--- a/tests/auto/qml/qqmlecmascript/tst_qqmlecmascript.cpp
+++ b/tests/auto/qml/qqmlecmascript/tst_qqmlecmascript.cpp
@@ -382,6 +382,7 @@ private slots:
     void semicolonAfterProperty();
     void hugeStack();
     void bindingOnQProperty();
+    void overwrittenBindingOnQProperty();
     void aliasOfQProperty();
     void bindingOnQPropertyContextProperty();
     void bindingContainingQProperty();
@@ -9182,6 +9183,19 @@ void tst_qqmlecmascript::bindingOnQProperty()
     QVERIFY(qprop.hasBinding());
 }
 
+void tst_qqmlecmascript::overwrittenBindingOnQProperty()
+{
+    QQmlEngine engine;
+    QQmlComponent component(&engine, testFileUrl("QPropertyOverwrite.qml"));
+    QVERIFY2(component.isReady(), qPrintable(component.errorString()));
+    QScopedPointer<QObject> root(component.create());
+    auto test = root->findChild<ClassWithQProperty *>("test");
+    QVERIFY(test);
+    QCOMPARE(test->value.value(), 13.f);
+    root->setProperty("value", 14.f);
+    QCOMPARE(test->value.value(), 14.0);
+}
+
 void tst_qqmlecmascript::aliasOfQProperty() {
     QQmlEngine engine;
     QQmlComponent component(&engine, testFileUrl("aliasOfQProperty.qml"));