diff options
Diffstat (limited to 'Mailman/Cgi/admin.py')
| -rw-r--r-- | Mailman/Cgi/admin.py | 8 |
1 files changed, 4 insertions, 4 deletions
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) |