bug #40834 [Notifier] Microsoft Teams: JSON structure error fix for simple message (KamilKubicki)

Nyholm · Nyholm · commit d7007d7785c3 · 2021-04-16T10:27:23.000+02:00
This PR was merged into the 5.3-dev branch. Discussion ---------- [Notifier] Microsoft Teams: JSON structure error fix for simple message Code caused following error - 'Unable to post the Microsoft Teams message: ... Summary or Text is required. '. The structure of the message should contain the 'text' attribute instead of 'title'. | Q | A | ------------- | --- | Branch? | 5.x | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | Fix | License | MIT Commits ------- 636dfc6 [Notifier] Microsoft Teams: JSON structure error fix for simple message
diff --git a/src/Symfony/Component/Notifier/Bridge/MicrosoftTeams/MicrosoftTeamsTransport.php b/src/Symfony/Component/Notifier/Bridge/MicrosoftTeams/MicrosoftTeamsTransport.php
@@ -60,7 +60,7 @@ protected function doSend(MessageInterface $message): SentMessage
         $endpoint = sprintf('https://%s%s', $this->getEndpoint(), $path);
         $response = $this->client->request('POST', $endpoint, [
             'json' => [
-                'title' => $message->getSubject(),
+                'text' => $message->getSubject(),
             ],
         ]);
 
diff --git a/src/Symfony/Component/Notifier/Bridge/MicrosoftTeams/Tests/MicrosoftTeamsTransportTest.php b/src/Symfony/Component/Notifier/Bridge/MicrosoftTeams/Tests/MicrosoftTeamsTransportTest.php
@@ -79,7 +79,7 @@ public function testSend()
     {
         $message = 'testMessage';
 
-        $expectedBody = json_encode(['title' => $message]);
+        $expectedBody = json_encode(['text' => $message]);
 
         $client = new MockHttpClient(function (string $method, string $url, array $options = []) use ($expectedBody): ResponseInterface {
             $this->assertJsonStringEqualsJsonString($expectedBody, $options['body']);

Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ public function testSend()`
`79`	`79`	`{`
`80`	`80`	`$message = 'testMessage';`
`81`	`81`
`82`		`- $expectedBody = json_encode(['title' => $message]);`
	`82`	`+ $expectedBody = json_encode(['text' => $message]);`
`83`	`83`
`84`	`84`	`$client = new MockHttpClient(function (string $method, string $url, array $options = []) use ($expectedBody): ResponseInterface {`
`85`	`85`	`$this->assertJsonStringEqualsJsonString($expectedBody, $options['body']);`