bug #34029 [Mailer][MailchimpBridge] Fix NamedAddress obsolete paths (ogizanagi)

fabpot · fabpot · commit d6b567d8ad06 · 2019-10-18T17:33:35.000+02:00
This PR was merged into the 4.4 branch. Discussion ---------- [Mailer][MailchimpBridge] Fix NamedAddress obsolete paths | Q | A | ------------- | --- | Branch? | 4.4  | Bug fix? | yes | New feature? | no  | Deprecations? | no  | Tickets | N/A  | License | MIT | Doc PR | N/A As NamedAddress was removed in #33270 (4.4) and this bridge does require `symfony/mailer >= 4.4`. Commits ------- f14d082 [Mailer][MailchimpBridge] Fix NamedAddress obsolete paths
diff --git a/src/Symfony/Component/Mailer/Bridge/Mailchimp/Transport/MandrillApiTransport.php b/src/Symfony/Component/Mailer/Bridge/Mailchimp/Transport/MandrillApiTransport.php
@@ -17,7 +17,6 @@
 use Symfony\Component\Mailer\SentMessage;
 use Symfony\Component\Mailer\Transport\AbstractApiTransport;
 use Symfony\Component\Mime\Email;
-use Symfony\Component\Mime\NamedAddress;
 use Symfony\Contracts\EventDispatcher\EventDispatcherInterface;
 use Symfony\Contracts\HttpClient\HttpClientInterface;
 use Symfony\Contracts\HttpClient\ResponseInterface;
@@ -76,15 +75,12 @@ private function getPayload(Email $email, Envelope $envelope): array
                 'html' => $email->getHtmlBody(),
                 'text' => $email->getTextBody(),
                 'subject' => $email->getSubject(),
+                'from_name' => $envelope->getSender()->getName(),
                 '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');
@@ -126,13 +122,10 @@ protected function getRecipients(Email $email, Envelope $envelope): array
 
             $recipientPayload = [
                 'email' => $recipient->getAddress(),
+                'name' => $recipient->getName(),
                 'type' => $type,
             ];
 
-            if ($recipient instanceof NamedAddress) {
-                $recipientPayload['name'] = $recipient->getName();
-            }
-
             $recipients[] = $recipientPayload;
         }
 
