From 5f4641c5e22edcdb1e3c3275a6353ee8da308c51 Mon Sep 17 00:00:00 2001
From: bwarsaw
Date: Sat, 26 Feb 2000 00:12:59 +0000
Subject: GetItemGuiValue(): slight reworking of the RadioButtonArray kludge at
 the top of this function.

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

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

diff --git a/Mailman/Cgi/admin.py b/Mailman/Cgi/admin.py
index 4c04e5451..9d8fb1b01 100644
--- a/Mailman/Cgi/admin.py
+++ b/Mailman/Cgi/admin.py
@@ -476,10 +476,10 @@ def GetItemGuiValue(mlist, kind, varname, params):
         # for the fact that there is one fewer option. correspondingly,
         # we alter the value back in the change options function -scott
         #
-        if varname == "subscribe_policy" and not mm_cfg.ALLOW_OPEN_SUBSCRIBE:
-            return RadioButtonArray(varname, params, getattr(mlist, varname)-1)
-        else:
-            return RadioButtonArray(varname, params, getattr(mlist, varname))
+        checked = getattr(mlist, varname)
+        if varname == 'subscribe_policy' and not mm_cfg.ALLOW_OPEN_SUBSCRIBE:
+            checked = checked - 1
+        return RadioButtonArray(varname, params, checked)
     elif (kind == mm_cfg.String or kind == mm_cfg.Email or
 	  kind == mm_cfg.Host or kind == mm_cfg.Number):
 	return TextBox(varname, getattr(mlist, varname), params)
-- 
cgit v1.2.3-70-g09d2