RetailCRM-Mirror/mailgun-php
1
0
Fork 0
mirror of https://github.com/retailcrm/mailgun-php.git synced 2024-11-25 22:36:06 +03:00
Code Issues Projects Releases Wiki Activity

Merge pull request #16 from travelton/BatchMessageFixing

Browse Source 
Fixed missing IDs for batch message recipients. Closes #15.
This commit is contained in:
Travis Swientek 2013-10-07 17:00:54 -07:00
commit 70cd750ace
2 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Mailgun/Messages/BatchMessage.php
View File

@ -36,7 +36,9 @@ class BatchMessage extends MessageBuilder{
}
$this->addRecipient("to", $address, $variables);
$attributes["id"] = $this->toRecipientCount;
if(!array_key_exists("id", $variables)){
$variables['id'] = $this->toRecipientCount;
}
$this->batchRecipientAttributes["$address"] = $variables;
}

10
tests/Mailgun/Tests/Messages/BatchMessageTest.php
View File

@ -59,6 +59,16 @@ class BatchMessageTest extends \Mailgun\Tests\MailgunTestCase{
$property->setAccessible(true);
$this->assertEquals(1, $property->getValue($message));
}
public function testDefaultIDInVariables() {
$message = $this->client->BatchMessage($this->sampleDomain);
$message->addToRecipient("test-user@samples.mailgun.org", array("first" => "Test", "last" => "User"));
$reflectionClass = new \ReflectionClass(get_class($message));
$property = $reflectionClass->getProperty('batchRecipientAttributes');
$property->setAccessible(True);
$propertyValue = $property->getValue($message);
$this->assertEquals(1, $propertyValue['test-user@samples.mailgun.org']['id']);
}
}
?>