Paul Kleeberg wrote:
>
>In attending to a subscription request to a sunsetted list (the request came 
>before the migration) via the web interface, I received an error.  Here is the 
>mailman error log (I have modified e-mail addresses and the domain name):
>
>Dec 13 08:35:55 2009 admin(81221): 
>@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@ 
>admin(81221): [----- Mailman Version: 2.1.12rc1 -----] 
>admin(81221): [----- Traceback ------] 
>admin(81221): Traceback (most recent call last):
>admin(81221):   File "/usr/share/mailman/scripts/driver", line 112, in run_main
>admin(81221):     main()
>admin(81221):   File 
>"/BinaryCache/mailman/mailman-132~28/Root/usr/share/mailman/Mailman/Cgi/admindb.py",
> line 164, in main
>admin(81221):   File 
>"/BinaryCache/mailman/mailman-132~28/Root/usr/share/mailman/Mailman/Cgi/admindb.py",
> line 773, in process_form
>admin(81221): AttributeError: 'str' object has no attribute 'get'


This is a bug in 2.1.12rc1. Upgrade to 2.1.12 or install the attached
admindb.patch.txt.

-- 
Mark Sapiro <m...@msapiro.net>        The highway is for gamblers,
San Francisco Bay Area, California    better use your sense - B. Dylan

=== modified file 'Mailman/Cgi/admindb.py'
--- Mailman/Cgi/admindb.py      2009-01-03 02:40:28 +0000
+++ Mailman/Cgi/admindb.py      2009-02-03 23:47:36 +0000
@@ -34,6 +34,7 @@
 from Mailman import Message
 from Mailman import i18n
 from Mailman.Handlers.Moderate import ModeratedMemberPost
+from Mailman.ListAdmin import HELDMSG
 from Mailman.ListAdmin import readMessage
 from Mailman.Cgi import Auth
 from Mailman.htmlformat import *
@@ -769,8 +770,12 @@
         forwardaddrkey = 'forward-addr-%d' % request_id
         bankey = 'ban-%d' % request_id
         # Defaults
-        msgdata = mlist.GetRecord(request_id)[5]
-        comment = msgdata.get('rejection_notice', _('[No explanation given]'))
+        if mlist.GetRecordType(request_id) == HELDMSG:
+            msgdata = mlist.GetRecord(request_id)[5]
+            comment = msgdata.get('rejection_notice',
+                                  _('[No explanation given]'))
+        else:
+            comment = _('[No explanation given]')
         preserve = 0
         forward = 0
         forwardaddr = ''

------------------------------------------------------
Mailman-Users mailing list Mailman-Users@python.org
http://mail.python.org/mailman/listinfo/mailman-users
Mailman FAQ: http://wiki.list.org/x/AgA3
Security Policy: http://wiki.list.org/x/QIA9
Searchable Archives: http://www.mail-archive.com/mailman-users%40python.org/
Unsubscribe: 
http://mail.python.org/mailman/options/mailman-users/archive%40jab.org

Reply via email to