Merge pull request #3 from ngodfraind/master

Notification works with anon.
iCAPLyon1 · Feb 7, 2014 · c047b92 · c047b92
2 parents ee098ad + fbd6d3f
commit c047b92
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/Service/Manager.php b/Service/Manager.php
@@ -129,7 +129,8 @@ public function createNotification(NotifiableInterface $notifiable)
                 $doer = $securityToken->getUser();
             }
         }
-        if ($doer !== null) {
+
+        if ($doer instanceof Claroline\CoreBundle\Entity\User) {
             $doerId = $doer->getId();
         }
 
@@ -188,8 +189,10 @@ public function notifyUsers(Notification $notification, NotifiableInterface $not
             $loggedUser = $token->getUser();
             $removeUserIds = array($notification->getUserId());
 
-            if (!empty($loggedUser) && $loggedUser->getId() != $notification->getUserId()) {
-                array_push($removeUserIds, $loggedUser->getId());
+            if ($loggedUser instanceof Claroline\CoreBundle\Entity\User) {
+                if (!empty($loggedUser) && $loggedUser->getId() != $notification->getUserId()) {
+                    array_push($removeUserIds, $loggedUser->getId());
+                }
             }
         }