From aabf4e4f0474d8d6eb065fbab700432718070ae0 Mon Sep 17 00:00:00 2001
From: Simon Hausmann <simon.hausmann@qt.io>
Date: Mon, 22 Oct 2018 10:59:13 +0200
Subject: Fix lookup of methods in the scope object

Commit 939014cb9cad2f3357f47b28a4580397c17b913c improved performance of
property lookups beyond the scope object, with the unfortunate
side-effect that the previously polymorphic lookup of methods broke.

Fix this by moving the handling to the caller side and falling back to
the string lookup for functions.

Fixes: QTBUG-71204
Change-Id: I2d9924034a9c14e7d161fa49d51b1f876ab5bc0f
Reviewed-by: Lars Knoll <lars.knoll@qt.io>
---
 .../qml/qqmllanguage/data/polymorphicFunctionLookup.qml    | 14 ++++++++++++++
 tests/auto/qml/qqmllanguage/tst_qqmllanguage.cpp           | 13 +++++++++++++
 2 files changed, 27 insertions(+)
 create mode 100644 tests/auto/qml/qqmllanguage/data/polymorphicFunctionLookup.qml

(limited to 'tests/auto/qml/qqmllanguage')

diff --git a/tests/auto/qml/qqmllanguage/data/polymorphicFunctionLookup.qml b/tests/auto/qml/qqmllanguage/data/polymorphicFunctionLookup.qml
new file mode 100644
index 0000000000..4a3cc52793
--- /dev/null
+++ b/tests/auto/qml/qqmllanguage/data/polymorphicFunctionLookup.qml
@@ -0,0 +1,14 @@
+import QtQml 2.0
+QtObject {
+    id: root
+    property bool testFunc;
+    property bool ok: false
+    property QtObject subObject: QtObject {
+        function testFunc()
+        {
+            root.ok = true
+        }
+
+        Component.onCompleted: testFunc()
+    }
+}
diff --git a/tests/auto/qml/qqmllanguage/tst_qqmllanguage.cpp b/tests/auto/qml/qqmllanguage/tst_qqmllanguage.cpp
index 157fd500ad..7a8de739f4 100644
--- a/tests/auto/qml/qqmllanguage/tst_qqmllanguage.cpp
+++ b/tests/auto/qml/qqmllanguage/tst_qqmllanguage.cpp
@@ -298,6 +298,8 @@ private slots:
 
     void retrieveQmlTypeId();
 
+    void polymorphicFunctionLookup();
+
 private:
     QQmlEngine engine;
     QStringList defaultImportPathList;
@@ -5049,6 +5051,17 @@ void tst_qqmllanguage::retrieveQmlTypeId()
     QVERIFY(qmlTypeId("Test", 1, 0, "MyTypeObjectSingleton") >= 0);
 }
 
+void tst_qqmllanguage::polymorphicFunctionLookup()
+{
+    QQmlEngine engine;
+    QQmlComponent component(&engine, testFileUrl("polymorphicFunctionLookup.qml"));
+    VERIFY_ERRORS(0);
+    QScopedPointer<QObject> o(component.create());
+    QVERIFY(!o.isNull());
+
+    QVERIFY(o->property("ok").toBool());
+}
+
 QTEST_MAIN(tst_qqmllanguage)
 
 #include "tst_qqmllanguage.moc"
-- 
cgit v1.2.3