[checkmk-commits] 4457 FIX BI aggregations: fixed BI aggregations showing state "MI"

Module: check_mk Branch: master Commit: b4b9f434025080d37cf38760c74e2c62a8cf2701 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=b4b9f434025080… Author: Andreas Boesl <ab(a)mathias-kettner.de> Date: Wed Mar 8 16:12:35 2017 +0100 4457 FIX BI aggregations: fixed BI aggregations showing state "MI" BI aggregations leafes had the state "MI" when the user had no permissions for this host/service. This has been fixed. The "MI" state was actually never shown in the web frontend before. Change-Id: I0bd34c89e4ee699246219d4b65ef2fabe4e28bc9 --- .werks/4457 | 11 +++++++++++ web/htdocs/bi.py | 6 +++--- 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/.werks/4457 b/.werks/4457 new file mode 100644 index 0000000..8392c94 --- /dev/null +++ b/.werks/4457 @@ -0,0 +1,11 @@ +Title: BI aggregations: fixed BI aggregations showing state "MI" +Level: 1 +Component: bi +Compatible: compat +Edition: cre +Version: 1.5.0i1 +Date: 1488985439 +Class: fix + +BI aggregations leafes had the state "MI" when the user had no permissions for this host/service. +This has been fixed. The "MI" state was actually never shown in the web frontend before. diff --git a/web/htdocs/bi.py b/web/htdocs/bi.py index 6a69ef2..ca20a3c 100644 --- a/web/htdocs/bi.py +++ b/web/htdocs/bi.py @@ -71,7 +71,7 @@ SINGLE = 'single' MULTIPLE = 'multi' # possible aggregated states -MISSING = -2 +MISSING = -2 # currently unused PENDING = -1 OK = 0 WARN = 1 @@ -2381,7 +2381,7 @@ def execute_leaf_node(node, status_info, aggregation_options): status = status_info.get((site, host)) if status == None: return ({ - "state" : MISSING, + "state" : None, "output" : _("Host %s not found") % host, "in_downtime" : 0, "acknowledged" : False, @@ -2431,7 +2431,7 @@ def execute_leaf_node(node, status_info, aggregation_options): return (state, assumed_state, node) return ({ - "state" : MISSING, + "state" : None, "output" : _("This host has no such service"), "in_downtime" : host_in_downtime, "acknowledged" : False,