8000 do not validate empty values · symfony/symfony@14d9d10 · GitHub
[go: up one dir, main page]
Skip to content

Commit 14d9d10

Browse files
xabbuhxabbuh
committed
do not validate empty values
1 parent 7acc345 commit 14d9d10

File tree

3 files changed

+13
-1
lines changed

3 files changed

+13
-1
lines changed

‎src/Symfony/Bridge/Doctrine/Validator/Constraints/UniqueEntityValidator.php‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,14 @@ public function validate($entity, Constraint $constraint)
6262
throw new ConstraintDefinitionException('At least one field has to be specified.');
6363
}
6464

65+
if (null === $entity) {
66+
return;
67+
}
68+
69+
if (!is_object($entity)) {
70+
throw new UnexpectedTypeException($entity, 'object');
71+
}
72+
6573
if ($constraint->em) {
6674
$em = $this->registry->getManager($constraint->em);
6775

‎src/Symfony/Component/Security/Core/Validator/Constraints/UserPasswordValidator.php‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,10 @@ public function validate($password, Constraint $constraint)
3939
throw new UnexpectedTypeException($constraint, __NAMESPACE__.'\UserPassword');
4040
}
4141

42+
if (null === $password || '' === $password) {
43+
return;
44+
}
45+
4246
$user = $this->tokenStorage->getToken()->getUser();
4347

4448
if (!$user instanceof UserInterface) {

‎src/Symfony/Component/Validator/Constraints/UrlValidator.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ public function validate($value, Constraint $constraint)
4848
throw new UnexpectedTypeException($constraint, __NAMESPACE__.'\Url');
4949
}
5050

51-
if (null === $value) {
51+
if (null === $value || '' === $value) {
5252
return;
5353
}
5454

0 commit comments

Comments
 (0)
0