bug #39522 [Notifier] Set message id on SentMessage (OskarStark)

fabpot · fabpot · commit 695de7c1a060 · 2020-12-17T17:30:10.000+01:00
This PR was squashed before being merged into the 5.2 branch. Discussion ---------- [Notifier] Set message id on SentMessage | Q | A | ------------- | --- | Branch? | 5.2 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | --- | License | MIT | Doc PR | --- @ismail1432 what do you think? Commits ------- 9dd09e632d [Notifier] Set message id on SentMessage
diff --git a/LinkedInTransport.php b/LinkedInTransport.php
@@ -90,7 +90,10 @@ protected function doSend(MessageInterface $message): SentMessage
             throw new TransportException(sprintf('Unable to post the Linkedin message : "%s".', $result['error']), $response);
         }
 
-        return new SentMessage($message, (string) $this);
+        $sentMessage = new SentMessage($message, (string) $this);
+        $sentMessage->setMessageId($result['id']);
+
+        return $sentMessage;
     }
 
     private function bodyFromMessageWithNoOption(MessageInterface $message): array

Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,10 @@ protected function doSend(MessageInterface $message): SentMessage`
`90`	`90`	`throw new TransportException(sprintf('Unable to post the Linkedin message : "%s".', $result['error']), $response);`
`91`	`91`	`}`
`92`	`92`
`93`		`- return new SentMessage($message, (string) $this);`
	`93`	`+ $sentMessage = new SentMessage($message, (string) $this);`
	`94`	`+ $sentMessage->setMessageId($result['id']);`
	`95`	`+`
	`96`	`+ return $sentMessage;`
`94`	`97`	`}`
`95`	`98`
`96`	`99`	`private function bodyFromMessageWithNoOption(MessageInterface $message): array`