[Mailer] Fix Mandrill Transport API payload with named addresses

Michaël Perrin · Michaël Perrin · commit 6dbac13a07c2 · 2019-10-15T15:11:00.000-05:00
diff --git a/src/Symfony/Component/Mailer/Bridge/Mailchimp/Http/Api/MandrillTransport.php b/src/Symfony/Component/Mailer/Bridge/Mailchimp/Http/Api/MandrillTransport.php
@@ -17,6 +17,7 @@
 use Symfony\Component\Mailer\SmtpEnvelope;
 use Symfony\Component\Mailer\Transport\Http\Api\AbstractApiTransport;
 use Symfony\Component\Mime\Email;
+use Symfony\Component\Mime\NamedAddress;
 use Symfony\Contracts\HttpClient\HttpClientInterface;
 
 /**
@@ -60,11 +61,15 @@ private function getPayload(Email $email, SmtpEnvelope $envelope): array
                 'html' => $email->getHtmlBody(),
                 'text' => $email->getTextBody(),
                 'subject' => $email->getSubject(),
-                'from_email' => $envelope->getSender()->toString(),
+                'from_email' => $envelope->getSender()->getAddress(),
                 'to' => $this->getRecipients($email, $envelope),
             ],
         ];
 
+        if ($envelope->getSender() instanceof NamedAddress) {
+            $payload['message']['from_name'] = $envelope->getSender()->getName();
+        }
+
         foreach ($email->getAttachments() as $attachment) {
             $headers = $attachment->getPreparedHeaders();
             $disposition = $headers->getHeaderBody('Content-Disposition');
@@ -104,10 +109,16 @@ protected function getRecipients(Email $email, SmtpEnvelope $envelope): array
                 $type = 'cc';
             }
 
-            $recipients[] = [
-                'email' => $recipient->toString(),
+            $recipientPayload = [
+                'email' => $recipient->getAddress(),
                 'type' => $type,
             ];
+
+            if ($recipient instanceof NamedAddress) {
+                $recipientPayload['name'] = $recipient->getName();
+            }
+
+            $recipients[] = $recipientPayload;
         }
 
         return $recipients;
