[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mailman-cvs
Subject:    [Mailman-checkins] SF.net SVN: mailman: [8169]
From:       msapiro () users ! sourceforge ! net
Date:       2007-03-20 23:13:27
Message-ID: E1HTnWB-0007vp-1B () sc8-pr-svn1 ! sourceforge ! net
[Download RAW message or body]

Revision: 8169
          http://svn.sourceforge.net/mailman/?rev=8169&view=rev
Author:   msapiro
Date:     2007-03-20 16:13:26 -0700 (Tue, 20 Mar 2007)

Log Message:
-----------
- 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.

Modified Paths:
--------------
    trunk/mailman/Mailman/OldStyleMemberships.py

Modified: trunk/mailman/Mailman/OldStyleMemberships.py
===================================================================
--- trunk/mailman/Mailman/OldStyleMemberships.py	2007-03-20 23:12:18 UTC (rev 8168)
+++ trunk/mailman/Mailman/OldStyleMemberships.py	2007-03-20 23:13:26 UTC (rev 8169)
@@ -213,7 +213,7 @@
             value = 0
         else:
             value = member
-            member = member.lower()
+        member = member.lower()
         if digest:
             self.__mlist.digest_members[member] = value
         else:
@@ -255,6 +255,8 @@
         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 @@
         # 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()


This was sent by the SourceForge.net collaborative development platform, the world's \
largest Open Source development site. _______________________________________________
Mailman-checkins mailing list
Mailman-checkins@python.org
Unsubscribe: http://mail.python.org/mailman/options/mailman-checkins/mailman-cvs%40progressive-comp.com



[prev in list] [next in list] [prev in thread] [next in thread]