Fix implicitly-required parameters

Author: nicolas-grekas

src/Symfony/Bundle/FrameworkBundle/Tests/Functional/Bundle/TestBundle/AutowiringTypes/AutowiredServices.php

@@ -21,7 +21,7 @@ class AutowiredServices
     private $dispatcher;
     private $cachePool;
 
-    public function __construct(?Reader $annotationReader = null, EventDispatcherInterface $dispatcher, CacheItemPoolInterface $cachePool)
+    public function __construct(?Reader $annotationReader, EventDispatcherInterface $dispatcher, CacheItemPoolInterface $cachePool)
     {
         $this->annotationReader = $annotationReader;
         $this->dispatcher = $dispatcher;

src/Symfony/Bundle/WebProfilerBundle/Controller/ProfilerController.php

@@ -40,7 +40,7 @@ class ProfilerController
     private $cspHandler;
     private $baseDir;
 
-    public function __construct(UrlGeneratorInterface $generator, ?Profiler $profiler = null, Environment $twig, array $templates, ?ContentSecurityPolicyHandler $cspHandler = null, ?string $baseDir = null)
+    public function __construct(UrlGeneratorInterface $generator, ?Profiler $profiler, Environment $twig, array $templates, ?ContentSecurityPolicyHandler $cspHandler = null, ?string $baseDir = null)
     {
         $this->generator = $generator;
         $this->profiler = $profiler;

src/Symfony/Component/Console/Tests/CI/GithubActionReporterTest.php

@@ -34,7 +34,7 @@ public function testIsGithubActionEnvironment()
     /**
      * @dataProvider annotationsFormatProvider
      */
-    public function testAnnotationsFormat(string $type, string $message, ?string $file = null, ?int $line = null, ?int $col = null, string $expected)
+    public function testAnnotationsFormat(string $type, string $message, ?string $file, ?int $line, ?int $col, string $expected)
     {
         $reporter = new GithubActionReporter($buffer = new BufferedOutput());
 

src/Symfony/Component/Form/Extension/DataCollector/FormDataCollector.php

@@ -298,7 +298,7 @@ private function &recursiveBuildPreliminaryFormTree(FormInterface $form, array &
         return $output;
     }
 
-    private function &recursiveBuildFinalFormTree(?FormInterface $form = null, FormView $view, array &$outputByHash)
+    private function &recursiveBuildFinalFormTree(?FormInterface $form, FormView $view, array &$outputByHash)
     {
         $viewHash = spl_object_hash($view);
         $formHash = null;

src/Symfony/Component/HttpKernel/Fragment/AbstractSurrogateFragmentRenderer.php

@@ -34,7 +34,7 @@ abstract class AbstractSurrogateFragmentRenderer extends RoutableFragmentRendere
      *
      * @param FragmentRendererInterface $inlineStrategy The inline strategy to use when the surrogate is not supported
      */
-    public function __construct(?SurrogateInterface $surrogate = null, FragmentRendererInterface $inlineStrategy, ?UriSigner $signer = null)
+    public function __construct(?SurrogateInterface $surrogate, FragmentRendererInterface $inlineStrategy, ?UriSigner $signer = null)
     {
         $this->surrogate = $surrogate;
         $this->inlineStrategy = $inlineStrategy;

src/Symfony/Component/HttpKernel/Tests/DependencyInjection/RegisterControllerArgumentLocatorsPassTest.php

@@ -512,7 +512,7 @@ public function fooAction(?NonExistentClass $nonExistent = null)
     {
     }
 
-    public function barAction(?NonExistentClass $nonExistent = null, $bar)
+    public function barAction(?NonExistentClass $nonExistent, $bar)
     {
     }
 }

src/Symfony/Component/PropertyInfo/Tests/Extractor/PhpDocExtractorTest.php

@@ -157,7 +157,7 @@ public static function typesProvider()
     /**
      * @dataProvider provideCollectionTypes
      */
-    public function testExtractCollection($property, ?array $type = null, $shortDescription, $longDescription)
+    public function testExtractCollection($property, ?array $type, $shortDescription, $longDescription)
     {
         if (!class_exists(Collection::class)) {
             $this->markTestSkipped('Collections are not implemented in current phpdocumentor/type-resolver version');

src/Symfony/Component/Validator/Context/ExecutionContext.php

@@ -152,7 +152,7 @@ public function __construct(ValidatorInterface $validator, $root, TranslatorInte
     /**
      * {@inheritdoc}
      */
-    public function setNode($value, ?object $object, ?MetadataInterface $metadata = null, string $propertyPath)
+    public function setNode($value, ?object $object, ?MetadataInterface $metadata, string $propertyPath)
     {
         $this->value = $value;
         $this->object = $object;

src/Symfony/Component/Validator/Context/ExecutionContextInterface.php

@@ -129,7 +129,7 @@ public function getObject();
      * @param object|null $object       The currently validated object
      * @param string      $propertyPath The property path to the current value
      */
-    public function setNode($value, ?object $object, ?MetadataInterface $metadata = null, string $propertyPath);
+    public function setNode($value, ?object $object, ?MetadataInterface $metadata, string $propertyPath);
 
     /**
      * Warning: Should not be called by user code, to be used by the validator engine only.

src/Symfony/Component/VarDumper/Tests/Caster/ReflectionCasterTest.php

@@ -727,6 +727,6 @@ public static function stub(): void
     }
 }
 
-function reflectionParameterFixture(?NotLoadableClass $arg1 = null, $arg2)
+function reflectionParameterFixture(?NotLoadableClass $arg1, $arg2)
 {
 }