@xabbuh awesome! I confirmed it works as would validation groups work for every other constraint, i.e. invalidation.xml you add groups toValid and now pass thevalidation_groups option when creating the form.Valid will only be applied if any of the groups invalidation_groups equals those specified invalidation.xml.

However, I do think there is one super useful use case not covered. Consider aProfile object havingprofilePicture andcoverPhoto properties - both of typeImage, which internally has afile property of typeFile. Let's say we have different maximum file sizes for both of them, so we add 2File constraints to theImage object. And both properties should be valid at all times forProfile.

<classname="Profile">  <propertyname="profilePicture">    <constraintname="Valid" />  </property>  <propertyname="coverPhoto">    <constraintname="Valid" />  </property></class><classname="Image">  <propertyname="file">    <constraintname="File">      <optionname="maxSize">100k</option>      <optionname="groups">        <value>profile-picture</value>      </option>    </constraint>    <constraintname="File">      <optionname="maxSize">1000k</option>      <optionname="groups">        <value>cover-photo</value>      </option>    </constraint>  </property></class>

InProfileType we'd have something like this probably.

$builder->add('profilePicture', ImageType::class, ['validation_groups' => ['profile-picture']]);$builder->add('coverPhoto', ImageType::class, ['validation_groups' => ['cover-photo']);

But try as you might, it seems to me that validation for theseFile objects does not kick in.

$form =$factory->create(ProfileType::class);$form->handleRequest($request);$form->isValid();// returns true for files larger than what was specified

Of course this is a bit nonsensical example, but I tried to simplify as much as possible. In one of my projects we for example haveDocument objects which wrap all files and in one of the entities one property can contain only XML files and the other one can accept a different whitelist of types.

@kgilden This seems logical to me as the root form itself is validated in theDefault group. Configurationvalidation_groups for the root form to be, for example,['Default', 'profile-picture', 'cover-photo'] should do the trick.

@xabbuh ahh yes, sorry for the confusion. I should note that this patch isstill required to make my example above work. The same groups must also be added toValid constraints and then it works as expected.

@kgilden You are right. But IMO this behaviour is straight-forward with how validation groups in Symfony work in general. Do you disagree with that?

@xabbuh it's straight-forward if for example theFile constraint is applied to two distinct properties on the root object, i.e.

<classname="Profile">  <propertyname="profilePicture">    <constraintname="File"><!-- distinct options-->    </constraint>  </property>  <propertyname="coverPhoto">    <constraintname="File"><!-- distinct options-->    </constraint>  </property></class>

What I think is currently not possible is when you want to validate the same objects deeper. After this PR it's possible to do as follows.

<classname="Profile">  <propertyname="profilePicture">    <constraintname="Valid">      <optionname="groups">        <value>profile-picture</value>      </option>    </constraint>  </property>  <propertyname="coverPhoto">    <constraintname="Valid">      <optionname="groups">        <value>cover-photo</value>      </option>    </constraint>  </property></class><classname="Image">  <propertyname="file">    <constraintname="File">      <optionname="maxSize">100k</option>      <optionname="groups">        <value>profile-picture</value>      </option>    </constraint>    <constraintname="File">      <optionname="maxSize">1000k</option>      <optionname="groups">        <value>cover-photo</value>      </option>    </constraint>  </property></class>

class ProfileTypeextends AbstractType{publicfunctionbuildForm(FormBuilderInterface$builder,array$options)    {$builder->add('profilePicture', ImageType::class);$builder->add('coverPhoto', ImageType::class);    }publicfunctionconfigureOptions(OptionsResolver$resolver)    {$resolver->setDefaults(['validation_groups' => ['Default','cover-photo','profile-picture']]);    }}

Correct me if I'm wrong, but I don't think there was a way to do it prior to this PR.

Indeed, that's not possible right now.

+1 for this feature. The use case described by kgilden happens regularly for me (deep cascade validation groups)

@xabbuh Can you add a note in the CHANGELOG before I can merge?

@fabpot done, but are we sure that this is the right way to implement it?

ping @symfony/deciders

Thank you@xabbuh.

Hi,
What if value is null? i need@Assert\Valid only after and when@Assert\NotNull is valid, currently i get 500 error because of that empty value

@Atiragram : This is already supposed to be fixed in#25297, available since 3.4.1. If you still encounter an issue, please open a new issue with a reproducer or a failing test case. Thanks!