minor #47401 Replace get_class() calls by ::class (deguif)

fabpot · fabpot · commit 6e2d05e8ce48 · 2022-09-01T04:03:22.000+02:00
This PR was squashed before being merged into the 6.2 branch.

Discussion
----------

Replace get_class() calls by ::class

| Q             | A
| ------------- | ---
| Branch?       | 6.2
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       |
| License       | MIT
| Doc PR        |

Replace `get_class()` by `::class`

Commits
-------

675601b8f7 Replace get_class() calls by ::class
diff --git a/CacheWarmer/ConfigBuilderCacheWarmer.php b/CacheWarmer/ConfigBuilderCacheWarmer.php
@@ -53,7 +53,7 @@ public function warmUp(string $cacheDir): array
             try {
                 $this->dumpExtension($extension, $generator);
             } catch (\Exception $e) {
-                $this->logger?->warning('Failed to generate ConfigBuilder for extension {extensionClass}.', ['exception' => $e, 'extensionClass' => \get_class($extension)]);
+                $this->logger?->warning('Failed to generate ConfigBuilder for extension {extensionClass}.', ['exception' => $e, 'extensionClass' => $extension::class]);
             }
         }
 
diff --git a/Command/AboutCommand.php b/Command/AboutCommand.php
@@ -67,7 +67,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             new TableSeparator(),
             ['<info>Kernel</>'],
             new TableSeparator(),
-            ['Type', \get_class($kernel)],
+            ['Type', $kernel::class],
             ['Environment', $kernel->getEnvironment()],
             ['Debug', $kernel->isDebug() ? 'true' : 'false'],
             ['Charset', $kernel->getCharset()],
diff --git a/Command/BuildDebugContainerTrait.php b/Command/BuildDebugContainerTrait.php
@@ -44,7 +44,7 @@ protected function getContainerBuilder(KernelInterface $kernel): ContainerBuilde
                 $this->initializeBundles();
 
                 return $this->buildContainer();
-            }, $kernel, \get_class($kernel));
+            }, $kernel, $kernel::class);
             $container = $buildContainer();
             $container->getCompilerPassConfig()->setRemovingPasses([]);
             $container->getCompilerPassConfig()->setAfterRemovingPasses([]);
diff --git a/Command/ContainerLintCommand.php b/Command/ContainerLintCommand.php
@@ -86,7 +86,7 @@ private function getContainerBuilder(): ContainerBuilder
                 $this->initializeBundles();
 
                 return $this->buildContainer();
-            }, $kernel, \get_class($kernel));
+            }, $kernel, $kernel::class);
             $container = $buildContainer();
 
             $skippedIds = [];
diff --git a/Console/Descriptor/Descriptor.php b/Console/Descriptor/Descriptor.php
@@ -158,7 +158,7 @@ protected function formatValue(mixed $value): string
         }
 
         if (\is_object($value)) {
-            return sprintf('object(%s)', \get_class($value));
+            return sprintf('object(%s)', $value::class);
         }
 
         if (\is_string($value)) {
@@ -335,7 +335,7 @@ private function getContainerEnvVars(ContainerBuilder $container): array
         $getDefaultParameter = function (string $name) {
             return parent::get($name);
         };
-        $getDefaultParameter = $getDefaultParameter->bindTo($bag, \get_class($bag));
+        $getDefaultParameter = $getDefaultParameter->bindTo($bag, $bag::class);
 
         $getEnvReflection = new \ReflectionMethod($container, 'getEnv');
 
diff --git a/Console/Descriptor/JsonDescriptor.php b/Console/Descriptor/JsonDescriptor.php
@@ -78,7 +78,7 @@ protected function describeContainerService(object $service, array $options = []
         } elseif ($service instanceof Definition) {
             $this->writeData($this->getContainerDefinitionData($service, isset($options['omit_tags']) && $options['omit_tags'], isset($options['show_arguments']) && $options['show_arguments'], $builder, $options['id']), $options);
         } else {
-            $this->writeData(\get_class($service), $options);
+            $this->writeData($service::class, $options);
         }
     }
 
@@ -108,7 +108,7 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o
             } elseif ($service instanceof Definition) {
                 $data['definitions'][$serviceId] = $this->getContainerDefinitionData($service, $omitTags, $showArguments, $builder, $serviceId);
             } else {
-                $data['services'][$serviceId] = \get_class($service);
+                $data['services'][$serviceId] = $service::class;
             }
         }
 
