aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat (limited to '')
-rw-r--r--Mailman/Handlers/WrapMessage.py21
-rwxr-xr-xNEWS3
2 files changed, 12 insertions, 12 deletions
diff --git a/Mailman/Handlers/WrapMessage.py b/Mailman/Handlers/WrapMessage.py
index d9f4e04a..9678f6fd 100644
--- a/Mailman/Handlers/WrapMessage.py
+++ b/Mailman/Handlers/WrapMessage.py
@@ -42,18 +42,15 @@ def process(mlist, msg, msgdata):
# is wrap this message or from_is_list applies and is wrap.
if not (msgdata.get('from_is_list') == 2 or
(mlist.from_is_list == 2 and msgdata.get('from_is_list') == 0)):
- # Now see if we're munging.
- if msgdata.get('from_is_list') == 1 or (mlist.from_is_list == 1 and
- msgdata.get('from_is_list') == 0):
- # Yes.
- a_h = msgdata.get('add_header')
- if a_h:
- if a_h.get('From'):
- del msg['from']
- msg['From'] = a_h.get('From')
- if a_h.get('Reply-To'):
- del msg['reply-to']
- msg['Reply-To'] = a_h.get('Reply-To')
+ # Now see if we need to add a From: and/or Reply-To: without wrapping.
+ a_h = msgdata.get('add_header')
+ if a_h:
+ if a_h.get('From'):
+ del msg['from']
+ msg['From'] = a_h.get('From')
+ if a_h.get('Reply-To'):
+ del msg['reply-to']
+ msg['Reply-To'] = a_h.get('Reply-To')
return
# There are various headers in msg that we don't want, so we basically
diff --git a/NEWS b/NEWS
index 717efcc2..d8a1822f 100755
--- a/NEWS
+++ b/NEWS
@@ -9,6 +9,9 @@ Here is a history of user visible changes to Mailman.
Bug fixes and other patches
+ - The Reply-To: munging options weren't honored if there was no
+ from_is_list action. (LP: #1313010)
+
- Changed from_is_list actions to insert the list address in Cc: if the
list is fully personalized. Otherwise, the list address is only in
From: and Reply-To: overrides it. (LP: #1312970)