Convert QObjects in QVariantMaps

QObjects that are present in an array are already converted to
an object identifier.  This does the same for variant maps, which
end up as ECMAScript objects.

This allows QObjects put into a QVariantMap to be properly
deserialized.  For example, if a property is declared as such:

Q_PROPERTY(QVariantMap propName READ propName CONSTANT)

And propName is:

QVariantMap propName() const {
  QVariantMap map;
  map.insert("theProperty", QVariant::fromValue(someQObject));
  return map;
}

The "theProperty" property will now properly refer to the object.

Change-Id: I3c6e71b860f6825a31eb337aeffa55302287c8ff
Reviewed-by: Milian Wolff <milian.wolff@kdab.com>

2 files changed, 20 insertions, 0 deletions

diff --git a/src/webchannel/qmetaobjectpublisher.cpp b/src/webchannel/qmetaobjectpublisher.cpp
index cd2d940..f1ccbe9 100644
--- a/src/webchannel/qmetaobjectpublisher.cpp
+++ b/src/webchannel/qmetaobjectpublisher.cpp
@@ -617,6 +617,9 @@ QJsonValue QMetaObjectPublisher::wrapResult(const QVariant &result, QWebChannelA
     } else if (result.canConvert<QVariantList>()) {
         // recurse and potentially wrap contents of the array
         return wrapList(result.toList(), transport);
+    } else if (result.canConvert<QVariantMap>()) {
+        // recurse and potentially wrap contents of the map
+        return wrapMap(result.toMap(), transport);
     }
 
     return QJsonValue::fromVariant(result);
@@ -631,6 +634,15 @@ QJsonArray QMetaObjectPublisher::wrapList(const QVariantList &list, QWebChannelA
     return array;
 }
 
+QJsonObject QMetaObjectPublisher::wrapMap(const QVariantMap &map, QWebChannelAbstractTransport *transport, const QString &parentObjectId)
+{
+    QJsonObject obj;
+    for (QVariantMap::const_iterator i = map.begin(); i != map.end(); i++) {
+        obj.insert(i.key(), wrapResult(i.value(), transport, parentObjectId));
+    }
+    return obj;
+}
+
 void QMetaObjectPublisher::deleteWrappedObject(QObject *object) const
 {
     if (!wrappedObjects.contains(registeredObjectIds.value(object))) {
diff --git a/src/webchannel/qmetaobjectpublisher_p.h b/src/webchannel/qmetaobjectpublisher_p.h
index d02a933..1535a70 100644
--- a/src/webchannel/qmetaobjectpublisher_p.h
+++ b/src/webchannel/qmetaobjectpublisher_p.h
@@ -199,6 +199,14 @@ public:
                           const QString &parentObjectId = QString());
 
     /**
+     * Convert a variant map for consumption by the client.
+     *
+     * This properly handles QML values and also wraps the result if required.
+     */
+    QJsonObject wrapMap(const QVariantMap &map, QWebChannelAbstractTransport *transport,
+                          const QString &parentObjectId = QString());
+
+    /**
      * Invoke delete later on @p object.
      */
     void deleteWrappedObject(QObject *object) const;