listinfo_overview(): Richard Barrett's patch # 828811 to reduce

listinfo and admin cgi process size by not keeping the entire mlist
object alive through a reference in the advertised list.  Only the
information used in the overview is kept.

1 files changed, 6 insertions, 5 deletions

diff --git a/Mailman/Cgi/listinfo.py b/Mailman/Cgi/listinfo.py
index fb9154ae..c1bdb796 100644
--- a/Mailman/Cgi/listinfo.py
+++ b/Mailman/Cgi/listinfo.py
@@ -91,8 +91,9 @@ def listinfo_overview(msg=''):
                 # List is for different identity of this host - skip it.
                 continue
             else:
-                advertised.append(mlist)
-
+                advertised.append((mlist.GetScriptURL('listinfo'),
+                                   mlist.real_name,
+                                   mlist.description))
     if msg:
         greeting = FontAttr(msg, color="ff5060", size="+1")
     else:
@@ -135,10 +136,10 @@ def listinfo_overview(msg=''):
                       Bold(FontAttr(_('Description'), size='+2'))
                       ])
         highlight = 1
-        for mlist in advertised:
+        for url, real_name, description in advertised:
             table.AddRow(
-                [Link(mlist.GetScriptURL('listinfo'), Bold(mlist.real_name)),
-                 mlist.description or Italic(_('[no description available]'))])
+                [Link(url, Bold(real_name)),
+                      description or Italic(_('[no description available]'))])
             if highlight and mm_cfg.WEB_HIGHLIGHT_COLOR:
                 table.AddRowInfo(table.GetCurrentRowIndex(),
                                  bgcolor=mm_cfg.WEB_HIGHLIGHT_COLOR)