@@ -204,7 +204,7 @@ protected function getRouteData(Route $route): array
             'hostRegex' => '' !== $route->getHost() ? $route->compile()->getHostRegex() : '',
             'scheme' => $route->getSchemes() ? implode('|', $route->getSchemes()) : 'ANY',
             'method' => $route->getMethods() ? implode('|', $route->getMethods()) : 'ANY',
-            'class' => \get_class($route),
+            'class' => $route::class,
             'defaults' => $route->getDefaults(),
             'requirements' => $route->getRequirements() ?: 'NO CUSTOM',
             'options' => $route->getOptions(),
@@ -382,7 +382,7 @@ private function getCallableData(mixed $callable): array
 
         if (method_exists($callable, '__invoke')) {
             $data['type'] = 'object';
-            $data['name'] = \get_class($callable);
+            $data['name'] = $callable::class;
 
             return $data;
         }
diff --git a/Console/Descriptor/MarkdownDescriptor.php b/Console/Descriptor/MarkdownDescriptor.php
@@ -51,7 +51,7 @@ protected function describeRoute(Route $route, array $options = [])
             ."\n".'- Host Regex: '.('' !== $route->getHost() ? $route->compile()->getHostRegex() : '')
             ."\n".'- Scheme: '.($route->getSchemes() ? implode('|', $route->getSchemes()) : 'ANY')
             ."\n".'- Method: '.($route->getMethods() ? implode('|', $route->getMethods()) : 'ANY')
-            ."\n".'- Class: '.\get_class($route)
+            ."\n".'- Class: '.$route::class
             ."\n".'- Defaults: '.$this->formatRouterConfig($route->getDefaults())
             ."\n".'- Requirements: '.($route->getRequirements() ? $this->formatRouterConfig($route->getRequirements()) : 'NO CUSTOM')
             ."\n".'- Options: '.$this->formatRouterConfig($route->getOptions());
@@ -101,7 +101,7 @@ protected function describeContainerService(object $service, array $options = []
         } elseif ($service instanceof Definition) {
             $this->describeContainerDefinition($service, $childOptions, $builder);
         } else {
-            $this->write(sprintf('**`%s`:** `%s`', $options['id'], \get_class($service)));
+            $this->write(sprintf('**`%s`:** `%s`', $options['id'], $service::class));
         }
     }
 
@@ -188,7 +188,7 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o
             $this->write("\n\nServices\n--------\n");
             foreach ($services['services'] as $id => $service) {
                 $this->write("\n");
-                $this->write(sprintf('- `%s`: `%s`', $id, \get_class($service)));
+                $this->write(sprintf('- `%s`: `%s`', $id, $service::class));
             }
         }
     }
@@ -399,7 +399,7 @@ protected function describeCallable(mixed $callable, array $options = [])
 
         if (method_exists($callable, '__invoke')) {
             $string .= "\n- Type: `object`";
-            $string .= "\n".sprintf('- Name: `%s`', \get_class($callable));
+            $string .= "\n".sprintf('- Name: `%s`', $callable::class);
 
             return $this->write($string."\n");
         }
diff --git a/Console/Descriptor/TextDescriptor.php b/Console/Descriptor/TextDescriptor.php
@@ -98,7 +98,7 @@ protected function describeRoute(Route $route, array $options = [])
             ['Scheme', $route->getSchemes() ? implode('|', $route->getSchemes()) : 'ANY'],
             ['Method', $route->getMethods() ? implode('|', $route->getMethods()) : 'ANY'],
             ['Requirements', $route->getRequirements() ? $this->formatRouterConfig($route->getRequirements()) : 'NO CUSTOM'],
-            ['Class', \get_class($route)],
+            ['Class', $route::class],
             ['Defaults', $this->formatRouterConfig($defaults)],
             ['Options', $this->formatRouterConfig($route->getOptions())],
         ];
@@ -156,7 +156,7 @@ protected function describeContainerService(object $service, array $options = []
             $options['output']->table(
                 ['Service ID', 'Class'],
                 [
-                    [$options['id'] ?? '-', \get_class($service)],
+                    [$options['id'] ?? '-', $service::class],
                 ]
             );
         }
