From bee5099a085550a1bed44091ef8bab54fdfbef72 Mon Sep 17 00:00:00 2001 From: Eric Lippmann Date: Tue, 16 Jul 2019 10:12:33 +0200 Subject: [PATCH] Fix show all services links in the servicegroup overview Before, the forced servicegroup_name paramemter may interfere with an existing servicegroup_name filter. --- .../application/views/scripts/list/servicegroups.phtml | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/modules/monitoring/application/views/scripts/list/servicegroups.phtml b/modules/monitoring/application/views/scripts/list/servicegroups.phtml index 1e4103f6a..431fc8e56 100644 --- a/modules/monitoring/application/views/scripts/list/servicegroups.phtml +++ b/modules/monitoring/application/views/scripts/list/servicegroups.phtml @@ -38,9 +38,12 @@ if (! $this->compact): ?> qlink( $serviceGroup->servicegroup_alias, - $this->url('monitoring/list/services')->addFilter($this->filterEditor->getFilter()), - array('servicegroup_name' => $serviceGroup->servicegroup_name, 'sort' => 'service_severity'), - array('title' => sprintf($this->translate('List all services in the group "%s"'), $serviceGroup->servicegroup_alias)) + $this + ->url('monitoring/list/services') + ->setParams(['servicegroup_name' => $serviceGroup->servicegroup_name]) + ->addFilter($this->filterEditor->getFilter()), + ['sort' => 'service_severity'], + ['title' => sprintf($this->translate('List all services in the group "%s"'), $serviceGroup->servicegroup_alias)] ) ?>