diff options
author | Ulf Hermann <ulf.hermann@qt.io> | 2020-06-15 15:48:40 +0200 |
---|---|---|
committer | Ulf Hermann <ulf.hermann@qt.io> | 2020-06-22 09:10:18 +0200 |
commit | 8a3f8595569a920cf2fa811704dec97ae31be15d (patch) | |
tree | 84496eca2ae5330492ae54c7cf08531471127e3f /src/qml | |
parent | 64fb25a2acada6d9f0897b7e8db4583de4d8d2be (diff) |
Revert "Decode directory separators in source URLs"
This reverts commit 7ec30c51b287159377761338fe6d3b48706d74ee.
We don't want to half-decode directory separators on assignment. This
just introduces inconsistency down the line.
[ChangeLog][QtQml][Important Behavior Changes] Percent-encoded directory
separators in URLs are not automatically decoded on assignment to url
properties anymore. This was obviously not a good idea to begin with.
Fixes: QTBUG-81244
Change-Id: I1938abbe8aada88beff0d628397674255e8b2472
Reviewed-by: Andy Shaw <andy.shaw@qt.io>
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Diffstat (limited to 'src/qml')
-rw-r--r-- | src/qml/doc/src/qmllanguageref/typesystem/basictypes.qdoc | 4 | ||||
-rw-r--r-- | src/qml/qml/qqmlobjectcreator.cpp | 7 | ||||
-rw-r--r-- | src/qml/qml/qqmlproperty.cpp | 19 |
3 files changed, 10 insertions, 20 deletions
diff --git a/src/qml/doc/src/qmllanguageref/typesystem/basictypes.qdoc b/src/qml/doc/src/qmllanguageref/typesystem/basictypes.qdoc index 1040baedc0..f35a56858b 100644 --- a/src/qml/doc/src/qmllanguageref/typesystem/basictypes.qdoc +++ b/src/qml/doc/src/qmllanguageref/typesystem/basictypes.qdoc @@ -295,9 +295,7 @@ property is only invoked when the property is reassigned to a different object v Additionally, URLs may contain encoded characters using the 'percent-encoding' scheme specified by \l {http://tools.ietf.org/html/rfc3986}{RFC 3986}. These characters will be preserved within properties of type \c url, to allow QML code to - construct precise URL values. An exception to this rule is the preemptive - decoding of directory-separator characters (\c '/') - these characters are decoded - to allow the URL to be correctly classified. + construct precise URL values. For example, a local file containing a '#' character, which would normally be interpreted as the beginning of the URL 'fragment' element, can be accessed by diff --git a/src/qml/qml/qqmlobjectcreator.cpp b/src/qml/qml/qqmlobjectcreator.cpp index cbf243b9c1..61d287fc88 100644 --- a/src/qml/qml/qqmlobjectcreator.cpp +++ b/src/qml/qml/qqmlobjectcreator.cpp @@ -451,10 +451,9 @@ void QQmlObjectCreator::setPropertyValue(const QQmlPropertyData *property, const break; case QMetaType::QUrl: { assertType(QV4::CompiledData::Binding::Type_String); - QString string = compilationUnit->bindingValueAsString(binding); - // Encoded dir-separators defeat QUrl processing - decode them first - string.replace(QLatin1String("%2f"), QLatin1String("/"), Qt::CaseInsensitive); - QUrl value = string.isEmpty() ? QUrl() : compilationUnit->finalUrl().resolved(QUrl(string)); + const QString string = compilationUnit->bindingValueAsString(binding); + QUrl value = string.isEmpty() ? QUrl() + : compilationUnit->finalUrl().resolved(QUrl(string)); // Apply URL interceptor value = engine->interceptUrl(value, QQmlAbstractUrlInterceptor::UrlString); property->writeProperty(_qobject, &value, propertyWriteFlags); diff --git a/src/qml/qml/qqmlproperty.cpp b/src/qml/qml/qqmlproperty.cpp index 864f97a417..adfa76ea24 100644 --- a/src/qml/qml/qqmlproperty.cpp +++ b/src/qml/qml/qqmlproperty.cpp @@ -1309,21 +1309,14 @@ bool QQmlPropertyPrivate::write( return property.writeProperty(object, const_cast<QVariant *>(&value), flags); } else if (isUrl) { QUrl u; - if (variantType == QMetaType::QUrl) { + if (variantType == QMetaType::QUrl) u = value.toUrl(); - } else if (variantType == QMetaType::QByteArray) { - QString input(QString::fromUtf8(value.toByteArray())); - // Encoded dir-separators defeat QUrl processing - decode them first - input.replace(QLatin1String("%2f"), QLatin1String("/"), Qt::CaseInsensitive); - u = QUrl(input); - } else if (variantType == QMetaType::QString) { - QString input(value.toString()); - // Encoded dir-separators defeat QUrl processing - decode them first - input.replace(QLatin1String("%2f"), QLatin1String("/"), Qt::CaseInsensitive); - u = QUrl(input); - } else { + else if (variantType == QMetaType::QByteArray) + u = QUrl(QString::fromUtf8(value.toByteArray())); + else if (variantType == QMetaType::QString) + u = QUrl(value.toString()); + else return false; - } if (context && u.isRelative() && !u.isEmpty()) u = context->resolvedUrl(u); |