From 4a0f3fb474c826b7ff85f952e30c4ba055d4f271 Mon Sep 17 00:00:00 2001
From: bwarsaw
Date: Mon, 1 Oct 2001 22:00:30 +0000
Subject: change_options(): Only print the "Successfully Removed:" message if  
   there actually were any members removed.

---
 Mailman/Cgi/admin.py | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

(limited to 'Mailman/Cgi/admin.py')

diff --git a/Mailman/Cgi/admin.py b/Mailman/Cgi/admin.py
index 4d470b7f3..8d1a9836f 100644
--- a/Mailman/Cgi/admin.py
+++ b/Mailman/Cgi/admin.py
@@ -1327,16 +1327,17 @@ def change_options(mlist, category, cgidata, doc):
             if newlang and newlang <> oldlang:
                 mlist.setMemberLanguage(user, newlang)
                   
-            for opt in ("hide", "nomail", "ack", "notmetoo", "plain"):
+            for opt in ('hide', 'nomail', 'ack', 'notmetoo', 'plain'):
                 opt_code = MailCommandHandler.option_info[opt]
                 if cgidata.has_key('%s_%s' % (user, opt)):
                     mlist.setMemberOption(user, opt_code, 1)
                 else:
                     mlist.setMemberOption(user, opt_code, 0)
         # Give some feedback on who's been removed
-        doc.AddItem(Header(5, _('Successfully Removed:')))
-        doc.AddItem(UnorderedList(*removes))
-        doc.AddItem('<p>')
+        if removes:
+            doc.AddItem(Header(5, _('Successfully Removed:')))
+            doc.AddItem(UnorderedList(*removes))
+            doc.AddItem('<p>')
         if errors:
             doc.AddItem(Header(5, _("Error Unsubscribing:")))
             items = ['%s -- %s' % (x[0], x[1]) for x in errors]
-- 
cgit v1.2.3-70-g09d2