ContactQuery: Change timeperiod joins to joinLeft()

refs #7020
Icinga · Aug 28, 2014 · c4c5bcd · c4c5bcd
1 parent dea7caa
commit c4c5bcd
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 9 deletions.
diff --git a/modules/monitoring/application/views/scripts/list/contacts.phtml b/modules/monitoring/application/views/scripts/list/contacts.phtml
@@ -36,14 +36,18 @@ $contactHelper = $this->getHelper('ContactFlags');
             <?php endif; ?>
             <div style="clear: both;"></div>
             <div class="notification-periods">
+                <?php if ($contact->contact_notify_service_timeperiod): ?>
                 <div>
                     <?= t('Service notification period') ?>:
                     <?= $this->escape($contact->contact_notify_service_timeperiod) ?>
                 </div>
+                <?php endif; ?>
+                <?php if ($contact->contact_notify_host_timeperiod): ?>
                 <div>
                     <?= t('Host notification period') ?>:
                     <?= $this->escape($contact->contact_notify_host_timeperiod) ?>
                 </div>
+                <?php endif; ?>
             </div>
         </div>
     <?php

diff --git a/modules/monitoring/application/views/scripts/show/contact.phtml b/modules/monitoring/application/views/scripts/show/contact.phtml
@@ -46,15 +46,22 @@ $contactHelper = $this->getHelper('ContactFlags');
             </tbody>
         </table>
 
-        <h4><?= $this->translate('Commands'); ?>:</h4>
-        <ul>
-        <?php foreach ($commands as $command): ?>
-            <li><?= $command->command_name; ?></li>
-        <?php endforeach; ?>
-        </ul>
+        <?php if (count($commands)): ?>
+            <h4><?= $this->translate('Commands'); ?>:</h4>
+            <ul>
+            <?php foreach ($commands as $command): ?>
+                <li><?= $command->command_name; ?></li>
+            <?php endforeach; ?>
+            </ul>
+        <?php endif; ?>
+
 
         <h4><?= $this->translate('Notifications'); ?>:</h4>
-        <?= $this->render('list/notifications.phtml') ?>
+        <?php if (count($notifications)): ?>
+            <?= $this->render('list/notifications.phtml') ?>
+        <?php else: ?>
+           <p><?= $this->translate('No notifications for this contact'); ?></p>
+        <?php endif; ?>
 
     <?php else: ?>
         <?= $this->translate('No such contact'); ?>: <?= $contactName; ?>

diff --git a/modules/monitoring/library/Monitoring/Backend/Ido/Query/ContactQuery.php b/modules/monitoring/library/Monitoring/Backend/Ido/Query/ContactQuery.php
@@ -95,12 +95,12 @@ protected function joinServices()
 
     protected function joinTimeperiods()
     {
-        $this->select->join(
+        $this->select->joinLeft(
             array('ht' => $this->prefix . 'timeperiods'),
             'ht.timeperiod_object_id = c.host_timeperiod_object_id',
             array()
         );
-        $this->select->join(
+        $this->select->joinLeft(
             array('st' => $this->prefix . 'timeperiods'),
             'st.timeperiod_object_id = c.service_timeperiod_object_id',
             array()