diff options
| author | Kitsune Ral <Kitsune-Ral@users.sf.net> | 2018-11-23 15:38:06 +0900 |
|---|---|---|
| committer | Kitsune Ral <Kitsune-Ral@users.sf.net> | 2018-11-23 15:55:16 +0900 |
| commit | 64799eaf667840c7f81d80810508d948f64f97d6 (patch) | |
| tree | fa16d293f20e5fc545b9711dfe8916ff64b854b9 /lib/syncdata.cpp | |
| parent | 53f3fe79ef91bb5ba318b61b3a073c12409abc72 (diff) | |
| download | libquotient-64799eaf667840c7f81d80810508d948f64f97d6.tar.gz libquotient-64799eaf667840c7f81d80810508d948f64f97d6.zip | |
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).
Diffstat (limited to 'lib/syncdata.cpp')
| -rw-r--r-- | lib/syncdata.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
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<size_t>(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)," |