Fix: Dont overwrite equal drafts

src/chat.rs

@@ -922,6 +922,25 @@ impl ChatId {
                     && old_draft.chat_id == self
                     && old_draft.state == MessageState::OutDraft
                 {
+                    // Do not overwrite draft if text and file are the same
+                    if old_draft.text == msg.text {
+                        if msg.param.get(Param::File).is_some() {
+                            let blob = msg
+                                .param
+                                .get_blob(Param::File, context, !msg.is_increation())
+                                .await?
+                                .context("no file stored in params")?;
+                            let old_blob = old_draft
+                                .param
+                                .get_blob(Param::File, context, false)
+                                .await?
+                                .context("no file stored in params")?;
+                            if blob == old_blob {
+                                return Ok(false);
+                            }
+                        }
+                        return Ok(false);
+                    };
                     context
                         .sql
                         .execute(
@@ -7696,4 +7715,19 @@ mod tests {
 
         Ok(())
     }
+
+    #[tokio::test(flavor = "multi_thread", worker_threads = 2)]
+    async fn test_do_not_overwrite_draft() -> Result<()> {
+        let mut tcm = TestContextManager::new();
+        let alice = tcm.alice().await;
+        let mut msg = Message::new_text("This is a draft message".to_string());
+        let self_chat = alice.get_self_chat().await.id;
+        self_chat.set_draft(&alice, Some(&mut msg)).await.unwrap();
+        let draft1 = self_chat.get_draft(&alice).await?.unwrap();
+        tokio::time::sleep(Duration::from_millis(800)).await;
+        self_chat.set_draft(&alice, Some(&mut msg)).await.unwrap();
+        let draft2 = self_chat.get_draft(&alice).await?.unwrap();
+        assert_eq!(draft1.timestamp_sort, draft2.timestamp_sort);
+        Ok(())
+    }
 }