This commit is contained in:
Nicolas Werner 2021-01-22 03:18:36 +01:00
parent 1a83a982fc
commit 1c98426536
2 changed files with 18 additions and 17 deletions

View file

@ -1235,11 +1235,11 @@ Cache::saveState(const mtx::responses::Sync &res)
updatedInfo.avatar_url = getRoomAvatarUrl(txn, statesdb, membersdb).toStdString(); updatedInfo.avatar_url = getRoomAvatarUrl(txn, statesdb, membersdb).toStdString();
updatedInfo.version = getRoomVersion(txn, statesdb).toStdString(); updatedInfo.version = getRoomVersion(txn, statesdb).toStdString();
bool has_new_tags = false;
// Process the account_data associated with this room // Process the account_data associated with this room
if (!room.second.account_data.events.empty()) { if (!room.second.account_data.events.empty()) {
auto accountDataDb = getAccountDataDb(txn, room.first); auto accountDataDb = getAccountDataDb(txn, room.first);
bool has_new_tags = false;
for (const auto &evt : room.second.account_data.events) { for (const auto &evt : room.second.account_data.events) {
std::visit( std::visit(
[&txn, &accountDataDb](const auto &event) { [&txn, &accountDataDb](const auto &event) {
@ -1266,21 +1266,21 @@ Cache::saveState(const mtx::responses::Sync &res)
nhlog::db()->debug("Fully read: {}", fr->content.event_id); nhlog::db()->debug("Fully read: {}", fr->content.event_id);
} }
} }
if (!has_new_tags) { }
// retrieve the old tags, they haven't changed if (!has_new_tags) {
lmdb::val data; // retrieve the old tags, they haven't changed
if (lmdb::dbi_get(txn, roomsDb_, lmdb::val(room.first), data)) { lmdb::val data;
try { if (lmdb::dbi_get(txn, roomsDb_, lmdb::val(room.first), data)) {
RoomInfo tmp = json::parse( try {
std::string_view(data.data(), data.size())); RoomInfo tmp =
updatedInfo.tags = tmp.tags; json::parse(std::string_view(data.data(), data.size()));
} catch (const json::exception &e) { updatedInfo.tags = tmp.tags;
nhlog::db()->warn( } catch (const json::exception &e) {
"failed to parse room info: room_id ({}), {}: {}", nhlog::db()->warn(
room.first, "failed to parse room info: room_id ({}), {}: {}",
std::string(data.data(), data.size()), room.first,
e.what()); std::string(data.data(), data.size()),
} e.what());
} }
} }
} }

View file

@ -145,11 +145,12 @@ RoomInfoListItem::init(QWidget *parent)
tagAction->setWhatsThis(tr("Adds or removes the specified tag.", tagAction->setWhatsThis(tr("Adds or removes the specified tag.",
"WhatsThis hint for tag menu actions")); "WhatsThis hint for tag menu actions"));
for (const auto &riTag : roomInfo.tags) for (const auto &riTag : roomInfo.tags) {
if (riTag == tag) { if (riTag == tag) {
tagAction->setChecked(true); tagAction->setChecked(true);
break; break;
} }
}
connect(tagAction, &QAction::triggered, this, [this, tag](bool checked) { connect(tagAction, &QAction::triggered, this, [this, tag](bool checked) {
if (checked) if (checked)