Bump minimum version of PHP to 8.1

Author: nicolas-grekas

Command/ConfigDebugCommand.php

@@ -127,7 +127,6 @@ private function compileContainer(): ContainerBuilder
         $kernel->boot();
 
         $method = new \ReflectionMethod($kernel, 'buildContainer');
-        $method->setAccessible(true);
         $container = $method->invoke($kernel);
         $container->getCompiler()->compile($container);
 

Command/ContainerDebugCommand.php

@@ -132,7 +132,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             $options = ['env-vars' => true, 'name' => $envVar];
         } elseif ($input->getOption('types')) {
             $options = [];
-            $options['filter'] = [$this, 'filterToServiceTypes'];
+            $options['filter'] = $this->filterToServiceTypes(...);
         } elseif ($input->getOption('parameters')) {
             $parameters = [];
             foreach ($object->getParameterBag()->all() as $k => $v) {

Command/ContainerLintCommand.php

@@ -104,7 +104,6 @@ private function getContainerBuilder(): ContainerBuilder
             (new XmlFileLoader($container = new ContainerBuilder($parameterBag = new EnvPlaceholderParameterBag()), new FileLocator()))->load($kernelContainer->getParameter('debug.container.dump'));
 
             $refl = new \ReflectionProperty($parameterBag, 'resolved');
-            $refl->setAccessible(true);
             $refl->setValue($parameterBag, true);
 
             $skippedIds = [];

Command/DebugAutowiringCommand.php

@@ -78,7 +78,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
         $builder = $this->getContainerBuilder($this->getApplication()->getKernel());
         $serviceIds = $builder->getServiceIds();
-        $serviceIds = array_filter($serviceIds, [$this, 'filterToServiceTypes']);
+        $serviceIds = array_filter($serviceIds, $this->filterToServiceTypes(...));
 
         if ($search = $input->getArgument('search')) {
             $searchNormalized = preg_replace('/[^a-zA-Z0-9\x7f-\xff $]++/', '', $search);
@@ -169,7 +169,7 @@ public function complete(CompletionInput $input, CompletionSuggestions $suggesti
         if ($input->mustSuggestArgumentValuesFor('search')) {
             $builder = $this->getContainerBuilder($this->getApplication()->getKernel());
 
-            $suggestions->suggestValues(array_filter($builder->getServiceIds(), [$this, 'filterToServiceTypes']));
+            $suggestions->suggestValues(array_filter($builder->getServiceIds(), $this->filterToServiceTypes(...)));
         }
     }
 }

Console/Descriptor/Descriptor.php

@@ -326,7 +326,6 @@ private function getContainerEnvVars(ContainerBuilder $container): array
         $getDefaultParameter = $getDefaultParameter->bindTo($bag, \get_class($bag));
 
         $getEnvReflection = new \ReflectionMethod($container, 'getEnv');
-        $getEnvReflection->setAccessible(true);
 
         $envs = [];
 

DependencyInjection/FrameworkExtension.php

@@ -248,7 +248,7 @@ public function load(array $configs, ContainerBuilder $container)
 
         $loader->load('web.php');
 
-        if (\PHP_VERSION_ID < 80100 || !class_exists(BackedEnumValueResolver::class)) {
+        if (!class_exists(BackedEnumValueResolver::class)) {
             $container->removeDefinition('argument_resolver.backed_enum_resolver');
         }
 

Routing/Router.php

@@ -49,9 +49,9 @@ public function __construct(ContainerInterface $container, mixed $resource, arra
         $this->setOptions($options);
 
         if ($parameters) {
-            $this->paramFetcher = \Closure::fromCallable([$parameters, 'get']);
+            $this->paramFetcher = $parameters->get(...);
         } elseif ($container instanceof SymfonyContainerInterface) {
-            $this->paramFetcher = \Closure::fromCallable([$container, 'getParameter']);
+            $this->paramFetcher = $container->getParameter(...);
         } else {
             throw new \LogicException(sprintf('You should either pass a "%s" instance or provide the $parameters argument of the "%s" method.', SymfonyContainerInterface::class, __METHOD__));
         }

Tests/CacheWarmer/AnnotationsCacheWarmerTest.php

@@ -151,10 +151,7 @@ public function testWarmupRemoveCacheMisses()
         $this->assertTrue(isset($data[0]['bar_hit']));
     }
 
-    /**
-     * @return MockObject&Reader
-     */
-    private function getReadOnlyReader(): Reader
+    private function getReadOnlyReader(): MockObject&Reader
     {
         $readerMock = $this->createMock(Reader::class);
         $readerMock->expects($this->exactly(0))->method('getClassAnnotations');

Tests/Command/CachePoolClearCommandTest.php

@@ -52,10 +52,7 @@ public function provideCompletionSuggestions()
         ];
     }
 
-    /**
-     * @return MockObject&KernelInterface
-     */
-    private function getKernel(): KernelInterface
+    private function getKernel(): MockObject&KernelInterface
     {
         $container = $this->createMock(ContainerInterface::class);
 

Tests/Command/CachePoolDeleteCommandTest.php

@@ -106,10 +106,7 @@ public function provideCompletionSuggestions()
         ];
     }
 
-    /**
-     * @return MockObject&KernelInterface
-     */
-    private function getKernel(): KernelInterface
+    private function getKernel(): MockObject&KernelInterface
     {
         $container = $this->createMock(ContainerInterface::class);
 

Tests/Command/CachePruneCommandTest.php

@@ -50,10 +50,7 @@ private function getEmptyRewindableGenerator(): RewindableGenerator
         }, 0);
     }
 
