From c4c5bcdd48449661afe1ad88df0e9b1ffaf51c59 Mon Sep 17 00:00:00 2001
From: Marius Hein <marius.hein@netways.de>
Date: Thu, 28 Aug 2014 13:39:49 +0200
Subject: [PATCH 1/2] ContactQuery: Change timeperiod joins to joinLeft()

refs #7020
---
 .../views/scripts/list/contacts.phtml         |  4 ++++
 .../views/scripts/show/contact.phtml          | 21 ++++++++++++-------
 .../Backend/Ido/Query/ContactQuery.php        |  4 ++--
 3 files changed, 20 insertions(+), 9 deletions(-)

diff --git a/modules/monitoring/application/views/scripts/list/contacts.phtml b/modules/monitoring/application/views/scripts/list/contacts.phtml
index 9fc05de8e..e4647e95b 100644
--- 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
index 45d751fd5..a2e720ec9 100644
--- 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
index 89fa37fcd..467e40fc0 100644
--- a/modules/monitoring/library/Monitoring/Backend/Ido/Query/ContactQuery.php
+++ b/modules/monitoring/library/Monitoring/Backend/Ido/Query/ContactQuery.php
@@ -95,12 +95,12 @@ class ContactQuery extends IdoQuery
 
     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()

From 0a2b504db85a8be8609e17cad31eef346c82f995 Mon Sep 17 00:00:00 2001
From: Marius Hein <marius.hein@netways.de>
Date: Thu, 28 Aug 2014 13:41:21 +0200
Subject: [PATCH 2/2] Gracatar: Pre sized with 60px because of display latency

refs #7020
---
 .../monitoring/application/views/scripts/list/contacts.phtml    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/monitoring/application/views/scripts/list/contacts.phtml b/modules/monitoring/application/views/scripts/list/contacts.phtml
index e4647e95b..a31a79553 100644
--- a/modules/monitoring/application/views/scripts/list/contacts.phtml
+++ b/modules/monitoring/application/views/scripts/list/contacts.phtml
@@ -18,7 +18,7 @@ $contactHelper = $this->getHelper('ContactFlags');
     }
     foreach ($contacts as $contact): ?>
         <div class="contact">
-            <img src="<?= $this->href('/static/gravatar', array('email' => $contact->contact_email )) ?>" />
+            <img style="width: 60px; height: 60px;" src="<?= $this->href('/static/gravatar', array('email' => $contact->contact_email )) ?>" />
             <a href="<?= $this->href(
                 'monitoring/show/contact',
                 array('contact' => $contact->contact_name)