Почему не отрабатывает new Assert\Optional при валидации полей формы?

Question

[RESURT]

Всем привет!
Почему может не отрабатывать опциональная проверка, когда например поле пустое, иначе получаю ошибку "This value should have exactly 10 characters"
То есть проверка на минимальное количество символов все равно проходит

$validator = Validation::createValidator();

            $constraint = new Assert\Collection([
                'financialpersoninn' => new Assert\Optional([
                    new Assert\Length(['min' => 10, 'max' => 10]),
                    new Assert\Regex('/^[0-9]*$/u')
                ]),
                'CustomsModeCode' => [
                    new Assert\Length(['max' => 2]),
                    new Assert\Regex('/^[0-9]*$/u')
                ],
                'transportidentifierTime' => new Assert\Type(['type' => 'string']),

            ]);

            $violations = $validator->validate($data, $constraint);

Comments

[tukreb]

вам нужно добавить Assert что поле не пустое.
https://symfony.com/doc/current/reference/constrai...

И regex вам тоже не нужно, если проверяете на числа, то есть Assert на числа, https://symfony.com/doc/current/reference/constrai...

[RESURT]

tukreb, 1) Зачем? Вопрос, как раз в том, что поле может быть пустым, необходимо наоборот, если поле не пустое, то валидировать его по указанным правилам.
2) Поле приходит как строка, если я пропишу тип число, будет ошибка

Answer 1

[Дмитрий Гординский]

Assert\Optional говорит лишь о том что поле может отсутствовать, но если оно присутствует, оно должно соответствовать описанным правилам.

То есть правило из вопроса подразумевает что валидируемые данные могут вовсе не содержать ключа financialpersoninn, и при этом будут считаться валидными. Если же такой ключ присутствует, его значение должно удовлетворять заданным правилам.

Как вариант можно использовать композитные правила AtLeastOneOf.

Comments

[RESURT]

Это ограничение проверяет, чтобы значение удовлетворяло хотя бы одно из заданных ограничений. Валидация прекращается, как только одно ограничение удовлетворено.

Необходимо, чтобы валидация не происходила, если ничего по ключу не передано, то есть пустота.
И выполняться должно оба условия, если данные передали.
Думаю, это не подойдет в данном случае.

[Дмитрий Гординский]

RESURT, объединить Regex и Length ограничения в одно правило можно при помощи другого композитного правила - All

[RESURT]

Дмитрий Гординский, они так и будут срабатывать, если поле пустое, это не поможет

[Дмитрий Гординский]

RESURT, поможет.
Вы хотите чтобы поле соблюдало хотя бы одно из двух правил валидации:
- строка пустая
- строка необходимой длины И соответствует регулярному выражению.

AtLeastOneOf позволяет реализовать логику одно из .
All позволяет объединить ограничения по длине И регулярной в одно правило.