Apply php-cs-fixer fix --rules nullable_type_declaration_for_default_null_value

Author: nicolas-grekas

.php-cs-fixer.dist.php

@@ -30,6 +30,7 @@
         '@Symfony:risky' => true,
         'protected_to_private' => false,
         'native_constant_invocation' => ['strict' => false],
+        'nullable_type_declaration_for_default_null_value' => true,
         'header_comment' => ['header' => $fileHeaderComment],
     ])
     ->setRiskyAllowed(true)

src/Symfony/Bridge/Doctrine/ContainerAwareEventManager.php

@@ -50,7 +50,7 @@ public function __construct(ContainerInterface $container, array $subscriberIds
      *
      * @return void
      */
-    public function dispatchEvent($eventName, EventArgs $eventArgs = null)
+    public function dispatchEvent($eventName, ?EventArgs $eventArgs = null)
     {
         if (!$this->initializedSubscribers) {
             $this->initializeSubscribers();

src/Symfony/Bridge/Doctrine/DataCollector/DoctrineDataCollector.php

@@ -39,7 +39,7 @@ class DoctrineDataCollector extends DataCollector
      */
     private $loggers = [];
 
-    public function __construct(ManagerRegistry $registry, DebugDataHolder $debugDataHolder = null)
+    public function __construct(ManagerRegistry $registry, ?DebugDataHolder $debugDataHolder = null)
     {
         $this->registry = $registry;
         $this->connections = $registry->getConnectionNames();
@@ -58,7 +58,7 @@ public function addLogger(string $name, DebugStack $logger)
     /**
      * {@inheritdoc}
      */
-    public function collect(Request $request, Response $response, \Throwable $exception = null)
+    public function collect(Request $request, Response $response, ?\Throwable $exception = null)
     {
         $this->data = [
             'queries' => $this->collectQueries(),

src/Symfony/Bridge/Doctrine/Form/ChoiceList/DoctrineChoiceLoader.php

@@ -35,7 +35,7 @@ class DoctrineChoiceLoader extends AbstractChoiceLoader
      *
      * @param string $class The class name of the loaded objects
      */
-    public function __construct(ObjectManager $manager, string $class, IdReader $idReader = null, EntityLoaderInterface $objectLoader = null)
+    public function __construct(ObjectManager $manager, string $class, ?IdReader $idReader = null, ?EntityLoaderInterface $objectLoader = null)
     {
         $classMetadata = $manager->getClassMetadata($class);
 

src/Symfony/Bridge/Doctrine/Form/ChoiceList/IdReader.php

@@ -78,7 +78,7 @@ public function isIntId(): bool
      *
      * This method assumes that the object has a single-column ID.
      */
-    public function getIdValue(object $object = null): string
+    public function getIdValue(?object $object = null): string
     {
         if (!$object) {
             return '';

src/Symfony/Bridge/Doctrine/IdGenerator/UlidGenerator.php

@@ -21,7 +21,7 @@ final class UlidGenerator extends AbstractIdGenerator
 {
     private $factory;
 
-    public function __construct(UlidFactory $factory = null)
+    public function __construct(?UlidFactory $factory = null)
     {
         $this->factory = $factory;
     }

src/Symfony/Bridge/Doctrine/IdGenerator/UuidGenerator.php

@@ -23,7 +23,7 @@ final class UuidGenerator extends AbstractIdGenerator
     private $factory;
     private $entityGetter;
 
-    public function __construct(UuidFactory $factory = null)
+    public function __construct(?UuidFactory $factory = null)
     {
         $this->protoFactory = $this->factory = $factory ?? new UuidFactory();
     }

src/Symfony/Bridge/Doctrine/Logger/DbalLogger.php

@@ -26,7 +26,7 @@ class DbalLogger implements SQLLogger
     protected $logger;
     protected $stopwatch;
 
-    public function __construct(LoggerInterface $logger = null, Stopwatch $stopwatch = null)
+    public function __construct(?LoggerInterface $logger = null, ?Stopwatch $stopwatch = null)
     {
         $this->logger = $logger;
         $this->stopwatch = $stopwatch;
@@ -37,7 +37,7 @@ public function __construct(LoggerInterface $logger = null, Stopwatch $stopwatch
      *
      * @return void
      */
-    public function startQuery($sql, array $params = null, array $types = null)
+    public function startQuery($sql, ?array $params = null, ?array $types = null)
     {
         if (null !== $this->stopwatch) {
             $this->stopwatch->start('doctrine', 'doctrine');

src/Symfony/Bridge/Doctrine/Messenger/AbstractDoctrineMiddleware.php

@@ -28,7 +28,7 @@ abstract class AbstractDoctrineMiddleware implements MiddlewareInterface
     protected $managerRegistry;
     protected $entityManagerName;
 
-    public function __construct(ManagerRegistry $managerRegistry, string $entityManagerName = null)
+    public function __construct(ManagerRegistry $managerRegistry, ?string $entityManagerName = null)
     {
         $this->managerRegistry = $managerRegistry;
         $this->entityManagerName = $entityManagerName;

src/Symfony/Bridge/Doctrine/Messenger/DoctrineOpenTransactionLoggerMiddleware.php

@@ -29,7 +29,7 @@ class DoctrineOpenTransactionLoggerMiddleware extends AbstractDoctrineMiddleware
     /** @var bool */
     private $isHandling = false;
 
-    public function __construct(ManagerRegistry $managerRegistry, string $entityManagerName = null, LoggerInterface $logger = null)
+    public function __construct(ManagerRegistry $managerRegistry, ?string $entityManagerName = null, ?LoggerInterface $logger = null)
     {
         parent::__construct($managerRegistry, $entityManagerName);
 

src/Symfony/Bridge/Doctrine/Middleware/Debug/DBAL3/Statement.php

@@ -36,7 +36,7 @@ public function __construct(
         DebugDataHolder $debugDataHolder,
         string $connectionName,
         string $sql,
-        Stopwatch $stopwatch = null
+        ?Stopwatch $stopwatch = null
     ) {
         $this->stopwatch = $stopwatch;
         $this->connectionName = $connectionName;

src/Symfony/Bridge/Doctrine/Security/User/EntityUserProvider.php

@@ -39,7 +39,7 @@ class EntityUserProvider implements UserProviderInterface, PasswordUpgraderInter
     private $class;
     private $property;
 
-    public function __construct(ManagerRegistry $registry, string $classOrAlias, string $property = null, string $managerName = null)
+    public function __construct(ManagerRegistry $registry, string $classOrAlias, ?string $property = null, ?string $managerName = null)
     {
         $this->registry = $registry;
         $this->managerName = $managerName;

src/Symfony/Bridge/Doctrine/Test/DoctrineTestHelper.php

@@ -40,7 +40,7 @@ class DoctrineTestHelper
      *
      * @return EntityManager
      */
-    public static function createTestEntityManager(Configuration $config = null)
+    public static function createTestEntityManager(?Configuration $config = null)
     {
         if (!\extension_loaded('pdo_sqlite')) {
             TestCase::markTestSkipped('Extension pdo_sqlite is required.');

src/Symfony/Bridge/Doctrine/Tests/Form/Type/EntityTypeTest.php

@@ -783,7 +783,7 @@ public function testOverrideChoicesValuesWithCallable()
             'em' => 'default',
             'class' => self::ITEM_GROUP_CLASS,
             'choice_label' => 'name',
-            'choice_value' => function (GroupableEntity $entity = null) {
+            'choice_value' => function (?GroupableEntity $entity = null) {
                 if (null === $entity) {
                     return '';
                 }

src/Symfony/Bridge/Doctrine/Tests/PropertyInfo/DoctrineExtractorTest.php

@@ -123,7 +123,7 @@ public function testTestGetPropertiesWithEmbedded()
     /**
      * @dataProvider typesProvider
      */
-    public function testExtract(string $property, array $type = null)
+    public function testExtract(string $property, ?array $type = null)
     {
         $this->assertEquals($type, $this->createExtractor()->getTypes(DoctrineDummy::class, $property, []));
     }

src/Symfony/Bridge/Doctrine/Tests/Validator/DoctrineLoaderTest.php

@@ -211,7 +211,7 @@ public function testFieldMappingsConfiguration()
     /**
      * @dataProvider regexpProvider
      */
-    public function testClassValidator(bool $expected, string $classValidatorRegexp = null)
+    public function testClassValidator(bool $expected, ?string $classValidatorRegexp = null)
     {
         $doctrineLoader = new DoctrineLoader(DoctrineTestHelper::createTestEntityManager(), $classValidatorRegexp, false);
 

src/Symfony/Bridge/Doctrine/Validator/Constraints/UniqueEntity.php

@@ -46,14 +46,14 @@ class UniqueEntity extends Constraint
      */
     public function __construct(
         $fields,
-        string $message = null,
-        string $service = null,
-        string $em = null,
-        string $entityClass = null,
-        string $repositoryMethod = null,
-        string $errorPath = null,
-        bool $ignoreNull = null,
-        array $groups = null,
+        ?string $message = null,
+        ?string $service = null,
+        ?string $em = null,
+        ?string $entityClass = null,
+        ?string $repositoryMethod = null,
+        ?string $errorPath = null,
+        ?bool $ignoreNull = null,
+        ?array $groups = null,
         $payload = null,
         array $options = []
     ) {

src/Symfony/Bridge/Doctrine/Validator/DoctrineLoader.php

@@ -36,7 +36,7 @@ final class DoctrineLoader implements LoaderInterface
     private $entityManager;
     private $classValidatorRegexp;
 
-    public function __construct(EntityManagerInterface $entityManager, string $classValidatorRegexp = null)
+    public function __construct(EntityManagerInterface $entityManager, ?string $classValidatorRegexp = null)
     {
         $this->entityManager = $entityManager;
         $this->classValidatorRegexp = $classValidatorRegexp;

src/Symfony/Bridge/Monolog/Formatter/ConsoleFormatter.php

@@ -182,7 +182,7 @@ private function replacePlaceHolder(array $record): array
         return $record;
     }
 
-    private function dumpData($data, bool $colors = null): string
+    private function dumpData($data, ?bool $colors = null): string
     {
         if (null === $this->dumper) {
             return '';

src/Symfony/Bridge/Monolog/Formatter/VarDumperFormatter.php

@@ -21,7 +21,7 @@ class VarDumperFormatter implements FormatterInterface
 {
     private $cloner;
 
-    public function __construct(VarCloner $cloner = null)
+    public function __construct(?VarCloner $cloner = null)
     {
         $this->cloner = $cloner ?? new VarCloner();
     }

src/Symfony/Bridge/Monolog/Handler/ConsoleHandler.php

@@ -60,7 +60,7 @@ class ConsoleHandler extends AbstractProcessingHandler implements EventSubscribe
      * @param array                $verbosityLevelMap Array that maps the OutputInterface verbosity to a minimum logging
      *                                                level (leave empty to use the default mapping)
      */
-    public function __construct(OutputInterface $output = null, bool $bubble = true, array $verbosityLevelMap = [], array $consoleFormatterOptions = [])
+    public function __construct(?OutputInterface $output = null, bool $bubble = true, array $verbosityLevelMap = [], array $consoleFormatterOptions = [])
     {
         parent::__construct(Logger::DEBUG, $bubble);
         $this->output = $output;

src/Symfony/Bridge/Monolog/Handler/ElasticsearchLogstashHandler.php

@@ -58,7 +58,7 @@ class ElasticsearchLogstashHandler extends AbstractHandler
     /**
      * @param string|int $level The minimum logging level at which this handler will be triggered
      */
-    public function __construct(string $endpoint = 'http://127.0.0.1:9200', string $index = 'monolog', HttpClientInterface $client = null, $level = Logger::DEBUG, bool $bubble = true, string $elasticsearchVersion = '1.0.0')
+    public function __construct(string $endpoint = 'http://127.0.0.1:9200', string $index = 'monolog', ?HttpClientInterface $client = null, $level = Logger::DEBUG, bool $bubble = true, string $elasticsearchVersion = '1.0.0')
     {
         if (!interface_exists(HttpClientInterface::class)) {
             throw new \LogicException(sprintf('The "%s" handler needs an HTTP client. Try running "composer require symfony/http-client".', __CLASS__));

src/Symfony/Bridge/Monolog/Logger.php

@@ -25,7 +25,7 @@ class Logger extends BaseLogger implements DebugLoggerInterface, ResetInterface
     /**
      * {@inheritdoc}
      */
-    public function getLogs(Request $request = null)
+    public function getLogs(?Request $request = null)
     {
         if ($logger = $this->getDebugLogger()) {
             return $logger->getLogs($request);
@@ -37,7 +37,7 @@ public function getLogs(Request $request = null)
     /**
      * {@inheritdoc}
      */
-    public function countErrors(Request $request = null)
+    public function countErrors(?Request $request = null)
     {
         if ($logger = $this->getDebugLogger()) {
             return $logger->countErrors($request);

src/Symfony/Bridge/Monolog/Processor/DebugProcessor.php

@@ -23,7 +23,7 @@ class DebugProcessor implements DebugLoggerInterface, ResetInterface
     private $errorCount = [];
     private $requestStack;
 
-    public function __construct(RequestStack $requestStack = null)
+    public function __construct(?RequestStack $requestStack = null)
     {
         $this->requestStack = $requestStack;
     }
@@ -68,7 +68,7 @@ public function __invoke(array $record)
     /**
      * {@inheritdoc}
      */
-    public function getLogs(Request $request = null)
+    public function getLogs(?Request $request = null)
     {
         if (null !== $request) {
             return $this->records[spl_object_hash($request)] ?? [];
@@ -84,7 +84,7 @@ public function getLogs(Request $request = null)
     /**
      * {@inheritdoc}
      */
-    public function countErrors(Request $request = null)
+    public function countErrors(?Request $request = null)
     {
         if (null !== $request) {
             return $this->errorCount[spl_object_hash($request)] ?? 0;

src/Symfony/Bridge/Monolog/Processor/WebProcessor.php

@@ -25,7 +25,7 @@
  */
 class WebProcessor extends BaseWebProcessor implements EventSubscriberInterface
 {
-    public function __construct(array $extraFields = null)
+    public function __construct(?array $extraFields = null)
     {
         // Pass an empty array as the default null value would access $_SERVER
         parent::__construct([], $extraFields);

src/Symfony/Bridge/Monolog/Tests/ClassThatInheritLogger.php

@@ -16,12 +16,12 @@
 
 class ClassThatInheritLogger extends Logger
 {
-    public function getLogs(Request $request = null): array
+    public function getLogs(?Request $request = null): array
     {
         return parent::getLogs($request);
     }
 
-    public function countErrors(Request $request = null): int
+    public function countErrors(?Request $request = null): int
     {
         return parent::countErrors($request);
     }

src/Symfony/Bridge/Monolog/Tests/Processor/ClassThatInheritDebugProcessor.php

@@ -16,12 +16,12 @@
 
 class ClassThatInheritDebugProcessor extends DebugProcessor
 {
-    public function getLogs(Request $request = null): array
+    public function getLogs(?Request $request = null): array
     {
         return parent::getLogs($request);
     }
 
-    public function countErrors(Request $request = null): int
+    public function countErrors(?Request $request = null): int
     {
         return parent::countErrors($request);
     }

src/Symfony/Bridge/PhpUnit/CoverageListener.php

@@ -26,7 +26,7 @@ class CoverageListener implements TestListener
     private $sutFqcnResolver;
     private $warningOnSutNotFound;
 
-    public function __construct(callable $sutFqcnResolver = null, bool $warningOnSutNotFound = false)
+    public function __construct(?callable $sutFqcnResolver = null, bool $warningOnSutNotFound = false)
     {
         $this->sutFqcnResolver = $sutFqcnResolver ?? static function (Test $test): ?string {
             $class = \get_class($test);

src/Symfony/Bridge/Twig/Command/DebugCommand.php

@@ -44,7 +44,7 @@ class DebugCommand extends Command
     private $filesystemLoaders;
     private $fileLinkFormatter;
 
-    public function __construct(Environment $twig, string $projectDir = null, array $bundlesMetadata = [], string $twigDefaultPath = null, FileLinkFormatter $fileLinkFormatter = null)
+    public function __construct(Environment $twig, ?string $projectDir = null, array $bundlesMetadata = [], ?string $twigDefaultPath = null, ?FileLinkFormatter $fileLinkFormatter = null)
     {
         parent::__construct();
 
@@ -218,7 +218,7 @@ private function displayPathsJson(SymfonyStyle $io, string $name)
         $io->writeln(json_encode($data));
     }
 
-    private function displayGeneralText(SymfonyStyle $io, string $filter = null)
+    private function displayGeneralText(SymfonyStyle $io, ?string $filter = null)
     {
         $decorated = $io->isDecorated();
         $types = ['functions', 'filters', 'tests', 'globals'];
@@ -280,7 +280,7 @@ private function displayGeneralJson(SymfonyStyle $io, ?string $filter)
         $io->writeln($decorated ? OutputFormatter::escape($data) : $data);
     }
 
-    private function getLoaderPaths(string $name = null): array
+    private function getLoaderPaths(?string $name = null): array
     {
         $loaderPaths = [];
         foreach ($this->getFilesystemLoaders() as $loader) {

src/Symfony/Bridge/Twig/Command/LintCommand.php

@@ -236,7 +236,7 @@ private function displayJson(OutputInterface $output, array $filesInfo)
         return min($errors, 1);
     }
 
-    private function renderException(SymfonyStyle $output, string $template, Error $exception, string $file = null, GithubActionReporter $githubReporter = null)
+    private function renderException(SymfonyStyle $output, string $template, Error $exception, ?string $file = null, ?GithubActionReporter $githubReporter = null)
     {
         $line = $exception->getTemplateLine();
 

src/Symfony/Bridge/Twig/DataCollector/TwigDataCollector.php

@@ -32,7 +32,7 @@ class TwigDataCollector extends DataCollector implements LateDataCollectorInterf
     private $twig;
     private $computed;
 
-    public function __construct(Profile $profile, Environment $twig = null)
+    public function __construct(Profile $profile, ?Environment $twig = null)
     {
         $this->profile = $profile;
         $this->twig = $twig;
@@ -41,7 +41,7 @@ public function __construct(Profile $profile, Environment $twig = null)
     /**
      * {@inheritdoc}
      */
-    public function collect(Request $request, Response $response, \Throwable $exception = null)
+    public function collect(Request $request, Response $response, ?\Throwable $exception = null)
     {
     }
 

src/Symfony/Bridge/Twig/ErrorRenderer/TwigErrorRenderer.php

@@ -32,7 +32,7 @@ class TwigErrorRenderer implements ErrorRendererInterface
     /**
      * @param bool|callable $debug The debugging mode as a boolean or a callable that should return it
      */
-    public function __construct(Environment $twig, HtmlErrorRenderer $fallbackErrorRenderer = null, $debug = false)
+    public function __construct(Environment $twig, ?HtmlErrorRenderer $fallbackErrorRenderer = null, $debug = false)
     {
         if (!\is_bool($debug) && !\is_callable($debug)) {
             throw new \TypeError(sprintf('Argument 3 passed to "%s()" must be a boolean or a callable, "%s" given.', __METHOD__, get_debug_type($debug)));