These changes represent the changes that make mailman's addressing

work like unix mail:  username portions of addresses are
case-preserved for delivery only.  All other address comparisons are
lowercase.
up'd data version in Defaults to 13
** MailList.py: added an __AddMember method that takes an address and
whether or not it is a digest address as args and populates the member
dictionary like this:
if string.lower(addr) == LCDomain(addr):
    member_dict[addr] = 0
else:
    member_dictp[string.lower(addr)] == LCDomain(addr)
added .GetDeliveryMembers() and .GetDigestDeliveryMembers() methods
for use by posting and digest delivery mechanisms.
changed a nested def portion in Post to use an explicit loop.
**Digester.py: uses .GetDigestDeliveryMembers instead of
  .GetDigestMembers().
**Utils.py: address comparisons are lc.
**versions.py: populate the member dicts according to the above
  formula.
**Cgi/admin.py: fixed a bug introduced from the first change to using
  member dictionaries: all members were showing up as digest members
  on the admin membership page.  changed a dict.get to dict.has_key to
  fix this.
NOTE: this code is tested only insofar as I posted and poked around on
the cgi's a bit, and let lists do the versions code.  These changes
should be tested more, but I *have* to go now and wanted to check them
in before someone made it difficult to do so by changing stuff
tomarrow, cause i'm gone all day tomarrow.
scott

1 files changed, 3 insertions, 3 deletions

diff --git a/Mailman/Cgi/admin.py b/Mailman/Cgi/admin.py
index bc2b0989d..0fe888b4d 100644
--- a/Mailman/Cgi/admin.py
+++ b/Mailman/Cgi/admin.py
@@ -518,7 +518,7 @@ def FormatMembershipOptions(lst):
         cells = [member + "<input type=hidden name=user value=%s>" % (member),
                  "subscribed "
                  + CheckBox(member + "_subscribed", "on", 1).Format()]
-        if lst.members.get(member):
+        if lst.members.has_key(member):
             cells.append("digest "
                          + CheckBox(member + "_digest", "off", 0).Format())
         else:
@@ -790,11 +790,11 @@ def ChangeOptions(lst, category, cgi_info, document):
                     del lst.digest_members[user]
                     dirty = 1
                 if not lst.members.has_key(user):
-                    lst.members[user] = 1
+                    lst.members[user] = 0
                     dirty = 1
             else:
                 if not lst.digest_members.has_key(user):
-                    lst.digest_members[user] = 1
+                    lst.digest_members[user] = 0
                     dirty = 1
                 if lst.members.has_key(user):
                     del lst.members[user]