From 7879952fde0c0a1f482af5f2ac4f2811b23ea9c9 Mon Sep 17 00:00:00 2001
From: Alexander Krotov <ilabdsf@gmail.com>
Date: Mon, 24 Feb 2020 01:22:28 +0300
Subject: [PATCH] Delete MDNs first in MsgId.delete_from_db()

---
 src/message.rs | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/message.rs b/src/message.rs
index 6dbe05665..bd9038fbc 100644
--- a/src/message.rs
+++ b/src/message.rs
@@ -101,17 +101,19 @@ impl MsgId {
 
     /// Deletes a message and corresponding MDNs from the database.
     pub fn delete_from_db(self, context: &Context) {
+        // We don't use transactions yet, so remove MDNs first to make
+        // sure they are not left while the message is deleted.
         sql::execute(
             context,
             &context.sql,
-            "DELETE FROM msgs WHERE id=?;",
+            "DELETE FROM msgs_mdns WHERE msg_id=?;",
             params![self],
         )
         .ok();
         sql::execute(
             context,
             &context.sql,
-            "DELETE FROM msgs_mdns WHERE msg_id=?;",
+            "DELETE FROM msgs WHERE id=?;",
             params![self],
         )
         .ok();