Fix setting Reply-To multiple times (#300)

This commit is contained in:
z38 2017-03-14 16:51:33 +01:00 committed by Tobias Nyholm
parent 83a20afbfb
commit b3f24e9d7a
2 changed files with 4 additions and 3 deletions

View File

@ -117,10 +117,10 @@ class MessageBuilder
{
$compiledAddress = $this->parseAddress($address, $variables);
if (isset($this->message[$headerName])) {
array_push($this->message[$headerName], $compiledAddress);
} elseif ($headerName == 'h:reply-to') {
if ($headerName === 'h:reply-to') {
$this->message[$headerName] = $compiledAddress;
} elseif (isset($this->message[$headerName])) {
array_push($this->message[$headerName], $compiledAddress);
} else {
$this->message[$headerName] = [$compiledAddress];
}

View File

@ -114,6 +114,7 @@ class MessageBuilderTest extends \Mailgun\Tests\MailgunTestCase
public function testSetReplyTo()
{
$message = $this->client->MessageBuilder();
$message->setReplyToAddress('overwritten@samples.mailgun.org');
$message->setReplyToAddress('test@samples.mailgun.org', ['first' => 'Test', 'last' => 'User']);
$messageObj = $message->getMessage();
$this->assertEquals(['h:reply-to' => "'Test User' <test@samples.mailgun.org>"], $messageObj);