diff options
author | Fabian Kosmale <fabian.kosmale@qt.io> | 2023-05-09 09:56:06 +0200 |
---|---|---|
committer | Fabian Kosmale <fabian.kosmale@qt.io> | 2023-05-09 21:28:07 +0200 |
commit | 57e272fbd2bb4af083c640e8721e5ecbe869e384 (patch) | |
tree | 6ec73c436f073657748fc4d27fdbe9feb676d1cc /tests/auto | |
parent | 71b4a5e6dfe07fe30c59b3e84c9db27f443a7b92 (diff) | |
download | qtdeclarative-57e272fbd2bb4af083c640e8721e5ecbe869e384.tar.gz |
Value types: Prefer assigning bindings instead of converting
When we're assigning a QJSValue to a value type property, we must first
check whether it is a binding created by Qt.binding. In that case, we
should directly set that binding up. Going through the conversion code
path might not necessarily fail, but might instead yield nonsense.
Pick-to: 6.5
Fixes: QTBUG-113472
Change-Id: If91e0843f0caf36c96b4c811b9ce8076adfc984f
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
Diffstat (limited to 'tests/auto')
-rw-r--r-- | tests/auto/qml/qqmlcomponent/data/createObjectWithScript.qml | 12 | ||||
-rw-r--r-- | tests/auto/qml/qqmlcomponent/tst_qqmlcomponent.cpp | 8 |
2 files changed, 19 insertions, 1 deletions
diff --git a/tests/auto/qml/qqmlcomponent/data/createObjectWithScript.qml b/tests/auto/qml/qqmlcomponent/data/createObjectWithScript.qml index 3391b3a266..3cc87afa16 100644 --- a/tests/auto/qml/qqmlcomponent/data/createObjectWithScript.qml +++ b/tests/auto/qml/qqmlcomponent/data/createObjectWithScript.qml @@ -11,6 +11,9 @@ Item{ property QtObject badRequired: null property QtObject goodRequired: null + property QtObject bindingAsInitial: null + property bool bindingUsed: false + Component{ id: a Rectangle { @@ -43,6 +46,11 @@ Item{ } } + Component { + id: e + Rectangle {} + } + Component.onCompleted: { root.declarativerectangle = a.createObject(root, {"x":17,"y":17, "color":"white", "border.width":3, "innerRect.border.width": 20}); root.declarativeitem = b.createObject(root, {"x":17,"y":17,"testBool":true,"testInt":17,"testObject":root}); @@ -52,5 +60,9 @@ Item{ root.badRequired = d.createObject(root, { "not_i": 42 }); root.goodRequired = d.createObject(root, { "i": 42 }); + + root.bindingAsInitial = e.createObject(root, {color: Qt.binding(() => { + root.bindingUsed = true; return '#ff0000' + })}); } } diff --git a/tests/auto/qml/qqmlcomponent/tst_qqmlcomponent.cpp b/tests/auto/qml/qqmlcomponent/tst_qqmlcomponent.cpp index 0af3db58fa..d6a14cd3ef 100644 --- a/tests/auto/qml/qqmlcomponent/tst_qqmlcomponent.cpp +++ b/tests/auto/qml/qqmlcomponent/tst_qqmlcomponent.cpp @@ -284,7 +284,7 @@ void tst_qqmlcomponent::qmlCreateObjectWithProperties() QTest::ignoreMessage( QtMsgType::QtWarningMsg, QRegularExpression( - ".*createObjectWithScript.qml:42:13: Required property i was not initialized")); + ".*createObjectWithScript.qml:45:13: Required property i was not initialized")); QQmlComponent component(&engine, testFileUrl("createObjectWithScript.qml")); QVERIFY2(component.errorString().isEmpty(), component.errorString().toUtf8()); @@ -344,6 +344,12 @@ void tst_qqmlcomponent::qmlCreateObjectWithProperties() QCOMPARE(goodRequired->parent(), object.data()); QCOMPARE(goodRequired->property("i").value<int>(), 42); } + + { + QScopedPointer<QObject> bindingAsInitial(object->property("bindingAsInitial").value<QObject *>()); + QVERIFY(bindingAsInitial); + QVERIFY(object->property("bindingUsed").toBool()); + } } void tst_qqmlcomponent::qmlCreateObjectClean() |