feature #36424 [Mailer][Messenger] add return statement for MessageHandler (ottaviano)

fabpot · fabpot · commit 0933f90fd5d2 · 2020-04-13T13:14:28.000+02:00
This PR was merged into the 5.1-dev branch. Discussion ---------- [Mailer][Messenger] add return statement for MessageHandler | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes | Deprecations? | no  | Tickets | | License | MIT | Doc PR | By returning the result of the transporter in `MessageHandler` we can get hold of `SentMessage` from `HandledStamp::getResult()`. ![image](https://user-images.githubusercontent.com/4582866/79046122-3bed9100-7c0f-11ea-878f-65a6eb610758.png) Commits ------- 7854cb4 [Mailer][Messenger] add return statement for MessageHandler
diff --git a/src/Symfony/Component/Mailer/Messenger/MessageHandler.php b/src/Symfony/Component/Mailer/Messenger/MessageHandler.php
@@ -11,6 +11,7 @@
 
 namespace Symfony\Component\Mailer\Messenger;
 
+use Symfony\Component\Mailer\SentMessage;
 use Symfony\Component\Mailer\Transport\TransportInterface;
 
 /**
@@ -25,8 +26,8 @@ public function __construct(TransportInterface $transport)
         $this->transport = $transport;
     }
 
-    public function __invoke(SendEmailMessage $message)
+    public function __invoke(SendEmailMessage $message): ?SentMessage
     {
-        $this->transport->send($message->getMessage(), $message->getEnvelope());
+        return $this->transport->send($message->getMessage(), $message->getEnvelope());
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -11,6 +11,7 @@`
`11`	`11`
`12`	`12`	`namespace Symfony\Component\Mailer\Messenger;`
`13`	`13`
	`14`	`+use Symfony\Component\Mailer\SentMessage;`
`14`	`15`	`use Symfony\Component\Mailer\Transport\TransportInterface;`
`15`	`16`
`16`	`17`	`/**`
`@@ -25,8 +26,8 @@ public function __construct(TransportInterface $transport)`
`25`	`26`	`$this->transport = $transport;`
`26`	`27`	`}`
`27`	`28`
`28`		`- public function __invoke(SendEmailMessage $message)`
	`29`	`+ public function __invoke(SendEmailMessage $message): ?SentMessage`
`29`	`30`	`{`
`30`		`- $this->transport->send($message->getMessage(), $message->getEnvelope());`
	`31`	`+ return $this->transport->send($message->getMessage(), $message->getEnvelope());`
`31`	`32`	`}`
`32`	`33`	`}`