Improve multi-view tabs

Display the current object count next to the tab title, to clarify that those views display a summary of multiple items. Only display the current tab for hosts or services, to avoid confusion.

refs #8565

modules/monitoring/application/controllers/HostsController.php

@@ -15,7 +15,6 @@ use Icinga\Module\Monitoring\Forms\Command\Object\AddCommentCommandForm;
 use Icinga\Module\Monitoring\Forms\Command\Object\SendCustomNotificationCommandForm;
 use Icinga\Module\Monitoring\Object\HostList;
 use Icinga\Web\Url;
-use Icinga\Web\Widget\Chart\InlinePie;
 use Icinga\Web\Widget\Tabextension\DashboardAction;
 
 class Monitoring_HostsController extends Controller
@@ -42,23 +41,11 @@ class Monitoring_HostsController extends Controller
                     $this->translate('Show summarized information for %u hosts'),
                     count($this->hostList)
                 ),
-                'label' => $this->translate('Hosts'),
+                'label' => sprintf($this->translate('Hosts (%d)'), count($this->hostList)),
                 'url'   => Url::fromRequest(),
                 'icon'  => 'host'
             )
-        )->extend(new DashboardAction());
+        )->extend(new DashboardAction())->activate('show');
-
-        $this->getTabs()->add(
-            'services',
-            array(
-                'title' => sprintf(
-                    $this->translate('Show summarized information for related services')
-                ),
-                'label' => $this->translate('Services'),
-                'url'   => Url::fromPath('monitoring/services/show')->setParams(Url::fromRequest()->getParams()),
-                'icon'  => 'services'
-            )
-        )->activate('show');
         
         $this->view->listAllLink = Url::fromRequest()->setPath('monitoring/list/hosts')->setQueryString($filterString);
     }

modules/monitoring/application/controllers/ServicesController.php

@@ -15,7 +15,6 @@ use Icinga\Module\Monitoring\Forms\Command\Object\DeleteCommentCommandForm;
 use Icinga\Module\Monitoring\Forms\Command\Object\SendCustomNotificationCommandForm;
 use Icinga\Module\Monitoring\Object\ServiceList;
 use Icinga\Web\Url;
-use Icinga\Web\Widget\Chart\InlinePie;
 use Icinga\Web\Widget\Tabextension\DashboardAction;
 
 class Monitoring_ServicesController extends Controller
@@ -34,18 +33,6 @@ class Monitoring_ServicesController extends Controller
         $this->serviceList = $serviceList;
         $this->view->listAllLink = Url::fromRequest()->setPath('monitoring/list/services');
 
-        $this->getTabs()->add(
-            'hosts',
-            array(
-                'title' => sprintf(
-                    $this->translate('Show summarized information for hosts')
-                ),
-                'label' => $this->translate('Hosts'),
-                'url'   => Url::fromPath('monitoring/hosts/show')->setParams(Url::fromRequest()->getParams()),
-                'icon' => 'host'
-            )
-        )->extend(new DashboardAction());
-
         $this->getTabs()->add(
             'show',
             array(
@@ -53,11 +40,11 @@ class Monitoring_ServicesController extends Controller
                     $this->translate('Show summarized information for %u services'),
                     count($this->serviceList)
                 ),
-                'label' => $this->translate('Services'),
+                'label' => sprintf($this->translate('Services (%d)'), count($this->serviceList)),
                 'url'   => Url::fromRequest(),
                 'icon'  => 'services'
             )
-        )->activate('show');
+        )->extend(new DashboardAction())->activate('show');
     }
 
     protected function handleCommandForm(ObjectsCommandForm $form)

modules/monitoring/library/Monitoring/Object/HostList.php

@@ -76,7 +76,7 @@ class HostList extends ObjectList
     /**
      * Returns a Filter that matches all hosts in this list
      *
-     * @return array    An
+     * @return Filter
      */
     public function objectsFilter($columns = array('host' => 'host'))
     {