From 64799eaf667840c7f81d80810508d948f64f97d6 Mon Sep 17 00:00:00 2001 From: Kitsune Ral Date: Fri, 23 Nov 2018 15:38:06 +0900 Subject: Connection::saveState: use null instead of an empty object for a room placeholder Otherwise placeholder objects are confused with normal room JSON objects when loading from the cache. Closes #257 (again). --- lib/connection.cpp | 11 ++--------- lib/syncdata.cpp | 8 +++++--- 2 files changed, 7 insertions(+), 12 deletions(-) (limited to 'lib') diff --git a/lib/connection.cpp b/lib/connection.cpp index 7feeb075..53835a80 100644 --- a/lib/connection.cpp +++ b/lib/connection.cpp @@ -1104,15 +1104,8 @@ void Connection::saveState() const QJsonObject rooms; QJsonObject inviteRooms; for (const auto* i : roomMap()) // Pass on rooms in Leave state - { - auto& targetArray = i->joinState() == JoinState::Invite - ? inviteRooms : rooms; - targetArray.insert(i->id(), QJsonObject()); - QElapsedTimer et1; et1.start(); - QCoreApplication::processEvents(); - if (et1.elapsed() > 1) - qCDebug(PROFILER) << "processEvents() borrowed" << et1; - } + (i->joinState() == JoinState::Invite ? inviteRooms : rooms) + .insert(i->id(), QJsonValue::Null); QJsonObject roomObj; if (!rooms.isEmpty()) diff --git a/lib/syncdata.cpp b/lib/syncdata.cpp index d141a7cc..1023ed6a 100644 --- a/lib/syncdata.cpp +++ b/lib/syncdata.cpp @@ -156,9 +156,9 @@ void SyncData::parseJson(const QJsonObject& json, const QString& baseDir) roomData.reserve(static_cast(rs.size())); for(auto roomIt = rs.begin(); roomIt != rs.end(); ++roomIt) { - auto roomJson = roomIt->isString() - ? loadJson(baseDir + fileNameForRoom(roomIt.key())) - : roomIt->toObject(); + auto roomJson = roomIt->isObject() + ? roomIt->toObject() + : loadJson(baseDir + fileNameForRoom(roomIt.key())); if (roomJson.isEmpty()) { unresolvedRoomIds.push_back(roomIt.key()); @@ -171,6 +171,8 @@ void SyncData::parseJson(const QJsonObject& json, const QString& baseDir) } totalRooms += rs.size(); } + if (!unresolvedRoomIds.empty()) + qCWarning(MAIN) << "Unresolved rooms:" << unresolvedRoomIds.join(','); if (totalRooms > 9 || et.nsecsElapsed() >= profilerMinNsecs()) qCDebug(PROFILER) << "*** SyncData::parseJson(): batch with" << totalRooms << "room(s)," -- cgit v1.2.3