Merge branch 'hotfix/36'

Close #36

CHANGELOG.md

@@ -22,7 +22,10 @@ All notable changes to this project will be documented in this file, in reverse
 
 ### Fixed
 
-- Nothing.
+- [#36](https://github.com/zendframework/zend-mime/pull/36) fixes
+  `Zend\Mime\Decode::splitMessage` to set `Zend\Mail\Headers`
+  instance always for `$headers` parameter. Before, when messages
+  without headers was provided, `$headers` was an empty array.
 
 ## 2.7.1 - 2018-05-14
 

src/Decode.php

@@ -108,7 +108,7 @@ public static function splitMessage($message, &$headers, &$body, $EOL = Mime::LI
         $firstlinePos = strpos($message, "\n");
         $firstline = $firstlinePos === false ? $message : substr($message, 0, $firstlinePos);
         if (! preg_match('%^[^\s]+[^:]*:%', $firstline)) {
-            $headers = [];
+            $headers = new Headers();
             // TODO: we're ignoring \r for now - is this function fast enough and is it safe to assume noone needs \r?
             $body = str_replace(["\r", "\n"], ['', $EOL], $message);
             return;

test/DecodeTest.php

@@ -0,0 +1,25 @@
+<?php
+/**
+ * @see       https://github.com/zendframework/zend-mime for the canonical source repository
+ * @copyright Copyright (c) 2019 Zend Technologies USA Inc. (https://www.zend.com)
+ * @license   https://github.com/zendframework/zend-mime/blob/master/LICENSE.md New BSD License
+ */
+
+namespace ZendTest\Mime;
+
+use PHPUnit\Framework\TestCase;
+use Zend\Mail\Headers;
+use Zend\Mime\Decode;
+
+class DecodeTest extends TestCase
+{
+    public function testDecodeMessageWithoutHeaders()
+    {
+        $text = 'This is a message body';
+
+        Decode::splitMessage($text, $headers, $body);
+
+        self::assertInstanceOf(Headers::class, $headers);
+        self::assertSame($text, $body);
+    }
+}