Hiyas, Three days ago I downloaded the latest CVS snapshot and I applied my patches to it. Now my users reported that their letters didn't posted to the list. I found that there's an error in MailList.py (I think). These fast fixes comes from experience and not from evidence :-> --- MailList.py.orig Mon Jan 11 09:33:44 1999 +++ MailList.py Mon Jan 18 15:33:35 1999 @@ -1140,7 +1140,7 @@ members = self.GetDeliveryMembers() if dont_send_to_sender: try: - recipients.remove(members) + recips.remove(members) # # sender not in list (case sensitive username problem?) # @@ -1149,14 +1149,14 @@ "couldn't remove %s from recipient list: %s", sender, str(members)) - recipients = [] + recips = [] for m in members: if not self.GetUserOption(m, mm_cfg.DisableDelivery): - recipients.append(m) + recips.append(m) self.LogMsg("post", "post to %s from %s size=%d", self._internal_name, msg.GetSender(), len(msg.body)) - self.DeliverToList(msg, recipients, + self.DeliverToList(msg, recips, header = self.msg_header % self.__dict__, footer = self.msg_footer % self.__dict__) if ack_post: -- #!/bin/perl -sp0777i<X+d*lMLa^*lN%0]dsXx++lMlN/dsM0<j]dsj $/=unpack('H*',$_);$_=`echo 16dio\U$k"SK$/SM$n\EsN0p[lN*1 lK[d2%Sa2/d0$^Ixp"|dc`;s/\W//g;$_=pack('H*',/((..)*)$/)
"BN" == Balazs Nagy <julian7@kva.hu> writes:
BN> Three days ago I downloaded the latest CVS snapshot and I
BN> applied my patches to it. Now my users reported that their
BN> letters didn't posted to the list.
BN> I found that there's an error in MailList.py (I think). These
BN> fast fixes comes from experience and not from evidence :->I just noticed this one myself today! Here's a better patch, IMO.
-Barry
-------------------- snip snip -------------------- Index: MailList.py
RCS file: /projects/cvsroot/mailman/Mailman/MailList.py,v retrieving revision 1.109 diff -c -r1.109 MailList.py *** MailList.py 1999/01/13 23:55:23 1.109 --- MailList.py 1999/01/24 19:30:29
*** 1138,1146 **** ack_post = 1 # Deliver the mail. members = self.GetDeliveryMembers() if dont_send_to_sender: try: ! recipients.remove(members) # # sender not in list (case sensitive username problem?) # --- 1138,1150 ---- ack_post = 1 # Deliver the mail. members = self.GetDeliveryMembers()
recipients = []for m in members:if not self.GetUserOption(m, mm_cfg.DisableDelivery):
if dont_send_to_sender: try: ! recipients.remove(sender) # # sender not in list (case sensitive username problem?) #recipients.append(m)
*** 1149,1158 **** "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))
--- 1153,1158 ----
participants (2)
-
Balazs Nagy -
Barry A. Warsaw