Deduplicate servicegroups list view's logic

refs #6778
Icinga · Nov 27, 2014 · 47f96ff · 47f96ff
1 parent 7d7663a
commit 47f96ff
Showing 1 changed file with 22 additions and 48 deletions.
diff --git a/modules/monitoring/application/views/scripts/list/servicegroups.phtml b/modules/monitoring/application/views/scripts/list/servicegroups.phtml
@@ -29,55 +29,29 @@
     <tbody>
     <?php foreach ($servicegroups as $s): ?>
       <tr href="<?= $this->href('monitoring/list/services', array('servicegroup' => $s->servicegroup)); ?>">
-       <?php if ($s->services_critical_last_state_change_unhandled): ?>
-        <td class="state change critical unhandled">
-          <strong><?= $this->translate('CRITICAL'); ?></strong>
+       <?php
+       foreach (array(
+           array('critical', '_handled',   '',           $this->translate('CRITICAL')),
+           array('critical', '_unhandled', ' unhandled', $this->translate('CRITICAL')),
+           array('warning',  '_handled',   '',           $this->translate('WARNING')),
+           array('warning',  '_unhandled', ' unhandled', $this->translate('WARNING')),
+           array('unknown',  '_handled',   '',           $this->translate('UNKNOWN')),
+           array('unknown',  '_unhandled', ' unhandled', $this->translate('UNKNOWN')),
+           array('ok',       '',           '',           $this->translate('OK')),
+           array('pending',  '',           '',           $this->translate('PENDING'))
+       ) as $combinations):
+           $state = $combinations[0];
+           $time = $s->{'services_' . $state . '_last_state_change' . $combinations[1]};
+           if ($time): ?>
+        <td class="state change <?= $state . $combinations[2] ?>">
+          <strong><?= $combinations[3] ?></strong>
           <br>
-          <?= $this->prefixedTimeSince($s->services_critical_last_state_change_unhandled); ?>
+          <?= $this->prefixedTimeSince($time) ?>
         </td>
-       <?php elseif ($s->services_unknown_last_state_change_unhandled): ?>
-        <td class="state change unknown unhandled">
-          <strong><?= $this->translate('UNKNOWN'); ?></strong>
-          <br>
-          <?= $this->prefixedTimeSince($s->services_unknown_last_state_change_unhandled); ?>
-        </td>
-       <?php elseif ($s->services_warning_last_state_change_unhandled): ?>
-        <td class="state change warning unhandled">
-          <strong><?= $this->translate('WARNING'); ?></strong>
-          <br>
-          <?= $this->prefixedTimeSince($s->services_warning_last_state_change_unhandled); ?>
-        </td>
-       <?php elseif ($s->services_critical_last_state_change_handled): ?>
-        <td class="state change critical">
-          <strong><?= $this->translate('CRITICAL'); ?></strong>
-          <br>
-          <?= $this->prefixedTimeSince($s->services_critical_last_state_change_handled); ?>
-        </td>
-       <?php elseif ($s->services_unknown_last_state_change_handled): ?>
-        <td class="state change unknown">
-          <strong><?= $this->translate('UNKNOWN'); ?></strong>
-          <br>
-          <?= $this->prefixedTimeSince($s->services_unknown_last_state_change_handled); ?>
-        </td>
-       <?php elseif ($s->services_warning_last_state_change_handled): ?>
-        <td class="state change warning">
-          <strong><?= $this->translate('WARNING'); ?></strong>
-          <br>
-          <?= $this->prefixedTimeSince($s->services_warning_last_state_change_handled); ?>
-        </td>
-       <?php elseif ($s->services_ok_last_state_change): ?>
-        <td class="state change ok">
-          <strong><?= $this->translate('OK'); ?></strong>
-          <br>
-          <?= $this->prefixedTimeSince($s->services_ok_last_state_change); ?>
-        </td>
-       <?php else: ?>
-        <td class="state change pending">
-          <strong><?= $this->translate('PENDING'); ?></strong>
-          <br>
-          <?= $this->prefixedTimeSince($s->services_pending_last_state_change); ?>
-        </td>
-       <?php endif ?>
+               <?php
+               break;
+           endif;
+       endforeach; ?>
         <td class="groupname">
           <a href="<?= $this->href('monitoring/list/services', array('servicegroup' => $s->servicegroup)); ?>">
             <?= $s->servicegroup; ?>
@@ -222,4 +196,4 @@
     <?php endforeach ?>
     </tbody>
   </table>
-</div>
+</div>