[Validator] Use LogicException for missing Property Access Component in comparison constraints

Lctrs · Lctrs · commit 6ac23169931e · 2019-06-19T10:51:43.000+02:00
diff --git a/src/Symfony/Component/Validator/Constraints/AbstractComparison.php b/src/Symfony/Component/Validator/Constraints/AbstractComparison.php
@@ -14,6 +14,7 @@
 use Symfony\Component\PropertyAccess\PropertyAccess;
 use Symfony\Component\Validator\Constraint;
 use Symfony\Component\Validator\Exception\ConstraintDefinitionException;
+use Symfony\Component\Validator\Exception\LogicException;
 
 /**
  * Used for the comparison of values.
@@ -46,7 +47,7 @@ public function __construct($options = null)
             }
 
             if (isset($options['propertyPath']) && !class_exists(PropertyAccess::class)) {
-                throw new ConstraintDefinitionException(sprintf('The "%s" constraint requires the Symfony PropertyAccess component to use the "propertyPath" option.', \get_class($this)));
+                throw new LogicException(sprintf('The "%s" constraint requires the Symfony PropertyAccess component to use the "propertyPath" option.', \get_class($this)));
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -14,6 +14,7 @@`
`14`	`14`	`use Symfony\Component\PropertyAccess\PropertyAccess;`
`15`	`15`	`use Symfony\Component\Validator\Constraint;`
`16`	`16`	`use Symfony\Component\Validator\Exception\ConstraintDefinitionException;`
	`17`	`+use Symfony\Component\Validator\Exception\LogicException;`
`17`	`18`
`18`	`19`	`/**`
`19`	`20`	`* Used for the comparison of values.`
`@@ -46,7 +47,7 @@ public function __construct($options = null)`
`46`	`47`	`}`
`47`	`48`
`48`	`49`	`if (isset($options['propertyPath']) && !class_exists(PropertyAccess::class)) {`
`49`		`- throw new ConstraintDefinitionException(sprintf('The "%s" constraint requires the Symfony PropertyAccess component to use the "propertyPath" option.', \get_class($this)));`
	`50`	`+ throw new LogicException(sprintf('The "%s" constraint requires the Symfony PropertyAccess component to use the "propertyPath" option.', \get_class($this)));`
`50`	`51`	`}`
`51`	`52`	`}`
`52`	`53`