From 437939736fadb19de9f754ab6e7580d11ad1c583 Mon Sep 17 00:00:00 2001 From: Pepijn Over Date: Mon, 24 Mar 2014 20:15:53 +0100 Subject: [PATCH] bug in status updater causing it to log all status checks instead of when notification settings are met --- src/psm/Util/Updater/StatusNotifier.class.php | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/psm/Util/Updater/StatusNotifier.class.php b/src/psm/Util/Updater/StatusNotifier.class.php index e41c7480..0d07ad43 100755 --- a/src/psm/Util/Updater/StatusNotifier.class.php +++ b/src/psm/Util/Updater/StatusNotifier.class.php @@ -144,6 +144,10 @@ class StatusNotifier { break; } + if(!$notify) { + return false; + } + // first add to log (we use the same text as the SMS message because its short..) if($this->save_logs) { psm_add_log( @@ -153,10 +157,6 @@ class StatusNotifier { ); } - if(!$notify) { - return false; - } - // check if email is enabled for this server if($this->send_emails && $this->server['email'] == 'yes') { // send email