Use InvalidArgumentException if an invalid participant is given

FriendsOfSymfony · Dec 7, 2011 · 33bec9b · 33bec9b
1 parent 4866719
commit 33bec9b
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/Entity/Message.php b/Entity/Message.php
@@ -44,12 +44,12 @@ public function isReadByParticipant(ParticipantInterface $participant)
      *
      * @param ParticipantInterface $participant
      * @param boolean $isRead
+     * @throws InvalidArgumentException if no metadata exists for the participant
      */
     public function setIsReadByParticipant(ParticipantInterface $participant, $isRead)
     {
-        $meta = $this->getMetadataForParticipant($participant);
-        if (!$meta) {
-            throw new \Exception(sprintf('No metadata setted for participant with id "%s"', $participant->getId()));
+        if (!$meta = $this->getMetadataForParticipant($participant)) {
+            throw new \InvalidArgumentException(sprintf('No metadata exists for participant with id "%s"', $participant->getId()));
         }
 
         $meta->setIsRead($isRead);