mirror/librenms-librenms
1
0
Fork 0
mirror of https://github.com/librenms/librenms.git synced 2024-10-07 16:52:45 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Add alert rule option to invert devices and groups "map to" list (#11038)

Browse Source 
* Add alert rule option to invert map to
* Update 2019_12_17_151314_add_invert_map_to_alert_rules.php
* fix invert map to does not work if several groups are set in map to
* Clarify labels
* fix invert map to at rule creation
* clarify invert label
* Fix device in map to - not group
* dummy commit
This commit is contained in:
louis-oui
2020-01-19 14:55:51 +01:00
committed by PipoCanaja
parent 25677e5edc
commit e235c59d2f
6 changed files with 71 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
includes/html/forms/alert-rules.inc.php
View File

@@ -58,6 +58,7 @@ $invert = mres(isset($_POST['invert']) ? $_POST['invert'] : null);
$name = mres($_POST['name']);
$proc = mres($_POST['proc']);
$recovery = ($vars['recovery']);
$invert_map = mres(isset($_POST['invert_map']) ? $_POST['invert_map'] : null);
$severity = mres($_POST['severity']);
if (!is_numeric($count)) {
@@ -81,6 +82,12 @@ if ($invert == 'on') {
$recovery = empty($recovery) ? $recovery = false : true;
if ($invert_map == 'on') {
$invert_map = true;
} else {
$invert_map = false;
}
$extra = array(
'mute' => $mute,
'count' => $count,
@@ -101,7 +108,8 @@ if (is_numeric($rule_id) && $rule_id > 0) {
'name' => $name,
'proc' => $proc,
'query' => $query,
'builder' => $builder_json
'builder' => $builder_json,
'invert_map' => $invert_map
),
'alert_rules',
'id=?',
@@ -128,7 +136,8 @@ if (is_numeric($rule_id) && $rule_id > 0) {
'name' => $name,
'proc' => $proc,
'query' => $query,
'builder' => $builder_json
'builder' => $builder_json,
'invert_map' => $invert_map
), 'alert_rules');
if ($rule_id) {

1
includes/html/forms/parse-alert-rule.inc.php
View File

@@ -77,5 +77,6 @@ if (is_array($rule)) {
'builder' => $builder,
'severity' => $rule['severity'],
'adv_query' => $rule['query'],
'invert_map' => $rule['invert_map'],
]);
}

22
includes/html/modal/new_alert_rule.inc.php
View File

@@ -98,7 +98,7 @@ if (Auth::user()->hasGlobalAdmin()) {
<div class='col-sm-2' title="Show alert status in the webui, but do not issue notifications.">
<input type="checkbox" name="mute" id="mute">
</div>
<label for='invert' class='col-sm-3 col-md-2 control-label' title="Alert when this rule doesn't match." style="vertical-align: top;">Invert match: </label>
<label for='invert' class='col-sm-3 col-md-3 control-label' title="Alert when this rule doesn't match." style="vertical-align: top;">Invert rule match: </label>
<div class='col-sm-2' title="Alert when this rule doesn't match.">
<input type='checkbox' name='invert' id='invert'>
</div>
@@ -109,11 +109,15 @@ if (Auth::user()->hasGlobalAdmin()) {
<input type='checkbox' name='recovery' id='recovery'>
</div>
</div>
<div class="form-group" title="Restricts this alert rule to the selected devices or groups.">
<label for='maps' class='col-sm-3 col-md-2 control-label'>Map To: </label>
<div class="col-sm-9 col-md-10">
<div class="form-group form-inline">
<label for='maps' class='col-sm-3 col-md-2 control-label' title="Restricts this alert rule to the selected devices and groups.">Match devices and groups list: </label>
<div class="col-sm-7" style="width: 56%;">
<select id="maps" name="maps[]" class="form-control" multiple="multiple"></select>
</div>
<div>
<label for='invert_map' class='col-md-1' style="width: 14.1333%;" text-align="left" title="If ON, alert rule check will run on all devices except the selected devices and groups.">All devices except in list: </label>
<input type='checkbox' name='invert_map' id='invert_map'>
</div>
</div>
<div class="form-group" title="Restricts this alert rule to specified transports.">
<label for="transports" class="col-sm-3 col-md-2 control-label">Transports: </label>
@@ -292,6 +296,7 @@ if (Auth::user()->hasGlobalAdmin()) {
$("#invert").bootstrapSwitch('state', false);
$("#recovery").bootstrapSwitch('state', true);
$("#override_query").bootstrapSwitch('state', false);
$("#invert_map").bootstrapSwitch('state', false);
$(this).find("input[type=text]").val("");
$('#count').val('-1');
$('#delay').val('1m');
@@ -365,12 +370,12 @@ if (Auth::user()->hasGlobalAdmin()) {
if (extra.adv_query) {
$('#adv_query').val(extra.adv_query);
}
$("[name='mute']").bootstrapSwitch('state', extra.mute);
$("[name='invert']").bootstrapSwitch('state', extra.invert);
if (typeof extra.recovery == 'undefined') {
extra.recovery = true;
}
if (typeof extra.options == 'undefined') {
extra.options = new Array();
}
@@ -378,6 +383,13 @@ if (Auth::user()->hasGlobalAdmin()) {
extra.options.override_query = false;
}
$("[name='recovery']").bootstrapSwitch('state', extra.recovery);
if (rule.invert_map == 1) {
$("[name='invert_map']").bootstrapSwitch('state', true);
}else{
$("[name='invert_map']").bootstrapSwitch('state', false);
}
$("[name='override_query']").bootstrapSwitch('state', extra.options.override_query);
} else {
$('#count').val('-1');