Module: check_mk
Branch: master
Commit: bea52dbae57f6e7fa3738853fb2da1dbfccff0b8
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=bea52dbae57f6e…
Author: Mathias Kettner <mk(a)mathias-kettner.de>
Date: Fri Dec 16 09:31:39 2011 +0100
WATO: Make sure tag ids are unique
---
.bugs/479 | 7 +++++--
.bugs/480 | 7 +++++--
web/htdocs/wato.py | 4 ++++
3 files changed, 14 insertions(+), 4 deletions(-)
diff --git a/.bugs/479 b/.bugs/479
index ba51093..4ee00a5 100644
--- a/.bugs/479
+++ b/.bugs/479
@@ -1,11 +1,14 @@
Title: Host tags reorder works only partially
Component: wato
+State: done
+Class: bug
Benefit: 1
-State: open
Cost: 1
Date: 2011-12-06 12:11:21
Targetversion: 1.2.0
-Class: bug
Changing the order of host tags in WATO does not
work with the down arrow on the first tag group.
+
+2011-12-16 09:28:03: changed state open -> done
+Already fixed.
diff --git a/.bugs/480 b/.bugs/480
index 1f1c46d..da1895a 100644
--- a/.bugs/480
+++ b/.bugs/480
@@ -1,11 +1,14 @@
Title: Check for duplicate host tags values
Component: wato
+State: done
+Class: bug
Benefit: 1
-State: open
Cost: 1
Date: 2011-12-06 12:18:18
Targetversion: 1.2.0
-Class: bug
When editing host tags make sure that within one tag group
each tag is unique.
+
+2011-12-16 09:31:23: changed state open -> done
+Is now checked.
diff --git a/web/htdocs/wato.py b/web/htdocs/wato.py
index 55a4a1b..10bc735 100644
--- a/web/htdocs/wato.py
+++ b/web/htdocs/wato.py
@@ -8130,6 +8130,10 @@ def mode_edit_hosttag(phase):
if have_none_tag:
raise MKUserError("id_%d" % nr, _("Only on tag may be empty."))
have_none_tag = True
+ # Make sure tag ID is unique within this group
+ if id in [ x[0] for x in new_choices ]:
+ raise MKUserError("id_%d" % nr, _("Tags IDs must be unique. You've used <b>%s</b> twice.") % id)
+
new_choices.append((id, descr))
if id:
# Make sure this ID is not used elsewhere