From e197907323da3dbe6f104b281bf0cc99e8c41ddf Mon Sep 17 00:00:00 2001
From: Eric Lippmann <eric.lippmann@netways.de>
Date: Thu, 24 Sep 2015 15:11:20 +0200
Subject: [PATCH] monitoring: Fix stupid thead creation in the lit/hostgroups
 view script

Drunk?

refs #5543
---
 .../views/scripts/list/hostgroups.phtml       | 32 +++++++------------
 1 file changed, 12 insertions(+), 20 deletions(-)

diff --git a/modules/monitoring/application/views/scripts/list/hostgroups.phtml b/modules/monitoring/application/views/scripts/list/hostgroups.phtml
index 93eeecdb3..d6f048810 100644
--- a/modules/monitoring/application/views/scripts/list/hostgroups.phtml
+++ b/modules/monitoring/application/views/scripts/list/hostgroups.phtml
@@ -11,25 +11,21 @@ if (! $this->compact): ?>
 </div>
 <?php endif ?>
 <div class="content">
-<?php
-
-$firstRow = true;
-foreach ($hostgroups as $hostgroup): ?>
-<?php if ($firstRow): ?>
-  <?php $firstRow = false ?>
-  <table class="groupview action" data-base-target="_next">
+<?php if (! $hostgroups->hasResult()): ?>
+    <p><?= $this->translate('No host groups found matching the filter.') ?></p>
+    </div>
+<?php return; endif ?>
+<table class="groupview action" data-base-target="_next">
     <thead>
-      <th><?= $this->translate('Last Problem') ?></th>
-      <th><?= $this->translate('Host Group') ?></th>
-      <th><?= $this->translate('Total Hosts') ?></th>
-      <th><?= $this->translate('Host States') ?></th>
-      <th><?= $this->translate('Total Services') ?></th>
-      <th><?= $this->translate('Service States') ?></th>
+        <th><?= $this->translate('Last Problem') ?></th>
+        <th><?= $this->translate('Host Group') ?></th>
+        <th><?= $this->translate('Total Hosts') ?></th>
+        <th><?= $this->translate('Host States') ?></th>
+        <th><?= $this->translate('Total Services') ?></th>
+        <th><?= $this->translate('Service States') ?></th>
     </thead>
     <tbody>
-<?php endif ?>
-<?php
-
+<?php foreach ($hostgroups as $hostgroup):
 if ($hostgroup->hosts_down_unhandled) {
     $handled = false;
     $state = Host::STATE_DOWN;
@@ -464,7 +460,6 @@ if ($hostgroup->hosts_down_unhandled) {
         </td>
       </tr>
 <?php endforeach ?>
-<?php if ($hostgroups->hasResult()): ?>
     </tbody>
   </table>
 <?php if ($hostgroups->hasMore()): ?>
@@ -478,7 +473,4 @@ if ($hostgroup->hosts_down_unhandled) {
     )
   ) ?>
 <?php endif ?>
-<?php else: ?>
-  <?= $this->translate('No hostgroups found matching the filter') ?>
-<?php endif ?>
 </div>