@@ -244,7 +244,7 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o
                 $alias = $definition;
                 $tableRows[] = array_merge([$styledServiceId, sprintf('alias for "%s"', $alias)], $tagsCount ? array_fill(0, $tagsCount, '') : []);
             } else {
-                $tableRows[] = array_merge([$styledServiceId, \get_class($definition)], $tagsCount ? array_fill(0, $tagsCount, '') : []);
+                $tableRows[] = array_merge([$styledServiceId, $definition::class], $tagsCount ? array_fill(0, $tagsCount, '') : []);
             }
         }
 
@@ -625,7 +625,7 @@ private function formatCallable(mixed $callable): string
         }
 
         if (method_exists($callable, '__invoke')) {
-            return sprintf('%s::__invoke()', \get_class($callable));
+            return sprintf('%s::__invoke()', $callable::class);
         }
 
         throw new \InvalidArgumentException('Callable is not describable.');
diff --git a/Console/Descriptor/XmlDescriptor.php b/Console/Descriptor/XmlDescriptor.php
@@ -164,7 +164,7 @@ private function getRouteDocument(Route $route, string $name = null): \DOMDocume
             $routeXML->setAttribute('name', $name);
         }
 
-        $routeXML->setAttribute('class', \get_class($route));
+        $routeXML->setAttribute('class', $route::class);
 
         $routeXML->appendChild($pathXML = $dom->createElement('path'));
         $pathXML->setAttribute('regex', $route->compile()->getRegex());
@@ -269,7 +269,7 @@ private function getContainerServiceDocument(object $service, string $id, Contai
         } else {
             $dom->appendChild($serviceXML = $dom->createElement('service'));
             $serviceXML->setAttribute('id', $id);
-            $serviceXML->setAttribute('class', \get_class($service));
+            $serviceXML->setAttribute('class', $service::class);
         }
 
         return $dom;
@@ -577,7 +577,7 @@ private function getCallableDocument(mixed $callable): \DOMDocument
 
         if (method_exists($callable, '__invoke')) {
             $callableXML->setAttribute('type', 'object');
-            $callableXML->setAttribute('name', \get_class($callable));
+            $callableXML->setAttribute('name', $callable::class);
 
             return $dom;
         }
diff --git a/Tests/Command/CacheClearCommand/CacheClearCommandTest.php b/Tests/Command/CacheClearCommand/CacheClearCommandTest.php
@@ -126,7 +126,7 @@ public function testCacheIsWarmedWithOldContainer()
         \Closure::bind(function (Container $class) {
             unset($class->loadedDynamicParameters['kernel.build_dir']);
             unset($class->parameters['kernel.build_dir']);
-        }, null, \get_class($container))($container);
+        }, null, $container::class)($container);
 
         $input = new ArrayInput(['cache:clear']);
         $application = new Application($kernel);
