From cf950f7430ecabc8d4ff370c4f9e6e01f7b44fd4 Mon Sep 17 00:00:00 2001
From: msapiro
Date: Tue, 20 Mar 2007 23:13:26 +0000
Subject: - Fixed a bug in OldStyleMemberships.addNewMember that allowed adding
 an address   with upper case in the domain if the local part was all lower
 case. - Changed the semantics of OldStyleMemberships.changeMemberAddress os
 that   in the case of a straightforward address change, i.e. nodelete = 0,  
 delivery status and time are preserved if BYUSER or BYADMIN.

---
 Mailman/OldStyleMemberships.py | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/Mailman/OldStyleMemberships.py b/Mailman/OldStyleMemberships.py
index 3a69a2dfc..2a72b8928 100644
--- a/Mailman/OldStyleMemberships.py
+++ b/Mailman/OldStyleMemberships.py
@@ -213,7 +213,7 @@ class OldStyleMemberships(MemberAdaptor.MemberAdaptor):
             value = 0
         else:
             value = member
-            member = member.lower()
+        member = member.lower()
         if digest:
             self.__mlist.digest_members[member] = value
         else:
@@ -255,6 +255,8 @@ class OldStyleMemberships(MemberAdaptor.MemberAdaptor):
         password = self.__mlist.passwords.get(memberkey,
                                               Utils.MakeRandomPassword())
         lang = self.getMemberLanguage(memberkey)
+        delivery = self.__mlist.delivery_status.get(member.lower(),
+                                              (MemberAdaptor.ENABLED,0))
         # First, possibly delete the old member
         if not nodelete:
             self.removeMember(memberkey)
@@ -264,6 +266,11 @@ class OldStyleMemberships(MemberAdaptor.MemberAdaptor):
         # Set the entire options bitfield
         if flags:
             self.__mlist.user_options[newaddress.lower()] = flags
+        # If this is a straightforward address change, i.e. nodelete = 0,
+        # preserve the delivery status and time if BYUSER or BYADMIN
+        if delivery[0] in (MemberAdaptor.BYUSER, MemberAdaptor.BYADMIN)\
+          and not nodelete:
+            self.__mlist.delivery_status[newaddress.lower()] = delivery
 
     def setMemberPassword(self, memberkey, password):
         assert self.__mlist.Locked()
-- 
cgit v1.3.1