diff options
| author | Ulf Hermann <ulf.hermann@qt.io> | 2021-02-12 14:35:48 +0100 |
|---|---|---|
| committer | Qt Cherry-pick Bot <cherrypick_bot@qt-project.org> | 2021-02-16 14:44:48 +0000 |
| commit | 04c16d7c1e06d6bfc33c61e7fda0c59e45c4daaa (patch) | |
| tree | 33e9d642de6db48065894cdb6be55ca7e7973acf /tests | |
| parent | c0775e9a9cfb9541fc61d788516cfc635338b4f8 (diff) | |
qmllint: Don't create properties for "on" assignments
"on" assignments are assignments to the default property, with the
property given interpreted as the target for the inner object.
Change-Id: Ia93a171f759964d2c00d6c0293a5434f588123af
Reviewed-by: Andrei Golubev <andrei.golubev@qt.io>
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
(cherry picked from commit 67c8afff346eae27c6fb833661d179326dd8b153)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/auto/qml/qmllint/data/onAssignment.qml | 7 | ||||
| -rw-r--r-- | tests/auto/qml/qmllint/tst_qmllint.cpp | 4 |
2 files changed, 11 insertions, 0 deletions
diff --git a/tests/auto/qml/qmllint/data/onAssignment.qml b/tests/auto/qml/qmllint/data/onAssignment.qml new file mode 100644 index 0000000000..b754a7f311 --- /dev/null +++ b/tests/auto/qml/qmllint/data/onAssignment.qml @@ -0,0 +1,7 @@ +import QtQuick + +Item { + property bool pressed: false + SequentialAnimation on pressed {} + property int wrong: pressed.loops +} diff --git a/tests/auto/qml/qmllint/tst_qmllint.cpp b/tests/auto/qml/qmllint/tst_qmllint.cpp index 18b5090dcf..12b8f5619f 100644 --- a/tests/auto/qml/qmllint/tst_qmllint.cpp +++ b/tests/auto/qml/qmllint/tst_qmllint.cpp @@ -292,6 +292,10 @@ void TestQmllint::dirtyQmlCode_data() << QStringLiteral("Signal handler for \"onSig\" has more formal parameters " "than the signal it handles.") << QString(); + QTest::newRow("OnAssignment") + << QStringLiteral("onAssignment.qml") + << QStringLiteral("Property \"loops\" not found on type \"bool\"") + << QString(); } void TestQmllint::dirtyQmlCode() |