10000 [Notifier] fix PR checks · symfony/symfony@9a467e7 · GitHub
[go: up one dir, main page]
Skip to content

Commit 9a467e7

Browse files
PEtanguyPEtanguy
committed
[Notifier] fix PR checks
1 parent 85520f2 commit 9a467e7

File tree

6 files changed

+42
-38
lines changed

6 files changed

+42
-38
lines changed

‎src/Symfony/Component/Notifier/Bridge/Brevo/BrevoTransport.php

+3-3
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ protected function doSend(MessageInterface $message): SentMessage
5757

5858
$sender = $message->getFrom() ?: $this->sender;
5959

60-
$response = $this->client->request('POST', 'https://' . $this->getEndpoint() . '/v3/transactionalSMS/sms', [
60+
$response = $this->client->request('POST', 'https://'. $this->getEndpoint(). '/v3/transactionalSMS/sms', [
6161
'json' => [
6262
'sender' => $sender,
6363
'recipient' => $message->getPhone(),
@@ -77,12 +77,12 @@ protected function doSend(MessageInterface $message): SentMessage
7777
if (201 !== $statusCode) {
7878
$error = $response->toArray(false);
7979

80-
throw new TransportException('Unable to send the SMS: ' . $error['message'], $response);
80+
throw new TransportException('Unable to send the SMS: '. $error['message'], $response);
8181
}
8282

8383
$success = $response->toArray(false);
8484

85-
$sentMessage = new SentMessage($message, (string)$this);
85+
$sentMessage = new SentMessage($message, (string) $this);
8686
$sentMessage->setMessageId($success['messageId']);
8787

8888
return $sentMessage;

‎src/Symfony/Component/Notifier/Bridge/Brevo/LICENSE

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
Copyright (c) 2022-present Fabien Potencier
1+
Copyright (c) 2023-present Fabien Potencier
22

33
Permission is hereby granted, free of charge, to any person obtaining a copy
44
of this software and associated documentation files (the "Software"), to deal

‎src/Symfony/Component/Notifier/Bridge/Brevo/Tests/BrevoTransportTest.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ public function testSendWithErrorResponseThrowsTransportException()
5454
->method('getContent')
5555
->willReturn(json_encode(['code' => 400, 'message' => 'bad request']));
5656

57-
$client = new MockHttpClient(static fn(): ResponseInterface => $response);
57+
$client = new MockHttpClient(static fn (): ResponseInterface => $response);
5858

5959
$transport = self::createTransport($client);
6060

‎src/Symfony/Component/Notifier/Bridge/Brevo/composer.json

+31-33
Original file line numberDiff line numberDiff line change
@@ -1,36 +1,34 @@
11
{
2-
"name": "symfony/brevo-notifier",
3-
"type": "symfony-notifier-bridge",
4-
"description": "Symfony Brevo Notifier Bridge",
5-
"keywords": [
6-
"brevo",
7-
"notifier"
8-
],
9-
"homepage": "https://symfony.com",
10-
"license": "MIT",
11-
"authors": [
12-
{
13-
"name": "Pierre Tanguy",
14-
"homepage": "https://github.com/petanguy"
2+
"name": "symfony/brevo-notifier",
3+
"type": "symfony-notifier-bridge",
4+
"description": "Symfony Brevo Notifier Bridge",
5+
"keywords": ["brevo", "notifier"],
6+
"homepage": "https://symfony.com",
7+
"license": "MIT",
8+
"authors": [
9+
{
10+
"name": "Pierre Tanguy",
11+
"homepage": "https://github.com/petanguy"
12+
},
13+
{
14+
"name": "Symfony Community",
15+
"homepage": "https://symfony.com/contributors"
16+
}
17+
],
18+
"require": {
19+
"php": ">=8.1",
20+
"symfony/http-client": "^5.4|^6.0",
21+
"symfony/notifier": "^6.3"
1522
},
16-
{
17-
"name": "Symfony Community",
18-
"homepage": "https://symfony.com/contributors"
19-
}
20-
],
21-
"require": {
22-
"php": ">=8.1",
23-
"symfony/http-client": "^5.4|^6.0",
24-
"symfony/notifier": "^6.3"
25-
},
26-
"require-dev": {
27-
"symfony/event-dispatcher": "^5.4|^6.0"
28-
},
29-
"autoload": {
30-
"psr-4": {"Symfony\\Component\\Notifier\\Bridge\\Brevo\\": ""},
31-
"exclude-from-classmap": [
32-
"/Tests/"
33-
]
34-
},
35-
"minimum-stability": "dev"
23+
"require-dev": {
24+
"symfony/event-dispatcher": "^5.4|^6.0"
25+
},
26+
"autoload": {
27+
"psr-4": {"Symfony\\Component\\Notifier\\Bridge\\Brevo\\": ""},
28+
"exclude-from-classmap": [
29+
"/Tests/"
30+
]
31+
},
32+
"minimum-stability": "dev"
3633
}
34+

‎src/Symfony/Component/Notifier/Bridge/Sendinblue/Tests/SendinblueTransportFactoryTest.php

+3
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,9 @@
1414
use Symfony\Component\Notifier\Bridge\Sendinblue\SendinblueTransportFactory;
1515
use Symfony\Component\Notifier\Test\TransportFactoryTestCase;
1616

17+
/**
18+
* @group legacy
19+
*/
1720
final class SendinblueTransportFactoryTest extends TransportFactoryTestCase
1821
{
1922
public function createFactory(): SendinblueTransportFactory

‎src/Symfony/Component/Notifier/Bridge/Sendinblue/Tests/SendinblueTransportTest.php

+3
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,9 @@
2121
use Symfony\Contracts\HttpClient\HttpClientInterface;
2222
use Symfony\Contracts\HttpClient\ResponseInterface;
2323

24+
/**
25+
* @group legacy
26+
*/
2427
final class SendinblueTransportTest extends TransportTestCase
2528
{
2629
public static function createTransport(HttpClientInterface $client = null): SendinblueTransport

0 commit comments

Comments
 (0)
0