Module: check_mk
Branch: master
Commit: c4e5600aa249a99adbbd34d1db514f478a5fa89f
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=c4e5600aa249a9…
Author: Mathias Kettner <mk(a)mathias-kettner.de>
Date: Wed Dec 7 20:53:30 2011 +0100
WATO: add confirmation for deletion of host tags
---
web/htdocs/wato.py | 19 ++++++++++++++++---
1 files changed, 16 insertions(+), 3 deletions(-)
diff --git a/web/htdocs/wato.py b/web/htdocs/wato.py
index aaabbb2..0163cbc 100644
--- a/web/htdocs/wato.py
+++ b/web/htdocs/wato.py
@@ -7985,6 +7985,16 @@ def mode_hosttags(phase):
operations = [ x[0] for x in e[2] ]
message = rename_host_tags_after_confirmation(del_id, operations)
+ if message == True: # no confirmation yet
+ c = wato_confirm(_("Confirm deletion of the host "
+ "tag group '%s'") % del_id,
+ _("Do you really want to delete the "
+ "host tag group '%s'?") % del_id)
+ if c == False:
+ return ""
+ elif c == None:
+ return None
+
if message:
hosttags = [ e for e in hosttags if e[0] != del_id ]
save_hosttags(hosttags)
@@ -8295,7 +8305,8 @@ def save_hosttags(hosttags):
def rename_host_tags_after_confirmation(tag_id, operations):
mode = html.var("_repair")
if mode == "abort":
- raise MKUserError("id_0", "Please refine your changes or go back to the list of tag groups.")
+ raise MKUserError("id_0", _("Aborting change."))
+
elif mode:
if type(operations) == list: # make attribute unknown to system, important for save() operations
undeclare_host_tag_attribute(tag_id)
@@ -8309,7 +8320,8 @@ def rename_host_tags_after_confirmation(tag_id, operations):
change_host_tags_in_folders(tag_id, operations, "check", g_root_folder)
if affected_folders:
- message += _("Affected folders with an explicit reference to this tag group and that are affected by the change") + ":<ul>"
+ message += _("Affected folders with an explicit reference to this tag "
+ "group and that are affected by the change") + ":<ul>"
for folder in affected_folders:
message += '<li><a href="%s">%s</a></li>' % (
make_link_to([("mode", "editfolder")], folder),
@@ -8317,7 +8329,8 @@ def rename_host_tags_after_confirmation(tag_id, operations):
message += "</ul>"
if affected_hosts:
- message += _("Hosts where this tag group is explicitely set and that are effected by the change") + ":<ul><li>"
+ message += _("Hosts where this tag group is explicitely set "
+ "and that are effected by the change") + ":<ul><li>"
for nr, host in enumerate(affected_hosts):
if nr > 20:
message += "... (%d more)" % (len(affected_hosts) - 20)
Module: check_mk
Branch: master
Commit: 854c47607cf10fe1adffe69aa5b4307197a388d0
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=854c47607cf10f…
Author: Mathias Kettner <mk(a)mathias-kettner.de>
Date: Wed Dec 7 20:36:31 2011 +0100
WATO: fix selection of empty tag
---
.bugs/481 | 10 +++++++---
web/htdocs/wato.py | 2 ++
2 files changed, 9 insertions(+), 3 deletions(-)
diff --git a/.bugs/481 b/.bugs/481
index ad05856..d43b2b2 100644
--- a/.bugs/481
+++ b/.bugs/481
@@ -1,11 +1,15 @@
Title: Tag group with empty tag does not show default
Component: wato
-Benefit: 1
-State: open
Cost: 1
Date: 2011-12-06 12:26:37
-Targetversion: 1.2.0
+State: done
Class: bug
+Benefit: 1
+Fun: 0
+Targetversion: 1.2.0
If a tag group has an empty tag then that choice is not
being shown in the combo box.
+
+2011-12-07 20:36:48: changed state open -> done
+fixed.
diff --git a/web/htdocs/wato.py b/web/htdocs/wato.py
index 2ec191e..d0a4e9b 100644
--- a/web/htdocs/wato.py
+++ b/web/htdocs/wato.py
@@ -3442,6 +3442,8 @@ class HostTagAttribute(Attribute):
# But case could occur if tags definitions have been changed.
def render_input(self, value):
+ if value == None:
+ value = "" # Important for tag groups with an empty tag entry
# Tag groups with just one entry are being displayed
# as checkboxes
choices = []