From 5909c15aa72ed244c64d0a0831a44da0fbedc6ba Mon Sep 17 00:00:00 2001 From: Mark Sapiro Date: Sat, 23 Feb 2008 15:14:01 -0800 Subject: MailList.py - Changed the already a member logic in the ChangeMemberAddress() method. This time I think I got it right. --- Mailman/MailList.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'Mailman/MailList.py') diff --git a/Mailman/MailList.py b/Mailman/MailList.py index a1374398..7cd2c5cb 100644 --- a/Mailman/MailList.py +++ b/Mailman/MailList.py @@ -1065,9 +1065,10 @@ class MailList(HTMLFormatter, Deliverer, ListAdmin, newaddr = Utils.LCDomain(newaddr) Utils.ValidateEmail(newaddr) # Raise an exception if this email address is already a member of the - # list, but only if the new address is the same case-wise as the old - # address and we're not doing a global change. - if not globally and newaddr == oldaddr and self.isMember(newaddr): + # list, but only if the new address is the same case-wise as the + # existing member address and we're not doing a global change. + if not globally and (self.isMember(newaddr) and + newaddr == self.getMemberCPAddress(newaddr)): raise Errors.MMAlreadyAMember if newaddr == self.GetListEmail().lower(): raise Errors.MMBadEmailError -- cgit v1.2.3