Fix PHP 8.4 deprecations nullable objects

symfony-cmf · Dec 2, 2024 · 000586d · 000586d
1 parent 3a3c645
commit 000586d
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 4 deletions.
diff --git a/.github/workflows/test-application.yaml b/.github/workflows/test-application.yaml
@@ -30,6 +30,8 @@ jobs:
                     - php-version: "8.2"
                     - php-version: "8.3"
                       symfony-version: "7"
+                    - php-version: "8.4"
+                      symfony-version: "7"
 
         steps:
             - name: "Checkout project"

diff --git a/src/ChainRouter.php b/src/ChainRouter.php
@@ -247,8 +247,8 @@ private function rebuildRequest(string $pathinfo): Request
 
     private function getErrorMessage(
         string $name,
-        RouterInterface|UrlGeneratorInterface|RequestMatcherInterface $router = null,
-        array $parameters = null
+        RouterInterface|UrlGeneratorInterface|RequestMatcherInterface|null $router = null,
+        ?array $parameters = null
     ): string {
         if ($router instanceof VersatileGeneratorInterface) {
             // the $parameters are not forced to be array, but versatile generator does typehint it

diff --git a/src/Event/RouterMatchEvent.php b/src/Event/RouterMatchEvent.php
@@ -18,7 +18,7 @@ final class RouterMatchEvent extends Event
 {
     private ?Request $request;
 
-    public function __construct(Request $request = null)
+    public function __construct(?Request $request = null)
     {
         $this->request = $request;
     }

diff --git a/src/ProviderBasedGenerator.php b/src/ProviderBasedGenerator.php
@@ -26,7 +26,7 @@ class ProviderBasedGenerator extends UrlGenerator implements VersatileGeneratorI
 {
     protected RouteProviderInterface $provider;
 
-    public function __construct(RouteProviderInterface $provider, LoggerInterface $logger = null)
+    public function __construct(RouteProviderInterface $provider, ?LoggerInterface $logger = null)
     {
         $this->provider = $provider;
         $this->logger = $logger ?: new NullLogger();