From 7e81b00a9669cb18212b615c45fec7c8e970a4ec Mon Sep 17 00:00:00 2001
From: Matthias Jentsch <matthias.jentsch@netways.de>
Date: Wed, 30 Sep 2015 16:48:19 +0200
Subject: [PATCH] Fix multi-view badge filters

Allow providing a base URL for StateBadges to display hosts and services based on a filter.

refs #5543
---
 library/Icinga/Web/Url.php                    | 16 ++++++++
 .../list/components/hostssummary.phtml        |  1 +
 .../list/components/servicesummary.phtml      |  1 +
 .../Monitoring/Web/Widget/StateBadges.php     | 37 ++++++++++++++++++-
 4 files changed, 54 insertions(+), 1 deletion(-)

diff --git a/library/Icinga/Web/Url.php b/library/Icinga/Web/Url.php
index aa7f652a30..9f31447ea5 100644
--- a/library/Icinga/Web/Url.php
+++ b/library/Icinga/Web/Url.php
@@ -207,6 +207,22 @@ public static function urlAddFilterOptional($url, $filter, $optional)
         return $url->setQueryString($f->toQueryString());
     }
 
+    /**
+     * Set the new Filter of the url to be the current filter and the given filter
+     *
+     * @param Filter $and
+     */
+    public function addFilter($and)
+    {
+        $this->setQueryString(
+            Filter::matchAll(
+                $and,
+                Filter::fromQueryString($this->getQueryString())
+            )->toQueryString()
+        );
+        return $this;
+    }
+
     /**
      * Overwrite the baseUrl
      *
diff --git a/modules/monitoring/application/views/scripts/list/components/hostssummary.phtml b/modules/monitoring/application/views/scripts/list/components/hostssummary.phtml
index 058620dc42..68d67b8827 100644
--- a/modules/monitoring/application/views/scripts/list/components/hostssummary.phtml
+++ b/modules/monitoring/application/views/scripts/list/components/hostssummary.phtml
@@ -19,6 +19,7 @@ if (! $stats instanceof stdClass) {
 <?php
 $stateBadges = new StateBadges();
 $stateBadges
+    ->setBaseFilter(isset($baseFilter) ? $baseFilter : null)
     ->setUrl('monitoring/list/hosts')
     ->add(
         StateBadges::STATE_UP,
diff --git a/modules/monitoring/application/views/scripts/list/components/servicesummary.phtml b/modules/monitoring/application/views/scripts/list/components/servicesummary.phtml
index 50dbf0dd2d..1f3cfa402d 100644
--- a/modules/monitoring/application/views/scripts/list/components/servicesummary.phtml
+++ b/modules/monitoring/application/views/scripts/list/components/servicesummary.phtml
@@ -22,6 +22,7 @@ if (! $stats instanceof stdClass) {
 <?php
 $stateBadges = new StateBadges();
 $stateBadges
+    ->setBaseFilter(isset($baseFilter) ? $baseFilter : null)
     ->setUrl('monitoring/list/services')
     ->add(
         StateBadges::STATE_OK,
diff --git a/modules/monitoring/library/Monitoring/Web/Widget/StateBadges.php b/modules/monitoring/library/Monitoring/Web/Widget/StateBadges.php
index 73d4169d66..1feb29d003 100644
--- a/modules/monitoring/library/Monitoring/Web/Widget/StateBadges.php
+++ b/modules/monitoring/library/Monitoring/Web/Widget/StateBadges.php
@@ -8,6 +8,7 @@
 use Icinga\Web\Navigation\NavigationItem;
 use Icinga\Web\Url;
 use Icinga\Web\Widget\AbstractWidget;
+use Icinga\Data\Filter\Filter;
 
 class StateBadges extends AbstractWidget
 {
@@ -123,6 +124,13 @@ class StateBadges extends AbstractWidget
      */
     protected $priority = 1;
 
+    /**
+     * The base filter applied to any badge link
+     *
+     * @var Filter
+     */
+    protected $baseFilter;
+
     /**
      * Base URL
      *
@@ -156,6 +164,29 @@ public function setUrl($url)
         return $this;
     }
 
+    /**
+     * Get the base filter
+     *
+     * @return  Filter
+     */
+    public function getBaseFilter()
+    {
+        return $this->baseFilter;
+    }
+
+    /**
+     * Set the base filter
+     *
+     * @param   Filter $baseFilter
+     *
+     * @return  $this
+     */
+    public function setBaseFilter($baseFilter)
+    {
+        $this->baseFilter = $baseFilter;
+        return $this;
+    }
+
     /**
      * Add a state badge
      *
@@ -193,6 +224,10 @@ public function createBadge($state, Navigation $badges)
     {
         if ($this->has($state)) {
             $badge = $this->get($state);
+            $url = clone $this->url->setParams($badge->filter);
+            if (isset($this->baseFilter)) {
+                $url->addFilter($this->baseFilter);
+            }
             $badges->addItem(new NavigationItem($state, array(
                 'attributes'    => array('class' => 'badge ' . $state),
                 'label'         => $badge->count,
@@ -201,7 +236,7 @@ public function createBadge($state, Navigation $badges)
                     mtp('monitoring', $badge->translateSingular, $badge->translatePlural, $badge->count),
                     $badge->translateArgs
                 ),
-                'url'           => clone $this->url->setParams($badge->filter)
+                'url'           => $url
             )));
         }
         return $this;