minor symfony#53615 fix method signatures (xabbuh)

This PR was merged into the 6.3 branch.

Discussion
----------

fix method signatures

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        |
| License       | MIT

Commits
-------

5c7aa68 fix method signatures

Author: nicolas-grekas

.github/expected-missing-return-types.diff

@@ -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/FrameworkBundle/Tests/Functional/Bundle/TestBundle/AutowiringTypes/AutowiredServices.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/Bundle/WebProfilerBundle/Controller/ProfilerController.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/Console/Tests/CI/GithubActionReporterTest.php

@@ -259,7 +259,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/Form/Extension/DataCollector/FormDataCollector.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/Fragment/AbstractSurrogateFragmentRenderer.php

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

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

@@ -134,7 +134,7 @@ public static function typesProvider()
     /**
      * @dataProvider provideCollectionTypes
      */
-    public function testExtractCollection($property, ?array $type = null, $shortDescription, $longDescription)
+    public function testExtractCollection($property, ?array $type, $shortDescription, $longDescription)
     {
         $this->testExtract($property, $type, $shortDescription, $longDescription);
     }

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

@@ -121,7 +121,7 @@ public function __construct(ValidatorInterface $validator, mixed $root, Translat
         $this->cachedObjectsRefs = new \SplObjectStorage();
     }
 
-    public function setNode(mixed $value, ?object $object, ?MetadataInterface $metadata = null, string $propertyPath): void
+    public function setNode(mixed $value, ?object $object, ?MetadataInterface $metadata, string $propertyPath): void
     {
         $this->value = $value;
         $this->object = $object;

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

@@ -126,7 +126,7 @@ public function getObject(): ?object;
      *
      * @return void
      */
-    public function setNode(mixed $value, ?object $object, ?MetadataInterface $metadata = null, string $propertyPath);
+    public function setNode(mixed $value, ?object $object, ?MetadataInterface $metadata, string $propertyPath);
 
     /**
      * Warning: Should not be called by user code, to be used by the validator engine only.