From ccb96246ffaa199a5d89e2dd5f9f5609bf4452ff Mon Sep 17 00:00:00 2001
From: bwarsaw
Date: Tue, 24 Dec 2002 15:38:31 +0000
Subject: show_helds_overview(): Finally tracked down a most annoying buglet.
 If a non-member sends a message which is held, but then the sender
 subsequently subscribes to the list, a confusing admindb page is shown.  Now
 we include a message that the sender has since joined the list.

---
 Mailman/Cgi/admindb.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

(limited to 'Mailman')

diff --git a/Mailman/Cgi/admindb.py b/Mailman/Cgi/admindb.py
index 3b21558e8..e6b71cda2 100644
--- a/Mailman/Cgi/admindb.py
+++ b/Mailman/Cgi/admindb.py
@@ -420,7 +420,10 @@ def show_helds_overview(mlist, form):
                     ' ' +
                     _("Clear this member's <em>moderate</em> flag")
                     ])
-                left.AddCellInfo(left.GetCurrentRowIndex(), 0, colspan=2)
+            else:
+                left.AddRow(
+                    [_('<em>The sender is now a member of this list</em>')])
+            left.AddCellInfo(left.GetCurrentRowIndex(), 0, colspan=2)
         elif sender not in (mlist.accept_these_nonmembers +
                             mlist.hold_these_nonmembers +
                             mlist.reject_these_nonmembers +
-- 
cgit v1.2.3-70-g09d2