Various fixes for PHPAnalyzer

2 files changed, 3 insertions, 1 deletions

diff --git a/app/Subscriber/Base.php b/app/Subscriber/Base.php
index f90d9604..97e230b5 100644
--- a/app/Subscriber/Base.php
+++ b/app/Subscriber/Base.php
@@ -10,6 +10,8 @@ use Pimple\Container;
  * @package  subscriber
  * @author   Frederic Guillot
  *
+ * @property \Integration\SlackWebhook     $slackWebhook
+ * @property \Integration\Hipchat          $hipchat
  * @property \Model\Board                  $board
  * @property \Model\Config                 $config
  * @property \Model\Comment                $comment
diff --git a/app/Subscriber/NotificationSubscriber.php b/app/Subscriber/NotificationSubscriber.php
index 1b7187fe..94815b56 100644
--- a/app/Subscriber/NotificationSubscriber.php
+++ b/app/Subscriber/NotificationSubscriber.php
@@ -49,7 +49,7 @@ class NotificationSubscriber extends Base implements EventSubscriberInterface
         $values = $this->getTemplateData($event);
         $users = $this->notification->getUsersList($values['task']['project_id']);
 
-        if ($users) {
+        if (! empty($users)) {
             $this->notification->sendEmails($this->templates[$event_name], $users, $values);
         }
     }