From 4e967acf9b45a6a6d24bc3d84c6d40f6dcffc1e8 Mon Sep 17 00:00:00 2001
From: Mark Sapiro <mark@msapiro.net>
Date: Sun, 6 Dec 2015 12:00:29 -0800
Subject: Submitting the user options form for a user who was asynchronously
 unsubscribed would throw an uncaught NotAMemberError.

---
 Mailman/Cgi/options.py | 8 ++++++++
 1 file changed, 8 insertions(+)

(limited to 'Mailman')

diff --git a/Mailman/Cgi/options.py b/Mailman/Cgi/options.py
index 7b2c70b5..cdc2bef3 100644
--- a/Mailman/Cgi/options.py
+++ b/Mailman/Cgi/options.py
@@ -297,6 +297,14 @@ def main():
     # options.  The first set of checks does not require the list to be
     # locked.
 
+    # However, if a form is submitted for a user who has been asynchronously
+    # unsubscribed, uncaught NotAMemberError exceptions can be thrown.
+
+    if not mlist.isMember(user):
+        loginpage(mlist, doc, user, language)
+        print doc.Format()
+        return
+
     if cgidata.has_key('logout'):
         print mlist.ZapCookie(mm_cfg.AuthUser, user)
         loginpage(mlist, doc, user, language)
-- 
cgit v1.2.3