Merge branch '5.4' into 6.3

nicolas-grekas · nicolas-grekas · commit 18daf06a6d4f · 2024-01-23T15:35:58.000+01:00
* 5.4:
  Fix implicitly-required parameters
  List CS fix in .git-blame-ignore-revs
  Apply php-cs-fixer fix --rules nullable_type_declaration_for_default_null_value
  [Messenger][AmazonSqs] Allow async-aws/sqs version 2
diff --git a/Channel/AbstractChannel.php b/Channel/AbstractChannel.php
@@ -23,7 +23,7 @@ abstract class AbstractChannel implements ChannelInterface
     protected $transport;
     protected $bus;
 
-    public function __construct(TransportInterface $transport = null, MessageBusInterface $bus = null)
+    public function __construct(?TransportInterface $transport = null, ?MessageBusInterface $bus = null)
     {
         if (null === $transport && null === $bus) {
             throw new LogicException(sprintf('"%s" needs a Transport or a Bus but both cannot be "null".', static::class));
diff --git a/Channel/BrowserChannel.php b/Channel/BrowserChannel.php
@@ -32,7 +32,7 @@ public function __construct(RequestStack $stack, FlashMessageImportanceMapperInt
         $this->mapper = $mapper;
     }
 
-    public function notify(Notification $notification, RecipientInterface $recipient, string $transportName = null): void
+    public function notify(Notification $notification, RecipientInterface $recipient, ?string $transportName = null): void
     {
         if (null === $request = $this->stack->getCurrentRequest()) {
             return;
diff --git a/Channel/ChannelInterface.php b/Channel/ChannelInterface.php
@@ -19,7 +19,7 @@
  */
 interface ChannelInterface
 {
-    public function notify(Notification $notification, RecipientInterface $recipient, string $transportName = null): void;
+    public function notify(Notification $notification, RecipientInterface $recipient, ?string $transportName = null): void;
 
     public function supports(Notification $notification, RecipientInterface $recipient): bool;
 }
diff --git a/Channel/ChatChannel.php b/Channel/ChatChannel.php
@@ -21,7 +21,7 @@
  */
 class ChatChannel extends AbstractChannel
 {
-    public function notify(Notification $notification, RecipientInterface $recipient, string $transportName = null): void
+    public function notify(Notification $notification, RecipientInterface $recipient, ?string $transportName = null): void
     {
         $message = null;
         if ($notification instanceof ChatNotificationInterface) {
diff --git a/Channel/EmailChannel.php b/Channel/EmailChannel.php
@@ -34,7 +34,7 @@ class EmailChannel implements ChannelInterface
     private string|Address|null $from;
     private ?Envelope $envelope;
 
-    public function __construct(TransportInterface $transport = null, MessageBusInterface $bus = null, string $from = null, Envelope $envelope = null)
+    public function __construct(?TransportInterface $transport = null, ?MessageBusInterface $bus = null, ?string $from = null, ?Envelope $envelope = null)
     {
         if (null === $transport && null === $bus) {
             throw new LogicException(sprintf('"%s" needs a Transport or a Bus but both cannot be "null".', static::class));
@@ -46,7 +46,7 @@ public function __construct(TransportInterface $transport = null, MessageBusInte
         $this->envelope = $envelope;
     }
 
-    public function notify(Notification $notification, RecipientInterface $recipient, string $transportName = null): void
+    public function notify(Notification $notification, RecipientInterface $recipient, ?string $transportName = null): void
     {
         $message = null;
         if ($notification instanceof EmailNotificationInterface) {
diff --git a/Channel/PushChannel.php b/Channel/PushChannel.php
@@ -21,7 +21,7 @@
  */
 class PushChannel extends AbstractChannel
 {
-    public function notify(Notification $notification, RecipientInterface $recipient, string $transportName = null): void
+    public function notify(Notification $notification, RecipientInterface $recipient, ?string $transportName = null): void
     {
         $message = null;
         if ($notification instanceof PushNotificationInterface) {
diff --git a/Channel/SmsChannel.php b/Channel/SmsChannel.php
@@ -22,7 +22,7 @@
  */
 class SmsChannel extends AbstractChannel
 {
-    public function notify(Notification $notification, RecipientInterface $recipient, string $transportName = null): void
+    public function notify(Notification $notification, RecipientInterface $recipient, ?string $transportName = null): void
     {
         $message = null;
         if ($notification instanceof SmsNotificationInterface) {
diff --git a/Chatter.php b/Chatter.php
@@ -27,7 +27,7 @@ final class Chatter implements ChatterInterface
     private ?MessageBusInterface $bus;
     private ?EventDispatcherInterface $dispatcher;
 
-    public function __construct(TransportInterface $transport, MessageBusInterface $bus = null, EventDispatcherInterface $dispatcher = null)
+    public function __construct(TransportInterface $transport, ?MessageBusInterface $bus = null, ?EventDispatcherInterface $dispatcher = null)
     {
         $this->transport = $transport;
         $this->bus = $bus;
diff --git a/DataCollector/NotificationDataCollector.php b/DataCollector/NotificationDataCollector.php
@@ -29,7 +29,7 @@ public function __construct(NotificationLoggerListener $logger)
         $this->logger = $logger;
     }
 
-    public function collect(Request $request, Response $response, \Throwable $exception = null): void
+    public function collect(Request $request, Response $response, ?\Throwable $exception = null): void
     {
         $this->data['events'] = $this->logger->getEvents();
     }
diff --git a/Event/NotificationEvents.php b/Event/NotificationEvents.php
@@ -35,7 +35,7 @@ public function getTransports(): array
     /**
      * @return MessageEvent[]
      */
-    public function getEvents(string $name = null): array
+    public function getEvents(?string $name = null): array
     {
         if (null === $name) {
             return $this->events;
@@ -54,7 +54,7 @@ public function getEvents(string $name = null): array
     /**
      * @return MessageInterface[]
      */
-    public function getMessages(string $name = null): array
+    public function getMessages(?string $name = null): array
     {
         $events = $this->getEvents($name);
         $messages = [];
diff --git a/Exception/IncompleteDsnException.php b/Exception/IncompleteDsnException.php
@@ -18,7 +18,7 @@ class IncompleteDsnException extends InvalidArgumentException
 {
     private ?string $dsn;
 
-    public function __construct(string $message, string $dsn = null, \Throwable $previous = null)
+    public function __construct(string $message, ?string $dsn = null, ?\Throwable $previous = null)
     {
         $this->dsn = $dsn;
         if ($dsn) {
diff --git a/Exception/MissingRequiredOptionException.php b/Exception/MissingRequiredOptionException.php
@@ -16,7 +16,7 @@
  */
 class MissingRequiredOptionException extends IncompleteDsnException
 {
-    public function __construct(string $option, string $dsn = null, \Throwable $previous = null)
+    public function __construct(string $option, ?string $dsn = null, ?\Throwable $previous = null)
     {
         $message = sprintf('The option "%s" is required but missing.', $option);
 
diff --git a/Exception/TransportException.php b/Exception/TransportException.php
@@ -21,7 +21,7 @@ class TransportException extends RuntimeException implements TransportExceptionI
     private ResponseInterface $response;
     private string $debug = '';
 
-    public function __construct(string $message, ResponseInterface $response, int $code = 0, \Throwable $previous = null)
+    public function __construct(string $message, ResponseInterface $response, int $code = 0, ?\Throwable $previous = null)
     {
         $this->response = $response;
         $this->debug .= $response->getInfo('debug') ?? '';
diff --git a/Exception/UnsupportedSchemeException.php b/Exception/UnsupportedSchemeException.php
@@ -285,7 +285,7 @@ class UnsupportedSchemeException extends LogicException
     /**
      * @param string[] $supported
      */
-    public function __construct(Dsn $dsn, string $name = null, array $supported = [], \Throwable $previous = null)
+    public function __construct(Dsn $dsn, ?string $name = null, array $supported = [], ?\Throwable $previous = null)
     {
         $provider = $dsn->getScheme();
         if (false !== $pos = strpos($provider, '+')) {
diff --git a/Message/ChatMessage.php b/Message/ChatMessage.php
@@ -23,7 +23,7 @@ class ChatMessage implements MessageInterface, FromNotificationInterface
     private ?MessageOptionsInterface $options;
     private ?Notification $notification = null;
 
-    public function __construct(string $subject, MessageOptionsInterface $options = null)
+    public function __construct(string $subject, ?MessageOptionsInterface $options = null)
     {
         $this->subject = $subject;
         $this->options = $options;
diff --git a/Message/EmailMessage.php b/Message/EmailMessage.php
@@ -29,7 +29,7 @@ class EmailMessage implements MessageInterface, FromNotificationInterface
     private ?Envelope $envelope;
     private ?Notification $notification = null;
 
-    public function __construct(RawMessage $message, Envelope $envelope = null)
+    public function __construct(RawMessage $message, ?Envelope $envelope = null)
     {
         $this->message = $message;
         $this->envelope = $envelope;
diff --git a/Message/PushMessage.php b/Message/PushMessage.php
@@ -24,7 +24,7 @@ class PushMessage implements MessageInterface, FromNotificationInterface
     private $options;
     private $notification;
 
-    public function __construct(string $subject, string $content, MessageOptionsInterface $options = null)
+    public function __construct(string $subject, string $content, ?MessageOptionsInterface $options = null)
     {
         $this->subject = $subject;
         $this->content = $content;
diff --git a/Message/SmsMessage.php b/Message/SmsMessage.php
@@ -27,7 +27,7 @@ class SmsMessage implements MessageInterface, FromNotificationInterface
     private ?MessageOptionsInterface $options;
     private ?Notification $notification = null;
 
-    public function __construct(string $phone, string $subject, string $from = '', MessageOptionsInterface $options = null)
+    public function __construct(string $phone, string $subject, string $from = '', ?MessageOptionsInterface $options = null)
     {
         if ('' === $phone) {
             throw new InvalidArgumentException(sprintf('"%s" needs a phone number, it cannot be empty.', __CLASS__));
diff --git a/Notification/ChatNotificationInterface.php b/Notification/ChatNotificationInterface.php
@@ -19,5 +19,5 @@
  */
 interface ChatNotificationInterface
 {
-    public function asChatMessage(RecipientInterface $recipient, string $transport = null): ?ChatMessage;
+    public function asChatMessage(RecipientInterface $recipient, ?string $transport = null): ?ChatMessage;
 }
diff --git a/Notification/EmailNotificationInterface.php b/Notification/EmailNotificationInterface.php
@@ -19,5 +19,5 @@
  */
 interface EmailNotificationInterface
 {
-    public function asEmailMessage(EmailRecipientInterface $recipient, string $transport = null): ?EmailMessage;
+    public function asEmailMessage(EmailRecipientInterface $recipient, ?string $transport = null): ?EmailMessage;
 }
diff --git a/Notification/PushNotificationInterface.php b/Notification/PushNotificationInterface.php
@@ -16,5 +16,5 @@
 
 interface PushNotificationInterface
 {
-    public function asPushMessage(RecipientInterface $recipient, string $transport = null): ?PushMessage;
+    public function asPushMessage(RecipientInterface $recipient, ?string $transport = null): ?PushMessage;
 }
diff --git a/Notification/SmsNotificationInterface.php b/Notification/SmsNotificationInterface.php
@@ -19,5 +19,5 @@
  */
 interface SmsNotificationInterface
 {
-    public function asSmsMessage(SmsRecipientInterface $recipient, string $transport = null): ?SmsMessage;
+    public function asSmsMessage(SmsRecipientInterface $recipient, ?string $transport = null): ?SmsMessage;
 }
diff --git a/Notifier.php b/Notifier.php
@@ -33,7 +33,7 @@ final class Notifier implements NotifierInterface
     /**
      * @param ChannelInterface[]|ContainerInterface $channels
      */
-    public function __construct(array|ContainerInterface $channels, ChannelPolicyInterface $policy = null)
+    public function __construct(array|ContainerInterface $channels, ?ChannelPolicyInterface $policy = null)
     {
         $this->channels = $channels;
         $this->policy = $policy;
diff --git a/Test/Constraint/NotificationCount.php b/Test/Constraint/NotificationCount.php
@@ -23,7 +23,7 @@ final class NotificationCount extends Constraint
     private $transport;
     private $queued;
 
-    public function __construct(int $expectedValue, string $transport = null, bool $queued = false)
+    public function __construct(int $expectedValue, ?string $transport = null, bool $queued = false)
     {
         $this->expectedValue = $expectedValue;
         $this->transport = $transport;
diff --git a/Test/TransportFactoryTestCase.php b/Test/TransportFactoryTestCase.php
@@ -86,7 +86,7 @@ public function testCreate(string $expected, string $dsn)
     /**
      * @dataProvider unsupportedSchemeProvider
      */
-    public function testUnsupportedSchemeException(string $dsn, string $message = null)
+    public function testUnsupportedSchemeException(string $dsn, ?string $message = null)
     {
         $factory = $this->createFactory();
 
@@ -103,7 +103,7 @@ public function testUnsupportedSchemeException(string $dsn, string $message = nu
     /**
      * @dataProvider incompleteDsnProvider
      */
-    public function testIncompleteDsnException(string $dsn, string $message = null)
+    public function testIncompleteDsnException(string $dsn, ?string $message = null)
     {
         $factory = $this->createFactory();
 
@@ -120,7 +120,7 @@ public function testIncompleteDsnException(string $dsn, string $message = null)
     /**
      * @dataProvider missingRequiredOptionProvider
      */
-    public function testMissingRequiredOptionException(string $dsn, string $message = null)
+    public function testMissingRequiredOptionException(string $dsn, ?string $message = null)
     {
         $factory = $this->createFactory();
 
diff --git a/Test/TransportTestCase.php b/Test/TransportTestCase.php
@@ -27,7 +27,7 @@ abstract class TransportTestCase extends TestCase
     protected const CUSTOM_HOST = 'host.test';
     protected const CUSTOM_PORT = 42;
 
-    abstract public static function createTransport(HttpClientInterface $client = null): TransportInterface;
+    abstract public static function createTransport(?HttpClientInterface $client = null): TransportInterface;
 
     /**
      * @return iterable<array{0: string, 1: TransportInterface}>
@@ -55,7 +55,7 @@ public function testToString(string $expected, TransportInterface $transport)
     /**
      * @dataProvider supportedMessagesProvider
      */
-    public function testSupportedMessages(MessageInterface $message, TransportInterface $transport = null)
+    public function testSupportedMessages(MessageInterface $message, ?TransportInterface $transport = null)
     {
         $transport ??= $this->createTransport();
 
@@ -65,7 +65,7 @@ public function testSupportedMessages(MessageInterface $message, TransportInterf
     /**
      * @dataProvider unsupportedMessagesProvider
      */
-    public function testUnsupportedMessages(MessageInterface $message, TransportInterface $transport = null)
+    public function testUnsupportedMessages(MessageInterface $message, ?TransportInterface $transport = null)
     {
         $transport ??= $this->createTransport();
 
@@ -75,7 +75,7 @@ public function testUnsupportedMessages(MessageInterface $message, TransportInte
     /**
      * @dataProvider unsupportedMessagesProvider
      */
-    public function testUnsupportedMessagesTrowUnsupportedMessageTypeExceptionWhenSend(MessageInterface $message, TransportInterface $transport = null)
+    public function testUnsupportedMessagesTrowUnsupportedMessageTypeExceptionWhenSend(MessageInterface $message, ?TransportInterface $transport = null)
     {
         $transport ??= $this->createTransport();
 
diff --git a/Tests/Channel/AbstractChannelTest.php b/Tests/Channel/AbstractChannelTest.php
@@ -32,7 +32,7 @@ public function testChannelCannotBeConstructedWithoutTransportAndBus()
 
 class DummyChannel extends AbstractChannel
 {
-    public function notify(Notification $notification, RecipientInterface $recipient, string $transportName = null): void
+    public function notify(Notification $notification, RecipientInterface $recipient, ?string $transportName = null): void
     {
         return;
     }
diff --git a/Tests/Event/FailedMessageEventTest.php b/Tests/Event/FailedMessageEventTest.php
@@ -58,7 +58,7 @@ public function testFailedMessageEventIsDisptachIfError()
         $transport = new class($clientMock, $eventDispatcherMock) extends AbstractTransport {
             public $exception;
 
-            public function __construct($client, EventDispatcherInterface $dispatcher = null)
+            public function __construct($client, ?EventDispatcherInterface $dispatcher = null)
             {
                 $this->exception = new NullTransportException();
                 parent::__construct($client, $dispatcher);
diff --git a/Tests/Mailer/DummyMailer.php b/Tests/Mailer/DummyMailer.php
@@ -22,7 +22,7 @@ class DummyMailer implements MailerInterface
 {
     private $sentMessage;
 
-    public function send(RawMessage $message, Envelope $envelope = null): void
+    public function send(RawMessage $message, ?Envelope $envelope = null): void
     {
         $this->sentMessage = $message;
     }
diff --git a/Tests/Transport/DsnTest.php b/Tests/Transport/DsnTest.php
@@ -21,7 +21,7 @@ final class DsnTest extends TestCase
     /**
      * @dataProvider constructProvider
      */
-    public function testConstruct(string $dsnString, string $scheme, string $host, string $user = null, string $password = null, int $port = null, array $options = [], string $path = null)
+    public function testConstruct(string $dsnString, string $scheme, string $host, ?string $user = null, ?string $password = null, ?int $port = null, array $options = [], ?string $path = null)
     {
         $dsn = new Dsn($dsnString);
         $this->assertSame($dsnString, $dsn->getOriginalDsn());
@@ -172,7 +172,7 @@ public static function invalidDsnProvider(): iterable
     /**
      * @dataProvider getOptionProvider
      */
-    public function testGetOption($expected, string $dsnString, string $option, string $default = null)
+    public function testGetOption($expected, string $dsnString, string $option, ?string $default = null)
     {
         $dsn = new Dsn($dsnString);
 
diff --git a/Texter.php b/Texter.php
@@ -27,7 +27,7 @@ final class Texter implements TexterInterface
     private ?MessageBusInterface $bus;
     private ?EventDispatcherInterface $dispatcher;
 
-    public function __construct(TransportInterface $transport, MessageBusInterface $bus = null, EventDispatcherInterface $dispatcher = null)
+    public function __construct(TransportInterface $transport, ?MessageBusInterface $bus = null, ?EventDispatcherInterface $dispatcher = null)
     {
         $this->transport = $transport;
         $this->bus = $bus;
diff --git a/Transport.php b/Transport.php
@@ -97,14 +97,14 @@ final class Transport
 
     private iterable $factories;
 
-    public static function fromDsn(#[\SensitiveParameter] string $dsn, EventDispatcherInterface $dispatcher = null, HttpClientInterface $client = null): TransportInterface
+    public static function fromDsn(#[\SensitiveParameter] string $dsn, ?EventDispatcherInterface $dispatcher = null, ?HttpClientInterface $client = null): TransportInterface
     {
         $factory = new self(self::getDefaultFactories($dispatcher, $client));
 
         return $factory->fromString($dsn);
     }
 
-    public static function fromDsns(#[\SensitiveParameter] array $dsns, EventDispatcherInterface $dispatcher = null, HttpClientInterface $client = null): TransportInterface
+    public static function fromDsns(#[\SensitiveParameter] array $dsns, ?EventDispatcherInterface $dispatcher = null, ?HttpClientInterface $client = null): TransportInterface
     {
         $factory = new self(iterator_to_array(self::getDefaultFactories($dispatcher, $client)));
 
@@ -171,7 +171,7 @@ private function createFromDsns(#[\SensitiveParameter] array $dsns): array
     /**
      * @return TransportFactoryInterface[]
      */
-    private static function getDefaultFactories(EventDispatcherInterface $dispatcher = null, HttpClientInterface $client = null): iterable
+    private static function getDefaultFactories(?EventDispatcherInterface $dispatcher = null, ?HttpClientInterface $client = null): iterable
     {
         foreach (self::FACTORY_CLASSES as $factoryClass) {
             if (class_exists($factoryClass)) {
diff --git a/Transport/AbstractTransport.php b/Transport/AbstractTransport.php
@@ -34,7 +34,7 @@ abstract class AbstractTransport implements TransportInterface
     protected $host;
     protected $port;
 
-    public function __construct(HttpClientInterface $client = null, EventDispatcherInterface $dispatcher = null)
+    public function __construct(?HttpClientInterface $client = null, ?EventDispatcherInterface $dispatcher = null)
     {
         $this->client = $client;
         if (null === $client) {
diff --git a/Transport/AbstractTransportFactory.php b/Transport/AbstractTransportFactory.php
@@ -24,7 +24,7 @@ abstract class AbstractTransportFactory implements TransportFactoryInterface
     protected $dispatcher;
     protected $client;
 
-    public function __construct(EventDispatcherInterface $dispatcher = null, HttpClientInterface $client = null)
+    public function __construct(?EventDispatcherInterface $dispatcher = null, ?HttpClientInterface $client = null)
     {
         $this->dispatcher = $dispatcher;
         $this->client = $client;
diff --git a/Transport/Dsn.php b/Transport/Dsn.php
@@ -74,7 +74,7 @@ public function getPassword(): ?string
         return $this->password;
     }
 
-    public function getPort(int $default = null): ?int
+    public function getPort(?int $default = null): ?int
     {
         return $this->port ?? $default;
     }
diff --git a/Transport/NullTransport.php b/Transport/NullTransport.php

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ abstract class AbstractChannel implements ChannelInterface`
`23`	`23`	`protected $transport;`
`24`	`24`	`protected $bus;`
`25`	`25`
`26`		`- public function __construct(TransportInterface $transport = null, MessageBusInterface $bus = null)`
	`26`	`+ public function __construct(?TransportInterface $transport = null, ?MessageBusInterface $bus = null)`
`27`	`27`	`{`
`28`	`28`	`if (null === $transport && null === $bus) {`
`29`	`29`	`throw new LogicException(sprintf('"%s" needs a Transport or a Bus but both cannot be "null".', static::class));`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ public function __construct(RequestStack $stack, FlashMessageImportanceMapperInt`
`32`	`32`	`$this->mapper = $mapper;`
`33`	`33`	`}`
`34`	`34`
`35`		`- public function notify(Notification $notification, RecipientInterface $recipient, string $transportName = null): void`
	`35`	`+ public function notify(Notification $notification, RecipientInterface $recipient, ?string $transportName = null): void`
`36`	`36`	`{`
`37`	`37`	`if (null === $request = $this->stack->getCurrentRequest()) {`
`38`	`38`	`return;`
Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@`
`19`	`19`	`*/`
`20`	`20`	`interface ChannelInterface`
`21`	`21`	`{`
`22`		`- public function notify(Notification $notification, RecipientInterface $recipient, string $transportName = null): void;`
	`22`	`+ public function notify(Notification $notification, RecipientInterface $recipient, ?string $transportName = null): void;`
`23`	`23`
`24`	`24`	`public function supports(Notification $notification, RecipientInterface $recipient): bool;`
`25`	`25`	`}`
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@`
`21`	`21`	`*/`
`22`	`22`	`class ChatChannel extends AbstractChannel`
`23`	`23`	`{`
`24`		`- public function notify(Notification $notification, RecipientInterface $recipient, string $transportName = null): void`
	`24`	`+ public function notify(Notification $notification, RecipientInterface $recipient, ?string $transportName = null): void`
`25`	`25`	`{`
`26`	`26`	`$message = null;`
`27`	`27`	`if ($notification instanceof ChatNotificationInterface) {`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ class EmailChannel implements ChannelInterface`
`34`	`34`	`private string\|Address\|null $from;`
`35`	`35`	`private ?Envelope $envelope;`
`36`	`36`
`37`		`- public function __construct(TransportInterface $transport = null, MessageBusInterface $bus = null, string $from = null, Envelope $envelope = null)`
	`37`	`+ public function __construct(?TransportInterface $transport = null, ?MessageBusInterface $bus = null, ?string $from = null, ?Envelope $envelope = null)`
`38`	`38`	`{`
`39`	`39`	`if (null === $transport && null === $bus) {`
`40`	`40`	`throw new LogicException(sprintf('"%s" needs a Transport or a Bus but both cannot be "null".', static::class));`
`@@ -46,7 +46,7 @@ public function __construct(TransportInterface $transport = null, MessageBusInte`
`46`	`46`	`$this->envelope = $envelope;`
`47`	`47`	`}`
`48`	`48`
`49`		`- public function notify(Notification $notification, RecipientInterface $recipient, string $transportName = null): void`
	`49`	`+ public function notify(Notification $notification, RecipientInterface $recipient, ?string $transportName = null): void`
`50`	`50`	`{`
`51`	`51`	`$message = null;`
`52`	`52`	`if ($notification instanceof EmailNotificationInterface) {`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@`
`22`	`22`	`*/`
`23`	`23`	`class SmsChannel extends AbstractChannel`
`24`	`24`	`{`
`25`		`- public function notify(Notification $notification, RecipientInterface $recipient, string $transportName = null): void`
	`25`	`+ public function notify(Notification $notification, RecipientInterface $recipient, ?string $transportName = null): void`
`26`	`26`	`{`
`27`	`27`	`$message = null;`
`28`	`28`	`if ($notification instanceof SmsNotificationInterface) {`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ final class Chatter implements ChatterInterface`
`27`	`27`	`private ?MessageBusInterface $bus;`
`28`	`28`	`private ?EventDispatcherInterface $dispatcher;`
`29`	`29`
`30`		`- public function __construct(TransportInterface $transport, MessageBusInterface $bus = null, EventDispatcherInterface $dispatcher = null)`
	`30`	`+ public function __construct(TransportInterface $transport, ?MessageBusInterface $bus = null, ?EventDispatcherInterface $dispatcher = null)`
`31`	`31`	`{`
`32`	`32`	`$this->transport = $transport;`
`33`	`33`	`$this->bus = $bus;`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ public function __construct(NotificationLoggerListener $logger)`
`29`	`29`	`$this->logger = $logger;`
`30`	`30`	`}`
`31`	`31`
`32`		`- public function collect(Request $request, Response $response, \Throwable $exception = null): void`
	`32`	`+ public function collect(Request $request, Response $response, ?\Throwable $exception = null): void`
`33`	`33`	`{`
`34`	`34`	`$this->data['events'] = $this->logger->getEvents();`
`35`	`35`	`}`