From 16b27700f5c7a872b6a8e91f4ecaa05c5b3c64ba Mon Sep 17 00:00:00 2001
From: Carl Schwan <carl@carlschwan.eu>
Date: Fri, 6 Sep 2024 16:16:09 +0200
Subject: [PATCH] Unify redaction handling

The code in messageeventmodel was calling to EventHandler::getBody who
already handles reaction handlings
---
 src/eventhandler.cpp               | 2 +-
 src/models/messagecontentmodel.cpp | 5 -----
 src/models/messageeventmodel.cpp   | 5 -----
 3 files changed, 1 insertion(+), 11 deletions(-)

diff --git a/src/eventhandler.cpp b/src/eventhandler.cpp
index 5019ed90e..fd912c7c4 100644
--- a/src/eventhandler.cpp
+++ b/src/eventhandler.cpp
@@ -277,7 +277,7 @@ QString EventHandler::getBody(const NeoChatRoom *room, const Quotient::RoomEvent
 {
     if (event->isRedacted()) {
         auto reason = event->redactedBecause()->reason();
-        return (reason.isEmpty()) ? i18n("<i>[This message was deleted]</i>") : i18n("<i>[This message was deleted: %1]</i>", reason);
+        return (reason.isEmpty()) ? i18n("<i>[This message was deleted]</i>") : i18n("<i>[This message was deleted: %1]</i>", reason.toHtmlEscaped());
     }
 
     const bool prettyPrint = (format == Qt::RichText);
diff --git a/src/models/messagecontentmodel.cpp b/src/models/messagecontentmodel.cpp
index 643c37f4f..0009fdda7 100644
--- a/src/models/messagecontentmodel.cpp
+++ b/src/models/messagecontentmodel.cpp
@@ -271,11 +271,6 @@ QVariant MessageContentModel::data(const QModelIndex &index, int role) const
         if (m_event == nullptr) {
             return QString();
         }
-        if (m_event->isRedacted()) {
-            auto reason = m_event->redactedBecause()->reason();
-            return (reason.isEmpty()) ? i18n("<i>[This message was deleted]</i>")
-                                      : i18n("<i>[This message was deleted: %1]</i>", m_event->redactedBecause()->reason());
-        }
         if (!component.content.isEmpty()) {
             return component.content;
         }
diff --git a/src/models/messageeventmodel.cpp b/src/models/messageeventmodel.cpp
index 9efc0540a..f90170c9e 100644
--- a/src/models/messageeventmodel.cpp
+++ b/src/models/messageeventmodel.cpp
@@ -433,11 +433,6 @@ QVariant MessageEventModel::data(const QModelIndex &idx, int role) const
     const auto &evt = isPending ? **pendingIt : **timelineIt;
 
     if (role == Qt::DisplayRole) {
-        if (evt.isRedacted()) {
-            auto reason = evt.redactedBecause()->reason();
-            return (reason.isEmpty()) ? i18n("<i>[This message was deleted]</i>")
-                                      : i18n("<i>[This message was deleted: %1]</i>", evt.redactedBecause()->reason().toHtmlEscaped());
-        }
         return EventHandler::richBody(m_currentRoom, &evt);
     }