[neomutt-devel] [PATCH] fix recipients when group replying with reply_self enabled

[Roger Pau Monne]

The current behavior when doing group reply with reply_self is not
correct in the following case. Given this example:

    From: myself
    To: someone
    Cc: alice, bob

Mutt will generate the following reply when doing a group reply with
reply_self enabled:

    From: myself
    To: someone, alice, bob

reply_self is completely ignored, and Mutt fill the To field joining
the To and Cc fields from the original message.

The following patch fixes this behavior, so that on group reply with
reply_self enabled Mutt will generate the following header:

    From: myself
    To: myself
    Cc: someone, alice, bob
---
 send.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/send.c b/send.c
index d41a161e..3b7cb8ac 100644
--- a/send.c
+++ b/send.c
@@ -672,7 +672,7 @@ void mutt_fix_reply_recipients(struct Envelope *env)
      * the user is the only recipient, it ends up on the TO: field
      */
     env->cc = remove_user(env->cc, (env->to == NULL));
-    env->to = remove_user(env->to, (env->cc == NULL));
+    env->to = remove_user(env->to, (env->cc == NULL) || option(OPT_REPLY_SELF));
   }
 
   /* the CC field can get cluttered, especially with lists */
-- 
2.13.5 (Apple Git-94)