bug #34034 [Mailer][MailchimpBridge] Don't send address names if empty string (ogizanagi)

This PR was merged into the 4.4 branch.

Discussion
----------

[Mailer][MailchimpBridge] Don't send address names if empty string

| Q             | A
| ------------- | ---
| Branch?       | 4.4 <!-- see below -->
| Bug fix?      | yes
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tickets       | symfony/symfony#34029 (comment) <!-- prefix each issue number with "Fix #", if any -->
| License       | MIT
| Doc PR        | N/A

Commits
-------

197d1236ee [Mailer][MailchimpBridge] Don't send address names if empty string

Author: fabpot

Mailchimp/Transport/MandrillApiTransport.php

@@ -75,12 +75,15 @@ 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()->getName()) {
+            $payload['message']['from_name'] = $envelope->getSender()->getName();
+        }
+
         foreach ($email->getAttachments() as $attachment) {
             $headers = $attachment->getPreparedHeaders();
             $disposition = $headers->getHeaderBody('Content-Disposition');
@@ -122,10 +125,13 @@ protected function getRecipients(Email $email, Envelope $envelope): array
 
             $recipientPayload = [
                 'email' => $recipient->getAddress(),
-                'name' => $recipient->getName(),
                 'type' => $type,
             ];
 
+            if ('' !== $recipient->getName()) {
+                $recipientPayload['name'] = $recipient->getName();
+            }
+
             $recipients[] = $recipientPayload;
         }
 

MandrillApiTransport.php

@@ -75,12 +75,15 @@ 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()->getName()) {
+            $payload['message']['from_name'] = $envelope->getSender()->getName();
+        }
+
         foreach ($email->getAttachments() as $attachment) {
             $headers = $attachment->getPreparedHeaders();
             $disposition = $headers->getHeaderBody('Content-Disposition');
@@ -122,10 +125,13 @@ protected function getRecipients(Email $email, Envelope $envelope): array
 
             $recipientPayload = [
                 'email' => $recipient->getAddress(),
-                'name' => $recipient->getName(),
                 'type' => $type,
             ];
 
+            if ('' !== $recipient->getName()) {
+                $recipientPayload['name'] = $recipient->getName();
+            }
+
             $recipients[] = $recipientPayload;
         }