-    /**
-     * @return MockObject&KernelInterface
-     */
-    private function getKernel(): KernelInterface
+    private function getKernel(): MockObject&KernelInterface
     {
         $container = $this->createMock(ContainerInterface::class);
 
@@ -71,10 +68,7 @@ private function getKernel(): KernelInterface
         return $kernel;
     }
 
-    /**
-     * @return MockObject&PruneableInterface
-     */
-    private function getPruneableInterfaceMock(): PruneableInterface
+    private function getPruneableInterfaceMock(): MockObject&PruneableInterface
     {
         $pruneable = $this->createMock(PruneableInterface::class);
         $pruneable

Tests/Console/Descriptor/AbstractDescriptorTest.php

@@ -122,9 +122,7 @@ public function getDescribeContainerDefinitionWithArgumentsShownTestData()
             $definitionsWithArgs[str_replace('definition_', 'definition_arguments_', $key)] = $definition;
         }
 
-        if (\PHP_VERSION_ID >= 80100) {
-            $definitionsWithArgs['definition_arguments_with_enum'] = (new Definition('definition_with_enum'))->setArgument(0, FooUnitEnum::FOO);
-        }
+        $definitionsWithArgs['definition_arguments_with_enum'] = (new Definition('definition_with_enum'))->setArgument(0, FooUnitEnum::FOO);
 
         return $this->getDescriptionTestData($definitionsWithArgs);
     }

Tests/Console/Descriptor/ObjectsProvider.php

@@ -71,10 +71,6 @@ public static function getContainerParameters()
             'array' => [12, 'Hello world!', true],
         ]);
 
-        if (\PHP_VERSION_ID < 80100) {
-            return;
-        }
-
         yield 'parameters_enums' => new ParameterBag([
             'unit_enum' => FooUnitEnum::BAR,
             'backed_enum' => Suit::Hearts,
@@ -259,7 +255,7 @@ public static function getCallables()
             'callable_5' => ['Symfony\\Bundle\\FrameworkBundle\\Tests\\Console\\Descriptor\\ExtendedCallableClass', 'parent::staticMethod'],
             'callable_6' => function () { return 'Closure'; },
             'callable_7' => new CallableClass(),
-            'callable_from_callable' => \Closure::fromCallable(new CallableClass()),
+            'callable_from_callable' => (new CallableClass())(...),
         ];
     }
 }

Tests/Functional/CachePoolClearCommandTest.php

@@ -86,7 +86,6 @@ public function testClearFailed()
         $pool->save($item);
         $r = new \ReflectionObject($pool);
         $p = $r->getProperty('directory');
-        $p->setAccessible(true);
         $poolDir = $p->getValue($pool);
 
         /** @var SplFileInfo $entry */

composer.json

@@ -16,7 +16,7 @@
         }
     ],
     "require": {
-        "php": ">=8.0.2",
+        "php": ">=8.1",
         "composer-runtime-api": ">=2.1",
         "ext-xml": "*",
         "symfony/cache": "^5.4|^6.0",
@@ -27,7 +27,6 @@
         "symfony/http-foundation": "^5.4|^6.0",
         "symfony/http-kernel": "^5.4|^6.0",
         "symfony/polyfill-mbstring": "~1.0",
-        "symfony/polyfill-php81": "^1.22",
         "symfony/filesystem": "^5.4|^6.0",
         "symfony/finder": "^5.4|^6.0",
         "symfony/routing": "^5.4|^6.0"