From bef7f446721324ad6529068bcd29013dd42fe099 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gr=C3=A9goire=20Pineau?= Date: Fri, 3 Apr 2026 10:57:23 +0200 Subject: [PATCH] [Validator] Fix PHPDoc of generate file `/**` is replaced by `/*` by php-cs-fixer (or something else) because the var is not bellow this line. So this line become useless. By moving to a `@param` we fix this issue. --- For reference: ``` >/tmp symfony new --webapp maker-bundle INFO A new Symfony CLI version is available (5.16.1, currently running 5.8.14). If you installed the Symfony CLI via a package manager, updates are going to be automatic. If not, upgrade by downloading the new version at https://github.com/symfony-cli/symfony-cli/releases And replace the current binary (symfony) by the new one. * Creating a new Symfony project with Composer (running /home/gregoire/.local/bin/composer create-project symfony/skeleton /tmp/maker-bundle --no-interaction) * Setting up the project under Git version control (running git init /tmp/maker-bundle) (running /home/gregoire/.local/bin/composer require webapp --no-interaction) [OK] Your project is now ready in /tmp/maker-bundle >/tmp cd maker-bundle /tmp/maker-bundle >/tmp/maker-bundle(main) bin/console make:validator Foobar created: src/Validator/FoobarValidator.php created: src/Validator/Foobar.php Success! Next: Open your new constraint & validators and add your logic. Find the documentation at http://symfony.com/doc/current/validation/custom_constraint.html >/tmp/maker-bundle(main %) rg constraint src/Validator/ src/Validator/FoobarValidator.php 5:use Symfony\Component\Validator\Constraint; 6:use Symfony\Component\Validator\ConstraintValidator; 8:final class FoobarValidator extends ConstraintValidator 10: public function validate(mixed $value, Constraint $constraint): void 12: /* @var Foobar $constraint */ 19: $this->context->buildViolation($constraint->message) ``` --- templates/validator/Validator.tpl.php | 5 +++-- tests/fixtures/make-validator/expected/FooBarValidator.php | 5 +++-- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/templates/validator/Validator.tpl.php b/templates/validator/Validator.tpl.php index b8f33b6db..74d676290 100644 --- a/templates/validator/Validator.tpl.php +++ b/templates/validator/Validator.tpl.php @@ -7,10 +7,11 @@ getClassDeclaration(); ?> { + /** + * @param $constraint + */ public function validate(mixed $value, Constraint $constraint): void { - /** @var $constraint */ - if (null === $value || '' === $value) { return; } diff --git a/tests/fixtures/make-validator/expected/FooBarValidator.php b/tests/fixtures/make-validator/expected/FooBarValidator.php index 99c7f7bee..aa9b9c2b2 100644 --- a/tests/fixtures/make-validator/expected/FooBarValidator.php +++ b/tests/fixtures/make-validator/expected/FooBarValidator.php @@ -7,10 +7,11 @@ final class FooBarValidator extends ConstraintValidator { + /** + * @param FooBar $constraint + */ public function validate(mixed $value, Constraint $constraint): void { - /* @var FooBar $constraint */ - if (null === $value || '' === $value) { return; }