bug #46368 [Mailer] Fix for missing sender name in case with usage of the EnvelopeListener (bobahvas)

fabpot · fabpot · commit e94f1f61848e · 2022-06-09T19:28:42.000+02:00
This PR was merged into the 4.4 branch. Discussion ---------- [Mailer] Fix for missing sender name in case with usage of the EnvelopeListener | Q | A | ------------- | --- | Branch? | 4.4, 5.4 or 6.0 | Bug fix? | yes | New feature? | no | Deprecations? | no | License | MIT ``` $eventDispatcher = new \Symfony\Component\EventDispatcher\EventDispatcher(); $eventDispatcher->addSubscriber( new \Symfony\Component\Mailer\EventListener\EnvelopeListener( new \Symfony\Component\Mime\Address('noreply@test.com', 'TestName') ) ); ``` Commits ------- c12039b Fix for missing sender name in case with usage of the EnvelopeListener
diff --git a/src/Symfony/Component/Mailer/EventListener/EnvelopeListener.php b/src/Symfony/Component/Mailer/EventListener/EnvelopeListener.php
@@ -48,7 +48,7 @@ public function onMessage(MessageEvent $event): void
             $message = $event->getMessage();
             if ($message instanceof Message) {
                 if (!$message->getHeaders()->has('Sender') && !$message->getHeaders()->has('From')) {
-                    $message->getHeaders()->addMailboxHeader('Sender', $this->sender->getAddress());
+                    $message->getHeaders()->addMailboxHeader('Sender', $this->sender);
                 }
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ public function onMessage(MessageEvent $event): void`
`48`	`48`	`$message = $event->getMessage();`
`49`	`49`	`if ($message instanceof Message) {`
`50`	`50`	`if (!$message->getHeaders()->has('Sender') && !$message->getHeaders()->has('From')) {`
`51`		`- $message->getHeaders()->addMailboxHeader('Sender', $this->sender->getAddress());`
	`51`	`+ $message->getHeaders()->addMailboxHeader('Sender', $this->sender);`
`52`	`52`	`}`
`53`	`53`	`}`
`54`	`54`	`}`