update entity exists validator

stfalcon-studio · May 1, 2024 · 934e1e6 · 934e1e6
1 parent 2bfb06b
commit 934e1e6
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 15 deletions.
diff --git a/Service/Repository/FindableByIdInterface.php b/Service/Repository/FindableByIdInterface.php
@@ -14,6 +14,8 @@
 
 /**
  * FindableByIdInterface.
+ *
+ * @deprecated
  */
 interface FindableByIdInterface
 {

diff --git a/Service/Repository/GettableOneByIdInterface.php b/Service/Repository/GettableOneByIdInterface.php
@@ -14,6 +14,8 @@
 
 /**
  * Gettable One By Id Interface.
+ *
+ * @deprecated
  */
 interface GettableOneByIdInterface
 {

diff --git a/Service/Repository/RepositoryService.php b/Service/Repository/RepositoryService.php
@@ -17,6 +17,8 @@
 
 /**
  * Repository Service.
+ *
+ * @deprecated
  */
 class RepositoryService
 {

diff --git a/Validator/Constraints/Entity/EntityExists.php b/Validator/Constraints/Entity/EntityExists.php
@@ -31,15 +31,18 @@ class EntityExists extends Constraint
 
     public string $message = 'entity_does_not_exist';
 
+    /** @var class-string<object> */
     public string $class;
+    public string $property;
 
     /**
-     * @param string     $class
-     * @param mixed      $options
-     * @param array|null $groups
-     * @param mixed      $payload
+     * @param class-string<object> $class
+     * @param mixed                $options
+     * @param array|null           $groups
+     * @param mixed                $payload
+     * @param string               $property
      */
-    public function __construct(string $class, mixed $options = null, array $groups = null, mixed $payload = null)
+    public function __construct(string $class, mixed $options = null, array $groups = null, mixed $payload = null, string $property = 'id')
     {
         parent::__construct($options, $groups, $payload);
 
@@ -52,5 +55,6 @@ public function __construct(string $class, mixed $options = null, array $groups
         }
 
         $this->class = $class;
+        $this->property = $property;
     }
 }
diff --git a/Validator/Constraints/Entity/EntityExistsValidator.php b/Validator/Constraints/Entity/EntityExistsValidator.php
@@ -13,7 +13,7 @@
 namespace StfalconStudio\ApiBundle\Validator\Constraints\Entity;
 
 use StfalconStudio\ApiBundle\Exception\Validator\UnexpectedConstraintException;
-use StfalconStudio\ApiBundle\Service\Repository\RepositoryService;
+use StfalconStudio\ApiBundle\Traits\EntityManagerTrait;
 use Symfony\Component\Validator\Constraint;
 use Symfony\Component\Validator\ConstraintValidator;
 
@@ -22,12 +22,7 @@
  */
 class EntityExistsValidator extends ConstraintValidator
 {
-    /**
-     * @param RepositoryService $repositoryService
-     */
-    public function __construct(private readonly RepositoryService $repositoryService)
-    {
-    }
+    use EntityManagerTrait;
 
     /**
      * @param mixed                   $value
@@ -41,15 +36,19 @@ public function validate(mixed $value, Constraint|EntityExists $constraint): voi
             throw new UnexpectedConstraintException($constraint, EntityExists::class);
         }
 
-        if (!\is_string($value)) {
+        if (!(\is_int($value) || \is_string($value))) {
             return;
         }
 
-        if (!$this->repositoryService->findEntityById($value, $constraint->class) instanceof $constraint->class) {
+        $repository = $this->em->getRepository($constraint->class);
+
+        if (!$repository->findOneBy([$constraint->property => $value]) instanceof $constraint->class) {
             $this->context
                 ->buildViolation($constraint->message)
                 ->setCode(EntityExists::ENTITY_DOES_NOT_EXIST)
-                ->setParameter('%id%', $value)
+                ->setParameter('%property%', $constraint->property)
+                ->setParameter('%value%', (string) $value)
+                ->setParameter('%class%', $constraint->class)
                 ->addViolation()
             ;
         }