8000 bug #41523 [Notifier] [Bridge] Remove hidden dependency on HttpFounda… · symfony/symfony@94826fd · GitHub
[go: up one dir, main page]

Skip to content

Commit 94826fd

Browse files
committed
bug #41523 [Notifier] [Bridge] Remove hidden dependency on HttpFoundation for SmsBiurasTransport (fre5h)
This PR was merged into the 5.3 branch. Discussion ---------- [Notifier] [Bridge] Remove hidden dependency on HttpFoundation for SmsBiurasTransport | Q | A | ------------- | --- | Branch? | 5.3 | Bug fix? | yes | New feature? | no | Deprecations? | no | License | MIT [SmsBiurasTransport](https://github.com/symfony/symfony/blob/5.3/src/Symfony/Component/Notifier/Bridge/SmsBiuras/SmsBiurasTransport.php#L93) has a hidden dependency on HttpFoundation. Similar to this pull request #40766 but one more class to be fixed Commits ------- c8d3f3e Remove hidden dependency on HttpFoundation for SmsBiurasTransport
2 parents 198448e + c8d3f3e commit 94826fd

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

src/Symfony/Component/Notifier/Bridge/SmsBiuras/SmsBiurasTransport.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Component\Notifier\Bridge\SmsBiuras;
1313

14-
use Symfony\Component\HttpFoundation\Response;
1514
use Symfony\Component\Notifier\Exception\TransportException;
1615
use Symfony\Component\Notifier\Exception\UnsupportedMessageTypeException;
1716
use Symfony\Component\Notifier\Message\MessageInterface;
@@ -90,7 +89,7 @@ protected function doSend(MessageInterface $message): SentMessage
9089
],
9190
]);
9291

93-
if (Response::HTTP_OK !== $response->getStatusCode()) {
92+
if (200 !== $response->getStatusCode()) {
9493
throw new TransportException('Unable to send the SMS.', $response);
9594
}
9695

0 commit comments

Comments
 (0)
0