diff options
| author | klm | 1998-11-24 04:06:37 +0000 |
|---|---|---|
| committer | klm | 1998-11-24 04:06:37 +0000 |
| commit | aa131f569c0394cb9868f989765d5e7013f73b5a (patch) | |
| tree | ad793aba76c8f3476cf336dd522d35017fd0a944 /Mailman/MailList.py | |
| parent | 215637878b3c7e80cd2c7541e0bf121423dff386 (diff) | |
| download | mailman-aa131f569c0394cb9868f989765d5e7013f73b5a.tar.gz mailman-aa131f569c0394cb9868f989765d5e7013f73b5a.tar.zst mailman-aa131f569c0394cb9868f989765d5e7013f73b5a.zip | |
.ProcessConfirmation(): Due to the elimination of the .Save()
in ApprovedAddMember, new confirmed subscriptions were being only
partially saved - the password entry was being left out. Which is a
problem. (In the five hours this has been happening on my site, i
wonder how many broken accounts there are? Not good.)
Diffstat (limited to 'Mailman/MailList.py')
| -rw-r--r-- | Mailman/MailList.py | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/Mailman/MailList.py b/Mailman/MailList.py index 9b526b18a..ff1384679 100644 --- a/Mailman/MailList.py +++ b/Mailman/MailList.py @@ -851,7 +851,6 @@ class MailList(MailCommandHandler, HTMLFormatter, Deliverer, ListAdmin, "member": name}, 1) msg = Message.IncomingMessage(txt) self.DeliverToOwner(msg, self.owner) - def ProcessConfirmation(self, cookie): from Pending import Pending @@ -864,8 +863,7 @@ class MailList(MailCommandHandler, HTMLFormatter, Deliverer, ListAdmin, self.AddRequest('add_member', digest, email_addr, password) raise Errors.MMNeedApproval, self.GetAdminEmail() self.ApprovedAddMember(email_addr, password, digest) - - + self.Save() def DeleteMember(self, name, whence=None, admin_notif=None): self.IsListInitialized() @@ -967,7 +965,6 @@ class MailList(MailCommandHandler, HTMLFormatter, Deliverer, ListAdmin, % (self.real_name, `stripped`)) return all - def HasMatchingHeader(self, msg): """True if named header field (case-insensitive) matches regexp. |