news: Properly take care of unicode string splitting

haecker-felix · Nov 3, 2023 · e1dadab · e1dadab
1 parent 2746373
commit e1dadab
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/.gitignore b/.gitignore
@@ -19,3 +19,4 @@ de.haeckerfelix.Hebbot.json
 /project_template.md
 ./section_template.md
 /section_template.md
+.flatpak-builder
diff --git a/src/bot.rs b/src/bot.rs
@@ -586,7 +586,6 @@ impl Bot {
         let message = {
             let is_editor = self.is_editor(member).await;
             let mut news_store = self.news_store.lock().unwrap();
-            let redacted_event_id = redacted_event_id;
             let link = self.message_link(redacted_event_id);
 
             // Redaction / deletion of the news entry itself

diff --git a/src/news.rs b/src/news.rs
@@ -47,7 +47,15 @@ impl News {
 
     pub fn message_summary(&self) -> String {
         if self.message.borrow().len() > 60 {
-            format!("{} …", self.message.borrow().clone().split_at(50).0)
+            format!(
+                "{} …",
+                self.message
+                    .borrow()
+                    .clone()
+                    .chars()
+                    .take(50)
+                    .collect::<String>()
+            )
         } else {
             self.message.borrow().clone()
         }