diff --git a/Tests/DependencyInjection/Compiler/ProfilerPassTest.php b/Tests/DependencyInjection/Compiler/ProfilerPassTest.php
@@ -105,12 +105,12 @@ public function testValidCollectorWithTemplateUsingAutoconfigure(TemplateAwareDa
         $profilerDefinition = $container->register('profiler', 'ProfilerClass');
 
         $container->registerForAutoconfiguration(DataCollectorInterface::class)->addTag('data_collector');
-        $container->register('mydatacollector', \get_class($dataCollector))->setAutoconfigured(true);
+        $container->register('mydatacollector', $dataCollector::class)->setAutoconfigured(true);
 
         (new ResolveInstanceofConditionalsPass())->process($container);
         (new ProfilerPass())->process($container);
 
-        $idForTemplate = \get_class($dataCollector);
+        $idForTemplate = $dataCollector::class;
         $this->assertSame(['mydatacollector' => [$idForTemplate, 'foo']], $container->getParameter('data_collector.templates'));
 
         // grab the method calls off of the "profiler" definition
diff --git a/Tests/Functional/Bundle/TestBundle/Controller/AnnotatedController.php b/Tests/Functional/Bundle/TestBundle/Controller/AnnotatedController.php
@@ -22,7 +22,7 @@ class AnnotatedController
      */
     public function requestDefaultNullAction(Request $request = null)
     {
-        return new Response($request ? \get_class($request) : null);
+        return new Response($request ? $request::class : null);
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ public function warmUp(string $cacheDir): array`
`53`	`53`	`try {`
`54`	`54`	`$this->dumpExtension($extension, $generator);`
`55`	`55`	`} catch (\Exception $e) {`
`56`		`- $this->logger?->warning('Failed to generate ConfigBuilder for extension {extensionClass}.', ['exception' => $e, 'extensionClass' => \get_class($extension)]);`
	`56`	`+ $this->logger?->warning('Failed to generate ConfigBuilder for extension {extensionClass}.', ['exception' => $e, 'extensionClass' => $extension::class]);`
`57`	`57`	`}`
`58`	`58`	`}`
`59`	`59`
Original file line number	Diff line number	Diff line change
`@@ -158,7 +158,7 @@ protected function formatValue(mixed $value): string`
`158`	`158`	`}`
`159`	`159`
`160`	`160`	`if (\is_object($value)) {`
`161`		`- return sprintf('object(%s)', \get_class($value));`
	`161`	`+ return sprintf('object(%s)', $value::class);`
`162`	`162`	`}`
`163`	`163`
`164`	`164`	`if (\is_string($value)) {`
`@@ -335,7 +335,7 @@ private function getContainerEnvVars(ContainerBuilder $container): array`
`335`	`335`	`$getDefaultParameter = function (string $name) {`
`336`	`336`	`return parent::get($name);`
`337`	`337`	`};`
`338`		`- $getDefaultParameter = $getDefaultParameter->bindTo($bag, \get_class($bag));`
	`338`	`+ $getDefaultParameter = $getDefaultParameter->bindTo($bag, $bag::class);`
`339`	`339`
`340`	`340`	`$getEnvReflection = new \ReflectionMethod($container, 'getEnv');`
`341`	`341`
Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ protected function describeContainerService(object $service, array $options = []`
`78`	`78`	`} elseif ($service instanceof Definition) {`
`79`	`79`	`$this->writeData($this->getContainerDefinitionData($service, isset($options['omit_tags']) && $options['omit_tags'], isset($options['show_arguments']) && $options['show_arguments'], $builder, $options['id']), $options);`
`80`	`80`	`} else {`
`81`		`- $this->writeData(\get_class($service), $options);`
	`81`	`+ $this->writeData($service::class, $options);`
`82`	`82`	`}`
`83`	`83`	`}`
`84`	`84`
`@@ -108,7 +108,7 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o`
`108`	`108`	`} elseif ($service instanceof Definition) {`
`109`	`109`	`$data['definitions'][$serviceId] = $this->getContainerDefinitionData($service, $omitTags, $showArguments, $builder, $serviceId);`
`110`	`110`	`} else {`
`111`		`- $data['services'][$serviceId] = \get_class($service);`
	`111`	`+ $data['services'][$serviceId] = $service::class;`
`112`	`112`	`}`
`113`	`113`	`}`
`114`	`114`
`@@ -204,7 +204,7 @@ protected function getRouteData(Route $route): array`
`204`	`204`	`'hostRegex' => '' !== $route->getHost() ? $route->compile()->getHostRegex() : '',`
`205`	`205`	`'scheme' => $route->getSchemes() ? implode('\|', $route->getSchemes()) : 'ANY',`
`206`	`206`	`'method' => $route->getMethods() ? implode('\|', $route->getMethods()) : 'ANY',`
`207`		`- 'class' => \get_class($route),`
	`207`	`+ 'class' => $route::class,`
`208`	`208`	`'defaults' => $route->getDefaults(),`
`209`	`209`	`'requirements' => $route->getRequirements() ?: 'NO CUSTOM',`
`210`	`210`	`'options' => $route->getOptions(),`
`@@ -382,7 +382,7 @@ private function getCallableData(mixed $callable): array`
`382`	`382`
`383`	`383`	`if (method_exists($callable, '__invoke')) {`
`384`	`384`	`$data['type'] = 'object';`
`385`		`- $data['name'] = \get_class($callable);`
	`385`	`+ $data['name'] = $callable::class;`
`386`	`386`
`387`	`387`	`return $data;`
`388`	`388`	`}`
Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ protected function describeRoute(Route $route, array $options = [])`
`51`	`51`	`."\n".'- Host Regex: '.('' !== $route->getHost() ? $route->compile()->getHostRegex() : '')`
`52`	`52`	`."\n".'- Scheme: '.($route->getSchemes() ? implode('\|', $route->getSchemes()) : 'ANY')`
`53`	`53`	`."\n".'- Method: '.($route->getMethods() ? implode('\|', $route->getMethods()) : 'ANY')`
`54`		`- ."\n".'- Class: '.\get_class($route)`
	`54`	`+ ."\n".'- Class: '.$route::class`
`55`	`55`	`."\n".'- Defaults: '.$this->formatRouterConfig($route->getDefaults())`
`56`	`56`	`."\n".'- Requirements: '.($route->getRequirements() ? $this->formatRouterConfig($route->getRequirements()) : 'NO CUSTOM')`
`57`	`57`	`."\n".'- Options: '.$this->formatRouterConfig($route->getOptions());`
`@@ -101,7 +101,7 @@ protected function describeContainerService(object $service, array $options = []`
`101`	`101`	`} elseif ($service instanceof Definition) {`
`102`	`102`	`$this->describeContainerDefinition($service, $childOptions, $builder);`
`103`	`103`	`} else {`
`104`		- $this->write(sprintf('`%s`: `%s`', $options['id'], \get_class($service)));
	`104`	+ $this->write(sprintf('`%s`: `%s`', $options['id'], $service::class));
`105`	`105`	`}`
`106`	`106`	`}`
`107`	`107`
`@@ -188,7 +188,7 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o`
`188`	`188`	`$this->write("\n\nServices\n--------\n");`
`189`	`189`	`foreach ($services['services'] as $id => $service) {`
`190`	`190`	`$this->write("\n");`
`191`		- $this->write(sprintf('- `%s`: `%s`', $id, \get_class($service)));
	`191`	+ $this->write(sprintf('- `%s`: `%s`', $id, $service::class));
`192`	`192`	`}`
`193`	`193`	`}`
`194`	`194`	`}`
`@@ -399,7 +399,7 @@ protected function describeCallable(mixed $callable, array $options = [])`
`399`	`399`
`400`	`400`	`if (method_exists($callable, '__invoke')) {`
`401`	`401`	$string .= "\n- Type: `object`";
`402`		- $string .= "\n".sprintf('- Name: `%s`', \get_class($callable));
	`402`	+ $string .= "\n".sprintf('- Name: `%s`', $callable::class);
`403`	`403`
`404`	`404`	`return $this->write($string."\n");`
`405`	`405`	`}`
Original file line number	Diff line number	Diff line change
`@@ -98,7 +98,7 @@ protected function describeRoute(Route $route, array $options = [])`
`98`	`98`	`['Scheme', $route->getSchemes() ? implode('\|', $route->getSchemes()) : 'ANY'],`
`99`	`99`	`['Method', $route->getMethods() ? implode('\|', $route->getMethods()) : 'ANY'],`
`100`	`100`	`['Requirements', $route->getRequirements() ? $this->formatRouterConfig($route->getRequirements()) : 'NO CUSTOM'],`
`101`		`- ['Class', \get_class($route)],`
	`101`	`+ ['Class', $route::class],`
`102`	`102`	`['Defaults', $this->formatRouterConfig($defaults)],`
`103`	`103`	`['Options', $this->formatRouterConfig($route->getOptions())],`
`104`	`104`	`];`
`@@ -156,7 +156,7 @@ protected function describeContainerService(object $service, array $options = []`
`156`	`156`	`$options['output']->table(`
`157`	`157`	`['Service ID', 'Class'],`
`158`	`158`	`[`
`159`		`- [$options['id'] ?? '-', \get_class($service)],`
	`159`	`+ [$options['id'] ?? '-', $service::class],`
`160`	`160`	`]`
`161`	`161`	`);`
`162`	`162`	`}`
`@@ -244,7 +244,7 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o`
`244`	`244`	`$alias = $definition;`
`245`	`245`	`$tableRows[] = array_merge([$styledServiceId, sprintf('alias for "%s"', $alias)], $tagsCount ? array_fill(0, $tagsCount, '') : []);`
`246`	`246`	`} else {`
`247`		`- $tableRows[] = array_merge([$styledServiceId, \get_class($definition)], $tagsCount ? array_fill(0, $tagsCount, '') : []);`
	`247`	`+ $tableRows[] = array_merge([$styledServiceId, $definition::class], $tagsCount ? array_fill(0, $tagsCount, '') : []);`
`248`	`248`	`}`
`249`	`249`	`}`
`250`	`250`
`@@ -625,7 +625,7 @@ private function formatCallable(mixed $callable): string`
`625`	`625`	`}`
`626`	`626`
`627`	`627`	`if (method_exists($callable, '__invoke')) {`
`628`		`- return sprintf('%s::__invoke()', \get_class($callable));`
	`628`	`+ return sprintf('%s::__invoke()', $callable::class);`
`629`	`629`	`}`
`630`	`630`
`631`	`631`	`throw new \InvalidArgumentException('Callable is not describable.');`