summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorbwarsaw1999-07-03 01:47:59 +0000
committerbwarsaw1999-07-03 01:47:59 +0000
commit97dbc07f57da9407cdff2bcc4aa7408e55d955a9 (patch)
treefd6ee626b1ae85c66e4d7aee8ea25abedeb00089
parentdec556341a6bac11ae67c80d571cc8cd7d0e9083 (diff)
downloadmailman-97dbc07f57da9407cdff2bcc4aa7408e55d955a9.tar.gz
mailman-97dbc07f57da9407cdff2bcc4aa7408e55d955a9.tar.zst
mailman-97dbc07f57da9407cdff2bcc4aa7408e55d955a9.zip
GetUserOption(), SetUserOption(): The invariant is that the keys for
list.user_option must be lowercase addresses (i.e. canonicalized addrs). The DATA_FILE_VERSION bump will fix existing lists and these two functions always lowercase the user address before checking it. Fixes PR#74
-rw-r--r--Mailman/MailList.py2
1 files changed, 2 insertions, 0 deletions
diff --git a/Mailman/MailList.py b/Mailman/MailList.py
index 65c91a018..673e6df5a 100644
--- a/Mailman/MailList.py
+++ b/Mailman/MailList.py
@@ -193,6 +193,7 @@ class MailList(MailCommandHandler, HTMLFormatter, Deliverer, ListAdmin,
def GetUserOption(self, user, option):
"""Return user's setting for option, defaulting to 0 if no settings."""
+ user = self.GetUserCanonicalAddress(user)
if option == mm_cfg.Digests:
return self.digest_members.has_key(user)
if not self.user_options.has_key(user):
@@ -200,6 +201,7 @@ class MailList(MailCommandHandler, HTMLFormatter, Deliverer, ListAdmin,
return not not self.user_options[user] & option
def SetUserOption(self, user, option, value, save_list=1):
+ user = self.GetUserCanonicalAddress(user)
if not self.user_options.has_key(user):
self.user_options[user] = 0
if value: