Skip to content

Commit f1e03d0

Browse files
committed
minor #47401 Replace get_class() calls by ::class (deguif)
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
2 parents b67d0f6 + 08ea159 commit f1e03d0

9 files changed

+11
-11
lines changed

Compiler/CheckTypeDeclarationsPass.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,7 @@ private function checkType(Definition $checkedDefinition, mixed $value, \Reflect
258258
} elseif ($value instanceof ServiceLocatorArgument) {
259259
$class = ServiceLocator::class;
260260
} elseif (\is_object($value)) {
261-
$class = \get_class($value);
261+
$class = $value::class;
262262
} else {
263263
$class = \gettype($value);
264264
$class = ['integer' => 'int', 'double' => 'float', 'boolean' => 'bool'][$class] ?? $class;

Compiler/Compiler.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,10 @@ public function addPass(CompilerPassInterface $pass, string $type = PassConfig::
5252
public function log(CompilerPassInterface $pass, string $message)
5353
{
5454
if (str_contains($message, "\n")) {
55-
$message = str_replace("\n", "\n".\get_class($pass).': ', trim($message));
55+
$message = str_replace("\n", "\n".$pass::class.': ', trim($message));
5656
}
5757

58-
$this->log[] = \get_class($pass).': '.$message;
58+
$this->log[] = $pass::class.': '.$message;
5959
}
6060

6161
public function getLog(): array

Compiler/MergeExtensionConfigurationPass.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ public function __construct(ExtensionInterface $extension, ParameterBagInterface
151151
{
152152
parent::__construct($parameterBag);
153153

154-
$this->extensionClass = \get_class($extension);
154+
$this->extensionClass = $extension::class;
155155
}
156156

157157
public function addCompilerPass(CompilerPassInterface $pass, string $type = PassConfig::TYPE_BEFORE_OPTIMIZATION, int $priority = 0): static

Compiler/ResolveInstanceofConditionalsPass.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ private function processDefinition(ContainerBuilder $container, string $id, Defi
107107
$definition->setBindings([]);
108108
$definition = serialize($definition);
109109

110-
if (Definition::class === \get_class($abstract)) {
110+
if (Definition::class === $abstract::class) {
111111
// cast Definition to ChildDefinition
112112
$definition = substr_replace($definition, '53', 2, 2);
113113
$definition = substr_replace($definition, 'Child', 44, 0);

Dumper/XmlDumper.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -381,7 +381,7 @@ public static function phpToXml(mixed $value): string
381381
case $value instanceof Parameter:
382382
return '%'.$value.'%';
383383
case $value instanceof \UnitEnum:
384-
return sprintf('%s::%s', \get_class($value), $value->name);
384+
return sprintf('%s::%s', $value::class, $value->name);
385385
case \is_object($value) || \is_resource($value):
386386
throw new RuntimeException('Unable to dump a service container if a parameter is an object or a resource.');
387387
default:

Dumper/YamlDumper.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -297,7 +297,7 @@ private function dumpValue(mixed $value): mixed
297297
} elseif ($value instanceof Definition) {
298298
return new TaggedValue('service', (new Parser())->parse("_:\n".$this->addService('_', $value), Yaml::PARSE_CUSTOM_TAGS)['_']['_']);
299299
} elseif ($value instanceof \UnitEnum) {
300-
return new TaggedValue('php/const', sprintf('%s::%s', \get_class($value), $value->name));
300+
return new TaggedValue('php/const', sprintf('%s::%s', $value::class, $value->name));
301301
} elseif ($value instanceof AbstractArgument) {
302302
return new TaggedValue('abstract', $value->getText());
303303
} elseif (\is_object($value) || \is_resource($value)) {

Tests/Compiler/RegisterServiceSubscribersPassTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -382,7 +382,7 @@ public static function getSubscribedServices(): array
382382
$container->setAlias('stdClass $someService', 'some.service');
383383
$container->setAlias('stdClass $some_service', 'some.service');
384384
$container->setAlias('stdClass $anotherService', 'some.service');
385-
$container->register('foo', \get_class($subscriber))
385+
$container->register('foo', $subscriber::class)
386386
->addMethodCall('setContainer', [new Reference(PsrContainerInterface::class)])
387387
->addTag('container.service_subscriber');
388388

@@ -433,7 +433,7 @@ public static function getSubscribedServices(): array
433433
};
434434

435435
$container->setParameter('parameter.1', 'foobar');
436-
$container->register('foo', \get_class($subscriber))
436+
$container->register('foo', $subscriber::class)
437437
->addMethodCall('setContainer', [new Reference(PsrContainerInterface::class)])
438438
->addTag('container.service_subscriber');
439439

Tests/Compiler/ResolveChildDefinitionsPassTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -293,7 +293,7 @@ public function testDeepDefinitionsResolving()
293293
$this->assertSame('parentClass', $factory[0]->getClass());
294294

295295
$argument = $container->getDefinition('sibling')->getArgument(0);
296-
$this->assertSame('Symfony\Component\DependencyInjection\Definition', \get_class($argument));
296+
$this->assertSame('Symfony\Component\DependencyInjection\Definition', $argument::class);
297297
$this->assertSame('parentClass', $argument->getClass());
298298

299299
$properties = $container->getDefinition('sibling')->getProperties();

Tests/ContainerBuilderTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -381,7 +381,7 @@ public function testCreateProxyWithRealServiceInstantiator()
381381
$foo1 = $builder->get('foo1');
382382

383383
$this->assertSame($foo1, $builder->get('foo1'), 'The same proxy is retrieved on multiple subsequent calls');
384-
$this->assertSame('Bar\FooClass', \get_class($foo1));
384+
$this->assertSame('Bar\FooClass', $foo1::class);
385385
}
386386

387387
public function testCreateLazyProxy()

0 commit comments

Comments
 (0)