diff options
| author | Ulf Hermann <ulf.hermann@qt.io> | 2023-06-19 09:29:34 +0200 |
|---|---|---|
| committer | Ulf Hermann <ulf.hermann@qt.io> | 2023-06-30 10:47:17 +0200 |
| commit | b9bfdea0e2c6721d2306af0ecc44f88da9988957 (patch) | |
| tree | 2cfc6b8f9b43a221d0cdb4c92d0bd868696ab952 /src/qml/jsruntime/qv4qobjectwrapper.cpp | |
| parent | 975a6bff84815f536abf1324394193b8180edeaa (diff) | |
QML: Un-specialcase QStringList and QVariantList conversion
Those are just regular sequences these days. They can be written back.
Drop some now-dead code and deduplicate the value type conversion code
in the process. We should try the (more common) value type conversion
before the sequence conversion, but after all the "simple" conversions.
[ChangeLog][QtQml][Important Behavior Changes] Converting a QVariantList
to a QJSValue via, for example QJSEngine::toScriptValue() does not
produce a JavaScript array anymore, but rather a better suited sequence
object that behaves almost like a JavaScript array. The only difference
is that its QJSValue::isArray() will return false now.
Fixes: QTBUG-113690
Change-Id: Ib176c34d59c45a6b5cff68d029c4b1b87d7aa192
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Diffstat (limited to 'src/qml/jsruntime/qv4qobjectwrapper.cpp')
| -rw-r--r-- | src/qml/jsruntime/qv4qobjectwrapper.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/qml/jsruntime/qv4qobjectwrapper.cpp b/src/qml/jsruntime/qv4qobjectwrapper.cpp index 6a0142a46e..38b9226c9f 100644 --- a/src/qml/jsruntime/qv4qobjectwrapper.cpp +++ b/src/qml/jsruntime/qv4qobjectwrapper.cpp @@ -205,12 +205,12 @@ static ReturnedValue loadProperty( // See if it's a sequence type. // Pass nullptr as data. It's lazy-loaded. - QV4::ScopedValue retn(scope, QV4::SequencePrototype::newSequence( - v4, propMetaType, nullptr, - wrapper, property.coreIndex(), - referenceFlags(scope.engine, property))); - if (!retn->isUndefined()) - return retn->asReturnedValue(); + const QQmlType qmlType = QQmlMetaType::qmlListType(propMetaType); + if (qmlType.isSequentialContainer()) { + return QV4::SequencePrototype::newSequence( + v4, propMetaType, qmlType.listMetaSequence(), nullptr, + wrapper, property.coreIndex(), referenceFlags(scope.engine, property)); + } QVariant v(propMetaType); property.readProperty(object, v.data()); |