diff options
| author | bwarsaw | 1999-01-24 19:41:27 +0000 |
|---|---|---|
| committer | bwarsaw | 1999-01-24 19:41:27 +0000 |
| commit | 7f6ba53b22d1945e32f2af601befa68c7ad8df48 (patch) | |
| tree | 9209586019d50797be1dd69139cbdd0ebc128d56 /Mailman/MailList.py | |
| parent | f3683337f36a36a9c6b47275d326ba1b491cecb8 (diff) | |
| download | mailman-7f6ba53b22d1945e32f2af601befa68c7ad8df48.tar.gz mailman-7f6ba53b22d1945e32f2af601befa68c7ad8df48.tar.zst mailman-7f6ba53b22d1945e32f2af601befa68c7ad8df48.zip | |
Post(): Fixed some bogosities in removing sender from recipients when
dont_send_to_sender is true. Code ordering problem, and argument to
recipients.remove() was wrong.
Diffstat (limited to 'Mailman/MailList.py')
| -rw-r--r-- | Mailman/MailList.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/Mailman/MailList.py b/Mailman/MailList.py index 53580dba9..5a80c52d2 100644 --- a/Mailman/MailList.py +++ b/Mailman/MailList.py @@ -1138,9 +1138,13 @@ class MailList(MailCommandHandler, HTMLFormatter, Deliverer, ListAdmin, ack_post = 1 # Deliver the mail. members = self.GetDeliveryMembers() + recipients = [] + for m in members: + if not self.GetUserOption(m, mm_cfg.DisableDelivery): + recipients.append(m) if dont_send_to_sender: try: - recipients.remove(members) + recipients.remove(sender) # # sender not in list (case sensitive username problem?) # @@ -1149,10 +1153,6 @@ class MailList(MailCommandHandler, HTMLFormatter, Deliverer, ListAdmin, "couldn't remove %s from recipient list: %s", sender, str(members)) - recipients = [] - for m in members: - if not self.GetUserOption(m, mm_cfg.DisableDelivery): - recipients.append(m) self.LogMsg("post", "post to %s from %s size=%d", self._internal_name, msg.GetSender(), len(msg.body)) |