Skip to content

Commit d6d8585

Browse files
DonCallistofabpot
authored andcommitted
Update ValidationListener.php
This comment was misleading. It's true that no groups (ie.: `Default`) are setted into validation process but it's not clear from the comment. I suppose that is better to be more explicit about this process. BTW I'm still not sure if `FormValidator` reference is necessary as we can change it and we should remember to update this comment and this does not seems to fit. WDYT, is better to leave `FormValidator` out of this comment? For reference: https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Form/Extension/Validator/Constraints/FormValidator.php#L46-L92
1 parent 26d4069 commit d6d8585

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Extension/Validator/EventListener/ValidationListener.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ public function validateForm(FormEvent $event)
5151
$form = $event->getForm();
5252

5353
if ($form->isRoot()) {
54-
// Validate the form in group "Default"
54+
// Form groups are validated internally (FormValidator). Here we don't set groups as they are retrieved into the validator.
5555
foreach ($this->validator->validate($form) as $violation) {
5656
// Allow the "invalid" constraint to be put onto
5757
// non-synchronized forms

0 commit comments

Comments
 (0)