Merge pull request #94 from victormacko/master

Removed deprecations for PHP 8.4

Author: mnapoli

src/Service/EventBridge/EventBridgeTransport.php

@@ -6,7 +6,6 @@
 use Exception;
 use Symfony\Component\Messenger\Envelope;
 use Symfony\Component\Messenger\Exception\TransportException;
-use Symfony\Component\Messenger\Transport\Serialization\PhpSerializer;
 use Symfony\Component\Messenger\Transport\Serialization\SerializerInterface;
 use Symfony\Component\Messenger\Transport\TransportInterface;
 use Throwable;

src/Service/Sns/SnsConsumer.php

@@ -28,8 +28,8 @@ public function __construct(
         BusDriver $busDriver,
         MessageBusInterface $bus,
         SerializerInterface $serializer,
-        string $transportName = null,
-        SnsTransportNameResolver $transportNameResolver = null,
+        ?string $transportName = null,
+        ?SnsTransportNameResolver $transportNameResolver = null,
     ) {
         $this->busDriver = $busDriver;
         $this->bus = $bus;

src/Service/Sns/SnsFifoStamp.php

@@ -8,7 +8,7 @@ class SnsFifoStamp implements NonSendableStampInterface {
     private ?string $messageGroupId;
     private ?string $messageDeduplicationId;
 
-    public function __construct(string $messageGroupId = null, string $messageDeduplicationId = null)
+    public function __construct(?string $messageGroupId = null, ?string $messageDeduplicationId = null)
     {
         $this->messageGroupId = $messageGroupId;
         $this->messageDeduplicationId = $messageDeduplicationId;

src/Service/Sns/SnsTransport.php

@@ -7,7 +7,6 @@
 use Exception;
 use Symfony\Component\Messenger\Envelope;
 use Symfony\Component\Messenger\Exception\TransportException;
-use Symfony\Component\Messenger\Transport\Serialization\PhpSerializer;
 use Symfony\Component\Messenger\Transport\Serialization\SerializerInterface;
 use Symfony\Component\Messenger\Transport\TransportInterface;
 use Throwable;

src/Service/Sqs/SqsConsumer.php

@@ -2,6 +2,7 @@
 
 namespace Bref\Symfony\Messenger\Service\Sqs;
 
+use Throwable;
 use Bref\Context\Context;
 use Bref\Event\Sqs\SqsEvent;
 use Bref\Event\Sqs\SqsHandler;
@@ -39,10 +40,10 @@ public function __construct(
         BusDriver $busDriver,
         MessageBusInterface $bus,
         SerializerInterface $serializer,
-        string $transportName = null,
-        LoggerInterface $logger = null,
+        ?string $transportName = null,
+        ?LoggerInterface $logger = null,
         bool $partialBatchFailure = false,
-        SqsTransportNameResolver $transportNameResolver = null
+        ?SqsTransportNameResolver $transportNameResolver = null
     ) {
         $this->busDriver = $busDriver;
         $this->bus = $bus;
@@ -102,7 +103,7 @@ public function handleSqs(SqsEvent $event, Context $context): void
             } catch (UnrecoverableExceptionInterface $exception) {
                 $this->logger->error(sprintf('SQS record with id "%s" failed to be processed. But failure was marked as unrecoverable. Message will be acknowledged.', $record->getMessageId()));
                 $this->logger->error($exception);
-            } catch (\Throwable $exception) {
+            } catch (Throwable $exception) {
                 if ($this->partialBatchFailure === false) {
                     throw $exception;
                 }