conversation_module: don't update removed conversations

When removing a contact, we need to set the "removed" timestamp for our
active conversations with that contact, but we shouldn't update the
timestamp for conversations that had already been removed in the past.

GitLab: #1048
Change-Id: Ic090775625fa8fb557e3081ec408ee10bf01aaff

src/jamidht/conversation_module.cpp

@@ -2859,12 +2859,14 @@ ConversationModule::removeContact(const std::string& uri, bool banned)
     auto removeConvInfo = [&](const auto& conv, const auto& members) {
         if ((isSelf && members.size() == 1)
             || (!isSelf && std::find(members.begin(), members.end(), uri) != members.end())) {
-            // Mark as removed
+            // Mark the conversation as removed if it wasn't already
+            if (!conv->info.isRemoved()) {
                 conv->info.removed = std::time(nullptr);
                 updateClient(conv->info.id);
                 pimpl_->addConvInfo(conv->info);
                 return true;
             }
+        }
         return false;
     };
     {