Merge pull request #35 from narendravaghela/3.x-fixsender

Adding sender header and fixing var names

.gitignore

@@ -1,3 +1,5 @@
 /composer.lock
 /phpunit.xml
 /vendor
+/.phpunit.result.cache
+/clover.xml

src/Mailer/Transport/MailgunTransport.php

@@ -333,6 +333,10 @@ protected function _prepareEmailAddresses(Email $email)
             $this->_formData->add('from', sprintf("%s <%s>", key($from), key($from)));
         }
 
+        foreach ($email->getSender() as $senderEmail => $senderName) {
+            $this->_formData->add('h:Sender', sprintf("%s <%s>", $senderName, $senderEmail));
+        }
+
         foreach ($email->getTo() as $toEmail => $toName) {
             $this->_formData->add('to', sprintf("%s <%s>", $toName, $toEmail));
         }

tests/TestCase/Mailer/Transport/MailgunTransportTest.php

@@ -86,6 +86,7 @@ public function testAdditionalEmailAddresses()
             ->addCC(['[email protected]', '[email protected]' => 'John'])
             ->addBcc(['[email protected]', '[email protected]' => 'John'])
             ->setReplyTo(['[email protected]' => 'John'])
+            ->setSender(['[email protected]' => 'John'])
             ->setEmailFormat('both')
             ->setSubject('Email from CakePHP Mailgun plugin')
             ->send('Hello there, <br> This is an email from CakePHP Mailgun Email plugin.');
@@ -101,13 +102,15 @@ public function testAdditionalEmailAddresses()
         $this->assertTextContains('Content-Disposition: form-data; name="cc"', $reqDataString);
         $this->assertTextContains('Content-Disposition: form-data; name="bcc"', $reqDataString);
         $this->assertTextContains('Content-Disposition: form-data; name="h:Reply-To"', $reqDataString);
+        $this->assertTextContains('Content-Disposition: form-data; name="h:Sender"', $reqDataString);
         $this->assertTextContains('[email protected] <[email protected]>', $reqDataString);
         $this->assertTextContains('[email protected] <[email protected]>', $reqDataString);
         $this->assertTextContains('[email protected] <[email protected]>', $reqDataString);
         $this->assertTextContains('John <[email protected]>', $reqDataString);
         $this->assertTextContains('[email protected] <[email protected]>', $reqDataString);
         $this->assertTextContains('John <[email protected]>', $reqDataString);
         $this->assertTextContains('John <[email protected]>', $reqDataString);
+        $this->assertTextContains('John <[email protected]>', $reqDataString);
     }
 
     public function testCustomHeaders()