Add ThreadInterface::getFirstMessage

merk · Jun 29, 2011 · 3a4f6c2 · 3a4f6c2
1 parent 65ee102
commit 3a4f6c2
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 6 deletions.
diff --git a/Document/Thread.php b/Document/Thread.php
@@ -119,7 +119,7 @@ public function getCreatedBy()
      * Sets the participant that created the thread
      * Generally the sender of the first message
      *
-     * @parm ParticipantInterface
+     * @param ParticipantInterface
      */
     public function setCreatedBy(ParticipantInterface $participant)
     {
@@ -241,13 +241,11 @@ protected function doCreatedByAndAt()
         if (isset($this->createdBy)) {
             return;
         }
-        $messages = $this->getMessages();
-        if (empty($messages)) {
+        if (!$message = $this->getFirstMessage()) {
             return;
         }
-        $firstMessage = reset($messages);
-        $thread->setCreatedBy($firstMessage->getSender());
-        $thread->setCreatedAt($firstMessage->getCreatedAt());
+        $this->setCreatedBy($message->getSender());
+        $this->setCreatedAt($message->getCreatedAt());
     }
 
     /**

diff --git a/Model/Thread.php b/Model/Thread.php
@@ -106,6 +106,21 @@ public function setIsReadByParticipant(ParticipantInterface $participant, $isRea
         }
     }
 
+    /**
+     * Gets the first message of the thread
+     *
+     * @return MessageInterface the first message
+     */
+    public function getFirstMessage()
+    {
+        $messages = $this->getMessages();
+        if(empty($messages)) {
+            return null;
+        }
+
+        return reset($messages);
+    }
+
     /**
      * Gets the last message of the thread
      *

diff --git a/Model/ThreadInterface.php b/Model/ThreadInterface.php
@@ -38,6 +38,13 @@ function getMessages();
      */
     function addMessage(MessageInterface $message);
 
+    /**
+     * Gets the first message of the thread
+     *
+     * @return MessageInterface the first message
+     */
+    function getFirstMessage();
+
     /**
      * Gets the last message of the thread
      *