From 73d674f6d91d5903bc6d9138fa24edfa2532f33c Mon Sep 17 00:00:00 2001
From: Eric Lippmann <eric.lippmann@netways.de>
Date: Mon, 28 Sep 2015 17:40:08 +0200
Subject: [PATCH] monitoring: Fix thead creation in the service groups overview

refs #5543
---
 .../views/scripts/list/servicegroups.phtml    | 28 ++++++++-----------
 1 file changed, 12 insertions(+), 16 deletions(-)

diff --git a/modules/monitoring/application/views/scripts/list/servicegroups.phtml b/modules/monitoring/application/views/scripts/list/servicegroups.phtml
index e5e95ab46..ce3a36b8a 100644
--- a/modules/monitoring/application/views/scripts/list/servicegroups.phtml
+++ b/modules/monitoring/application/views/scripts/list/servicegroups.phtml
@@ -8,22 +8,22 @@
 </div>
 <?php endif ?>
 <div class="content">
+<?php if ($servicegroups->hasResult()): ?>
+  <p><?= $this->translate('No servicegroups found matching the filter') ?></p>
+</div>
+<?php endif ?>
+<table class="groupview action" data-base-target="_next">
+  <thead>
+  <th><?= $this->translate('Last Problem') ?></th>
+  <th><?= $this->translate('Service Group') ?></th>
+  <th><?= $this->translate('Total Services') ?></th>
+  <th><?= $this->translate('Service States') ?></th>
+  </thead>
+  <tbody>
 <?php
 
 $servicegroups->peekAhead($this->compact);
-$firstRow = true;
 foreach ($servicegroups as $s): ?>
-<?php if ($firstRow): ?>
-  <?php $firstRow = false ?>
-  <table class="groupview action" data-base-target="_next">
-    <thead>
-      <th><?= $this->translate('Last Problem') ?></th>
-      <th><?= $this->translate('Service Group') ?></th>
-      <th><?= $this->translate('Total Services') ?></th>
-      <th><?= $this->translate('Service States') ?></th>
-    </thead>
-    <tbody>
-<?php endif ?>
       <tr href="<?= $this->href('monitoring/list/services', array('servicegroup_name' => $s->servicegroup_name)) ?>">
        <?php if ($s->services_critical_last_state_change_unhandled): ?>
         <td class="state change critical unhandled">
@@ -299,7 +299,6 @@ foreach ($servicegroups as $s): ?>
         </td>
       </tr>
 <?php endforeach ?>
-<?php if ($servicegroups->hasResult()): ?>
     </tbody>
   </table>
 <?php if ($servicegroups->hasMore()): ?>
@@ -313,7 +312,4 @@ foreach ($servicegroups as $s): ?>
     )
   ) ?>
 <?php endif ?>
-<?php else: ?>
-  <?= $this->translate('No servicegroups found matching the filter') ?>
-<?php endif ?>
 </div>