Report for symfony/symfony#60874@8df6691
[FrameworkBundle] Allow using their name without added suffix when using `#[Target]` for custom services
by @ValmonzoSome issues have been detected in this pull request
Issues that can be fixed by applying a patch
Review the proposed patch then download it to apply it manually or execute the following command from the repository root directory:
curl https://fabbot.io/patch/symfony/symfony/60874/8df669125bf4f44c337409e0ea7da73eec53c7e1/cs.diff | patch -p0diff -ru src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php
--- src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php 2025-06-24 14:20:51.891352912 +0000
+++ src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php 2025-06-24 14:21:17.817398866 +0000
@@ -1164,7 +1164,7 @@
$workflow['definition_validators'][] = match ($workflow['type']) {
'state_machine' => Workflow\Validator\StateMachineValidator::class,
'workflow' => Workflow\Validator\WorkflowValidator::class,
- default => throw new \LogicException(\sprintf('Invalid workflow type "%s".', $workflow['type'])),
+ default => throw new \LogicException(\sprintf('Invalid workflow type "%s".', $workflow['type'])),
};
// Create Workflow
@@ -3324,13 +3324,13 @@
throw new LogicException(\sprintf('Compound rate limiter "%s" requires at least one sub-limiter.', $name));
}
- if (\array_diff($limiterConfig['limiters'], $limiters)) {
+ if (array_diff($limiterConfig['limiters'], $limiters)) {
throw new LogicException(\sprintf('Compound rate limiter "%s" requires at least one sub-limiter to be configured.', $name));
}
$container->register($limiterId = 'limiter.'.$name, CompoundRateLimiterFactory::class)
->addTag('rate_limiter', ['name' => $name])
- ->addArgument(new IteratorArgument(\array_map(
+ ->addArgument(new IteratorArgument(array_map(
static fn (string $name) => new Reference('limiter.'.$name),
$limiterConfig['limiters']
)))
