[Mailman-Users] Discarding held message generates an error.

[Barry A. Warsaw]

Here's the fix that'll be in b12.

-Barry

Index: ListAdmin.py
===================================================================
RCS file: /projects/cvsroot/mailman/Mailman/ListAdmin.py,v
retrieving revision 1.20
retrieving revision 1.21
diff -c -r1.20 -r1.21
*** ListAdmin.py	1999/03/31 17:52:42	1.20
--- ListAdmin.py	1999/04/09 01:20:36	1.21
***************
*** 147,163 ****
  	else:
              # Discard.
  	    rejection = "Discarded"
  	if rejection:
              note = '''%(listname)s: %(rejection)s posting:
  \tFrom: %(sender)s
! \tSubject: %(subject)s''' % {'listname' : self._internal_name,
!                              'rejection': rejection,
!                              'sender'   : msg.GetSender(),
!                              'subject'  : msg.getheader('subject', '<none>'),
!                              }
!             def strquote(s):
!                 return string.replace(s, '%', '%%')
!             
  	    if data[1]:
  		note = note + '\n\tHeld: ' + strquote(data[1])
  	    if comment:
--- 147,165 ----
  	else:
              # Discard.
  	    rejection = "Discarded"
+ 
+         def strquote(s):
+             return string.replace(s, '%', '%%')
+ 
  	if rejection:
              note = '''%(listname)s: %(rejection)s posting:
  \tFrom: %(sender)s
! \tSubject: %(subject)s''' % {
!                 'listname' : self._internal_name,
!                 'rejection': rejection,
!                 'sender'   : msg.GetSender(),
!                 'subject'  : strquote(msg.getheader('subject', '<none>')),
!                 }
  	    if data[1]:
  		note = note + '\n\tHeld: ' + strquote(data[1])
  	    if comment: