[DoctrineBridge] fix calling get_class on non-object

kbond · kbond · commit a7f90b43f52b · 2021-11-30T13:35:08.000-05:00
diff --git a/src/Symfony/Bridge/Doctrine/Tests/Validator/Constraints/UniqueEntityValidatorTest.php b/src/Symfony/Bridge/Doctrine/Tests/Validator/Constraints/UniqueEntityValidatorTest.php
@@ -821,6 +821,21 @@ public function testValidateUniquenessWithEmptyIterator($entity, $result)
         $this->assertNoViolation();
     }
 
+    public function testValidateNonEntity()
+    {
+        $constraint = new UniqueEntity([
+            'message' => 'myMessage',
+            'fields' => ['name'],
+            'em' => self::EM_NAME,
+        ]);
+
+        $this->validator->validate(null, $constraint);
+
+        $this->assertNoViolation();
+
+        $this->validator->validate('foo', $constraint);
+    }
+
     public function resultWithEmptyIterator(): array
     {
         $entity = new SingleIntIdEntity(1, 'foo');
diff --git a/src/Symfony/Bridge/Doctrine/Validator/Constraints/UniqueEntityValidator.php b/src/Symfony/Bridge/Doctrine/Validator/Constraints/UniqueEntityValidator.php
@@ -57,7 +57,7 @@ public function validate($entity, Constraint $constraint)
             throw new ConstraintDefinitionException('At least one field has to be specified.');
         }
 
-        if (null === $entity) {
+        if (!\is_object($entity)) {
             return;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ public function validate($entity, Constraint $constraint)`
`57`	`57`	`throw new ConstraintDefinitionException('At least one field has to be specified.');`
`58`	`58`	`}`
`59`	`59`
`60`		`- if (null === $entity) {`
	`60`	`+ if (!\is_object($entity)) {`
`61`	`61`	`return;`
`62`	`62`	`}`
`63`	`63`