From 23ce186d5090cdd52d43c68f844b4060ebeb15cd Mon Sep 17 00:00:00 2001 From: bwarsaw Date: Fri, 4 Jan 2002 06:39:43 +0000 Subject: get_item_gui_value(): Filter out volatile attributes when using getattr(), since they won't exist. --- Mailman/Cgi/admin.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/Mailman/Cgi/admin.py b/Mailman/Cgi/admin.py index bf5247202..33930095c 100644 --- a/Mailman/Cgi/admin.py +++ b/Mailman/Cgi/admin.py @@ -617,7 +617,8 @@ def get_item_gui_value(mlist, category, kind, varname, params): if hasattr(gui, 'GetValue'): missing = [] value = gui.GetValue(mlist, kind, varname, params) - if value is None: + # Filter out None, and volatile attributes + if value is None and not varname.startswith('_'): value = getattr(mlist, varname) # Now create the widget for this value if kind == mm_cfg.Radio or kind == mm_cfg.Toggle: @@ -629,7 +630,7 @@ def get_item_gui_value(mlist, category, kind, varname, params): # function -scott # # TBD: this is an ugly ugly hack. - if varname[0] == '_': + if varname.startswith('_'): checked = 0 else: checked = value -- cgit v1.3.1