Message: From header can be absent

ddeboer · Oct 12, 2017 · 93a150c · 93a150c
1 parent 63d472b
commit 93a150c
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 4 deletions.
diff --git a/src/Message/AbstractMessage.php b/src/Message/AbstractMessage.php
@@ -26,9 +26,9 @@ final public function getId(): string
     /**
      * Get message sender (from headers).
      *
-     * @return EmailAddress
+     * @return null|EmailAddress
      */
-    final public function getFrom(): EmailAddress
+    final public function getFrom()
     {
         return $this->getHeaders()->get('from');
     }

diff --git a/src/Message/BasicMessageInterface.php b/src/Message/BasicMessageInterface.php
@@ -39,9 +39,9 @@ public function getId(): string;
     /**
      * Get message sender (from headers).
      *
-     * @return EmailAddress
+     * @return null|EmailAddress
      */
-    public function getFrom(): EmailAddress;
+    public function getFrom();
 
     /**
      * Get To recipients.

diff --git a/tests/MessageTest.php b/tests/MessageTest.php
@@ -650,6 +650,14 @@ public function testInlineAttachment()
         $this->assertNull($inline->getFilename());
     }
 
+    public function testMissingFromHeader()
+    {
+        $this->mailbox->addMessage($this->getFixture('missing_from'));
+        $message = $this->mailbox->getMessage(1);
+
+        $this->assertNull($message->getFrom());
+    }
+
     public function testAttachmentMustNotBeCharsetDecoded()
     {
         $parts = [];