------------------------------------------------------------
revno: 1051
committer: Mark Sapiro <[EMAIL PROTECTED]>
branch nick: 2.1
timestamp: Sat 2008-02-23 15:14:01 -0800
message:
  MailList.py - Changed the already a member logic in the ChangeMemberAddress()
                method.  This time I think I got it right.
modified:
  Mailman/MailList.py

=== modified file 'Mailman/MailList.py'
--- a/Mailman/MailList.py       2008-02-23 21:43:59 +0000
+++ b/Mailman/MailList.py       2008-02-23 23:14:01 +0000
@@ -1065,9 +1065,10 @@
         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



--
Stable, maintained release series
https://code.launchpad.net/~mailman-coders/mailman/2.1

You are receiving this branch notification because you are subscribed to it.
_______________________________________________
Mailman-checkins mailing list
[email protected]
Unsubscribe: 
http://mail.python.org/mailman/options/mailman-checkins/archive%40jab.org

Reply via email to