From d49c79d03286aaec04dd478dc527d2c51ee85acc Mon Sep 17 00:00:00 2001 From: Gabriele Date: Thu, 9 Apr 2015 13:13:25 +0200 Subject: [PATCH] WebUI: Use QVariantMap instead of QVariantHash Currently QJsonDocument::fromVariant() does not support QVariantHash, so use QVariantMap instead. Closes #2849. --- src/webui/btjson.cpp | 15 ++++----------- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/src/webui/btjson.cpp b/src/webui/btjson.cpp index 34bd0f986..049e2b226 100644 --- a/src/webui/btjson.cpp +++ b/src/webui/btjson.cpp @@ -157,7 +157,7 @@ static const char KEY_SUFFIX_REMOVED[] = "_removed"; QVariantMap getTranserInfoMap(); QVariantMap toMap(const QTorrentHandle& h); void processMap(QVariantMap prevData, QVariantMap data, QVariantMap &syncData); -void processHash(QVariantHash prevData, QVariantHash data, QVariantMap &syncData, QVariantList &removedItems); +void processHash(QVariantMap prevData, QVariantMap data, QVariantMap &syncData, QVariantList &removedItems); void processList(QVariantList prevData, QVariantList data, QVariantList &syncData, QVariantList &removedItems); QVariantMap generateSyncData(int acceptedResponseId, QVariantMap data, QVariantMap &lastAcceptedData, QVariantMap &lastData); @@ -314,7 +314,7 @@ QByteArray btjson::getSyncMainData(int acceptedResponseId, QVariantMap &lastData { QVariantMap data; - QVariantHash torrents; + QVariantMap torrents; std::vector torrentsList = QBtSession::instance()->getTorrents(); std::vector::const_iterator it = torrentsList.begin(); @@ -600,14 +600,7 @@ void processMap(QVariantMap prevData, QVariantMap data, QVariantMap &syncData) switch (data[key].type()) { case QVariant::Map: { QVariantMap map; - processMap(prevData[key].toMap(), data[key].toMap(), map); - if (!map.isEmpty()) - syncData[key] = map; - } - break; - case QVariant::Hash: { - QVariantMap map; - processHash(prevData[key].toHash(), data[key].toHash(), map, removedItems); + processHash(prevData[key].toMap(), data[key].toMap(), map, removedItems); if (!map.isEmpty()) syncData[key] = map; if (!removedItems.isEmpty()) @@ -643,7 +636,7 @@ void processMap(QVariantMap prevData, QVariantMap data, QVariantMap &syncData) // Compare two lists of structures (prevData, data) and calculate difference (syncData, removedItems). // Structures encoded as map. // Lists are encoded as hash table (indexed by structure key value) to improve ease of searching for removed items. -void processHash(QVariantHash prevData, QVariantHash data, QVariantMap &syncData, QVariantList &removedItems) +void processHash(QVariantMap prevData, QVariantMap data, QVariantMap &syncData, QVariantList &removedItems) { // initialize output variables syncData.clear();