From a48d0492c85fe067d851dd075975726f8ad66b4b Mon Sep 17 00:00:00 2001
From: Alexander Krotov <ilabdsf@gmail.com>
Date: Mon, 6 Jan 2020 11:20:41 +0100
Subject: [PATCH] Create only one MarkseenMdnOnImap job even if message has
 multiple reports

All created jobs have the same parameters and move the same message,
so it does not make sense to create more than one.
---
 src/mimeparser.rs | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/mimeparser.rs b/src/mimeparser.rs
index df19a0318..6ee10630e 100644
--- a/src/mimeparser.rs
+++ b/src/mimeparser.rs
@@ -776,9 +776,8 @@ impl<'a> MimeMessage<'a> {
             return;
         }
 
-        for report in &self.reports {
             let mut mdn_recognized = false;
-
+        for report in &self.reports {
                 if let Some((chat_id, msg_id)) = message::mdn_from_ext(
                     self.context,
                     from_id,
@@ -788,6 +787,7 @@ impl<'a> MimeMessage<'a> {
                     self.context.call_cb(Event::MsgRead { chat_id, msg_id });
                     mdn_recognized = true;
                 }
+        }
 
             if self.has_chat_version() || mdn_recognized {
                 let mut param = Params::new();
@@ -798,7 +798,6 @@ impl<'a> MimeMessage<'a> {
                 }
                 job_add(self.context, Action::MarkseenMdnOnImap, 0, param, 0);
             }
-        }
     }
 }