Fix deprecation and CS

Author: norkunas

src/DataCollector/GeocoderDataCollector.php

@@ -47,7 +47,7 @@ public function reset()
     /**
      * @return void
      */
-    public function collect(Request $request, Response $response, \Throwable $exception = null)
+    public function collect(Request $request, Response $response, ?\Throwable $exception = null)
     {
         if (!empty($this->data['queries'])) {
             // To avoid collection more that once.

src/DependencyInjection/BazingaGeocoderExtension.php

@@ -30,9 +30,9 @@
 use Symfony\Component\Config\FileLocator;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException;
+use Symfony\Component\DependencyInjection\Extension\Extension;
 use Symfony\Component\DependencyInjection\Loader\YamlFileLoader;
 use Symfony\Component\DependencyInjection\Reference;
-use Symfony\Component\HttpKernel\DependencyInjection\Extension;
 
 /**
  * @author William Durand <[email protected]>.

src/Plugin/FakeIpPlugin.php

@@ -30,7 +30,7 @@ class FakeIpPlugin implements Plugin
     private ?string $replacement;
     private ?Generator $faker = null;
 
-    public function __construct(?string $needle, string $replacement = null, bool $useFaker = false)
+    public function __construct(?string $needle, ?string $replacement = null, bool $useFaker = false)
     {
         $this->needle = $needle;
         $this->replacement = $replacement;

src/ProviderFactory/AbstractFactory.php

@@ -32,7 +32,7 @@ abstract class AbstractFactory implements ProviderFactoryInterface
 
     protected ?ClientInterface $httpClient;
 
-    public function __construct(ClientInterface $httpClient = null)
+    public function __construct(?ClientInterface $httpClient = null)
     {
         $this->httpClient = $httpClient;
     }

src/Validator/Constraint/Address.php

@@ -49,7 +49,7 @@ class Address extends Constraint
     /**
      * @param string[]|null $options
      */
-    public function __construct(array $options = null, string $message = null, array $groups = null, $payload = null)
+    public function __construct(?array $options = null, ?string $message = null, ?array $groups = null, $payload = null)
     {
         parent::__construct($options, $groups, $payload);
 

tests/Functional/Helper/CacheHelperV8.php

@@ -23,7 +23,7 @@ public function get(string $key, mixed $default = null): mixed
     {
     }
 
-    public function set(string $key, mixed $value, int|\DateInterval $ttl = null): bool
+    public function set(string $key, mixed $value, int|\DateInterval|null $ttl = null): bool
     {
         return true;
     }
@@ -43,7 +43,7 @@ public function getMultiple(iterable $keys, mixed $default = null): iterable
         return [];
     }
 
-    public function setMultiple(iterable $values, int|\DateInterval $ttl = null): bool
+    public function setMultiple(iterable $values, int|\DateInterval|null $ttl = null): bool
     {
         return true;
     }