Merge pull request #286 from alexander-schranz/enhancement/php-84-deprecations

Fix PHP 8.4 deprecations nullable objects

Author: dbu

.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"

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

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;
     }

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();