Merge pull request #1019 from spiral/bugfix/uuid-caster

[spiral/filters] Fix method supports in UuidCaster
spiral · Dec 1, 2023 · ac995f5 · ac995f5
2 parents cc5594a + 05188b6
commit ac995f5
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/Filters/src/Model/Mapper/UuidCaster.php b/src/Filters/src/Model/Mapper/UuidCaster.php
@@ -19,7 +19,9 @@ public function supports(\ReflectionNamedType $type): bool
             $this->interfaceExists = \interface_exists(UuidInterface::class);
         }
 
-        return $this->interfaceExists && $this->implements($type->getName(), UuidInterface::class);
+        return $this->interfaceExists &&
+            !$type->isBuiltin() &&
+            $this->implements($type->getName(), UuidInterface::class);
     }
 
     public function setValue(FilterInterface $filter, \ReflectionProperty $property, mixed $value): void
@@ -40,6 +42,10 @@ private function implements(string $haystack, string $interface): bool
             return true;
         }
 
+        if (!\class_exists($haystack)) {
+            return false;
+        }
+
         foreach ((array)\class_implements($haystack) as $implements) {
             if ($implements === $interface) {
                 return true;

diff --git a/src/Filters/tests/Mapper/UuidCasterTest.php b/src/Filters/tests/Mapper/UuidCasterTest.php
@@ -43,6 +43,7 @@ public static function supportsDataProvider(): \Traversable
     {
         $ref = new \ReflectionClass(UserFilter::class);
 
+        yield 'string' => [$ref->getProperty('name'), false];
         yield 'enum' => [$ref->getProperty('status'), false];
         yield 'uuid' => [$ref->getProperty('groupUuid'), true];
     }