Module: check_mk
Branch: master
Commit: 19135fe771a16c78eb5426722ec2bbf558b57ccc
URL:
http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=19135fe771a16c…
Author: Lars Michelsen <lm(a)mathias-kettner.de>
Date: Wed Apr 12 11:08:36 2017 +0200
Minor code cleanup
Change-Id: Ib1cca07411c9716e07f2e33aa5453ef18b3fac7b
---
web/htdocs/wato.py | 29 +++++++++++++++--------------
1 file changed, 15 insertions(+), 14 deletions(-)
diff --git a/web/htdocs/wato.py b/web/htdocs/wato.py
index 1612259..39897bc 100644
--- a/web/htdocs/wato.py
+++ b/web/htdocs/wato.py
@@ -6737,7 +6737,7 @@ def render_global_configuration_variables(group_names,
default_values, current_s
forms.header(group_name, isopen=search)
header_is_painted = True
- defaultvalue = default_values.get(varname, valuespec.default_value())
+ default_value = default_values.get(varname, valuespec.default_value())
edit_url = folder_preserving_link([("mode", edit_mode),
("varname", varname),
@@ -6749,7 +6749,7 @@ def render_global_configuration_variables(group_names,
default_values, current_s
if varname in current_settings:
to_text = valuespec.value_to_text(current_settings[varname])
else:
- to_text = valuespec.value_to_text(defaultvalue)
+ to_text = valuespec.value_to_text(default_value)
# Is this a simple (single) value or not? change styling in these cases...
simple = True
@@ -6758,20 +6758,21 @@ def render_global_configuration_variables(group_names,
default_values, current_s
forms.section(title, simple=simple)
toggle_url = html.makeactionuri([("_action", "toggle"),
("_varname", varname)])
+
if varname in current_settings:
- if is_a_checkbox(valuespec):
- html.icon_button(toggle_url, _("Immediately toggle this
setting"),
- "snapin_switch_" + (current_settings[varname] and
"on" or "off"),
- cssclass="modified")
- else:
- html.a(HTML(to_text), href=edit_url, class_="modified")
+ value = current_settings[varname]
+ modified_cls = "modified"
else:
- if is_a_checkbox(valuespec):
- html.icon_button(toggle_url, _("Immediately toggle this
setting"),
- # "snapin_greyswitch_" + (defaultvalue and "on"
or "off"))
- "snapin_switch_" + (defaultvalue and "on" or
"off"))
- else:
- html.a(HTML(to_text), href=edit_url)
+ value = default_value
+ modified_cls = None
+
+ if is_a_checkbox(valuespec):
+ html.icon_button(toggle_url, _("Immediately toggle this
setting"),
+ "snapin_switch_" + (value and "on" or
"off"),
+ cssclass=modified_cls)
+
+ else:
+ html.a(HTML(to_text), href=edit_url, class_=modified_cls)
if header_is_painted:
forms.end()