From 40f53455869d2340ab8d7aba3d1a6bed9bcbde7e Mon Sep 17 00:00:00 2001
From: strawberry <strawberry@puppygock.gay>
Date: Wed, 17 Jan 2024 13:16:00 -0500
Subject: [PATCH] fix formatting and links of user reports more

Signed-off-by: strawberry <strawberry@puppygock.gay>
---
 src/api/client_server/report.rs | 54 ++++++++++++++++++---------------
 1 file changed, 30 insertions(+), 24 deletions(-)

diff --git a/src/api/client_server/report.rs b/src/api/client_server/report.rs
index 85e3f5430..343d9a28b 100644
--- a/src/api/client_server/report.rs
+++ b/src/api/client_server/report.rs
@@ -72,32 +72,38 @@ pub async fn report_event_route(
     };
 
     // send admin room message that we received the report with an @room ping for urgency
-    services().admin
+    services()
+        .admin
         .send_message(message::RoomMessageEventContent::text_html(
-            format!(
-                "@room Report received from: {}\n\n\
-                Event ID: {:?}\n\
-                Room ID: {:?}\n\
-                Sent By: {:?}\n\n\
-                Report Score: {:#?}\n\
-                Report Reason: {:?}",
-                sender_user.to_owned(), pdu.event_id, pdu.room_id, pdu.sender, body.score, body.reason
-            ),
-            format!(
-                "<details><summary>@room Report received from: <a href=\"https://matrix.to/#/{0}\">{0}\
-                </a></summary><ul><li>Event Info<ul><li>Event ID: <code>{1:?}</code>\
-                <a href=\"https://matrix.to/#/{2:?}/{1:?}\">🔗</a></li><li>Room ID: <code>{2:?}</code>\
-                </li><li>Sent By: <a href=\"https://matrix.to/#/{3:?}\">{3:?}</a></li></ul></li><li>\
-                Report Info<ul><li>Report Score: {4:?}</li><li>Report Reason: {5}</li></ul></li>\
+        format!(
+            "@room Report received from: {}\n\n\
+                Event ID: {}\n\
+                Room ID: {}\n\
+                Sent By: {}\n\n\
+                Report Score: {}\n\
+                Report Reason: {}",
+            sender_user.to_owned(),
+            pdu.event_id,
+            pdu.room_id,
+            pdu.sender.to_owned(),
+            body.score.unwrap_or(ruma::Int::from(0)),
+            body.reason.as_deref().unwrap_or("")
+        ),
+        format!(
+            "<details><summary>@room Report received from: <a href=\"https://matrix.to/#/{0}\">{0}\
+                </a></summary><ul><li>Event Info<ul><li>Event ID: <code>{1}</code>\
+                <a href=\"https://matrix.to/#/{2}/{1}\">🔗</a></li><li>Room ID: <code>{2}</code>\
+                </li><li>Sent By: <a href=\"https://matrix.to/#/{3}\">{3}</a></li></ul></li><li>\
+                Report Info<ul><li>Report Score: {4}</li><li>Report Reason: {5}</li></ul></li>\
                 </ul></details>",
-                sender_user.to_owned(),
-                pdu.event_id,
-                pdu.room_id,
-                pdu.sender,
-                body.score,
-                HtmlEscape(body.reason.as_deref().unwrap_or(""))
-            ),
-        ));
+            sender_user.to_owned(),
+            pdu.event_id.to_owned(),
+            pdu.room_id.to_owned(),
+            pdu.sender.to_owned(),
+            body.score.unwrap_or(ruma::Int::from(0)),
+            HtmlEscape(body.reason.as_deref().unwrap_or(""))
+        ),
+    ));
 
     // even though this is kinda security by obscurity, let's still make a small random delay sending a successful response
     // per spec suggestion regarding enumerating for potential events existing in our server.
-- 
GitLab