PHP 8.4 preparation: Make implicit-nullable parameters explicit (Case 212955)

src/Entity/BlockedEmailAddressHash.php

@@ -18,12 +18,12 @@ class BlockedEmailAddressHash implements BlockedEmailAddressHashInterface
 
     public static function fromEmailAddress(
         EmailAddress $emailAddress,
-        DateTimeImmutable $blockDate = null
+        ?DateTimeImmutable $blockDate = null
     ): BlockedEmailAddressHashInterface {
         return new self($emailAddress->getHash(), $blockDate);
     }
 
-    public function __construct(string $hash, DateTimeImmutable $blockDate = null)
+    public function __construct(string $hash, ?DateTimeImmutable $blockDate = null)
     {
         $this->hash = $hash;
         $this->blockDate = $blockDate ?? new DateTimeImmutable();

src/Entity/BlockedEmailAddressHashInterface.php

@@ -7,7 +7,7 @@
 
 interface BlockedEmailAddressHashInterface
 {
-    public static function fromEmailAddress(EmailAddress $emailAddress, DateTimeImmutable $blockDate = null): self;
+    public static function fromEmailAddress(EmailAddress $emailAddress, ?DateTimeImmutable $blockDate = null): self;
 
     public function getBlockedUntilDate(DateInterval $blockDuration): DateTimeImmutable;
 }

src/Exception/EmailAddressCanNotBeHashedWithoutSecretException.php

@@ -7,7 +7,7 @@
 
 class EmailAddressCanNotBeHashedWithoutSecretException extends WebfactoryNewsletterRegistrationException
 {
-    public function __construct(EmailAddress $emailAddress, $code = 0, Throwable $previous = null)
+    public function __construct(EmailAddress $emailAddress, $code = 0, ?Throwable $previous = null)
     {
         parent::__construct(
             'The email address "'.$emailAddress->getEmailAddress().'" could not be hashed as this email '

src/Exception/EmailAddressDoesNotMatchHashOfPendingOptInException.php

@@ -12,7 +12,7 @@ public function __construct(
         EmailAddress $emailAddress,
         PendingOptInInterface $pendingOptIn,
         $code = 0,
-        Throwable $previous = null
+        ?Throwable $previous = null
     ) {
         parent::__construct(
             'The email address "'.$emailAddress->getEmailAddress().'" does not correspond to the hash of the '

src/Exception/PendingOptInClassCouldNotBeDeterminedException.php

@@ -8,7 +8,7 @@
 
 class PendingOptInClassCouldNotBeDeterminedException extends WebfactoryNewsletterRegistrationException
 {
-    public function __construct($code = 0, Throwable $previous = null)
+    public function __construct($code = 0, ?Throwable $previous = null)
     {
         parent::__construct(
             'We were unable to determine your '.RecipientInterface::class.' implementation. Consider replacing'

src/Exception/PendingOptInIsOutdatedException.php

@@ -7,7 +7,7 @@
 
 class PendingOptInIsOutdatedException extends WebfactoryNewsletterRegistrationException
 {
-    public function __construct(PendingOptInInterface $pendingOptIn, $code = 0, Throwable $previous = null)
+    public function __construct(PendingOptInInterface $pendingOptIn, $code = 0, ?Throwable $previous = null)
     {
         parent::__construct(
             'The PendingOptIn with uuid '.$pendingOptIn->getUuid().' is outdated and can no longer be '

src/Exception/RecipientClassCouldNotBeDeterminedException.php

@@ -8,7 +8,7 @@
 
 class RecipientClassCouldNotBeDeterminedException extends WebfactoryNewsletterRegistrationException
 {
-    public function __construct($code = 0, Throwable $previous = null)
+    public function __construct($code = 0, ?Throwable $previous = null)
     {
         parent::__construct(
             'We were unable to determine your '.RecipientInterface::class.' implementation. Consider replacing'

src/StartRegistration/HoneypotType.php

@@ -26,7 +26,7 @@ class HoneypotType extends AbstractType
     protected TranslatorInterface $translator;
     protected LoggerInterface $logger;
 
-    public function __construct(TranslatorInterface $translator, LoggerInterface $logger = null)
+    public function __construct(TranslatorInterface $translator, ?LoggerInterface $logger = null)
     {
         $this->translator = $translator;
         $this->logger = $logger ?: new NullLogger();