refactoring: fix multiple entity transformer validation (#202)

* refactoring: fix multiple entity transformer validation * f
troytft · Feb 15, 2022 · 0d9dc7f · 0d9dc7f
1 parent 9c8f017
commit 0d9dc7f
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/Services/Mapper/Transformer/DoctrineEntityTransformer.php b/src/Services/Mapper/Transformer/DoctrineEntityTransformer.php
@@ -53,10 +53,12 @@ private function transformSingleItem(string $class, string $fieldName, mixed $va
 
     private function transformMultipleItems(string $class, string $fieldName, mixed $value): array
     {
+        $firstCollectionItem = $value[0] ?? null;
         $columnType = RestApiBundle\Helper\DoctrineHelper::extractColumnType($class, $fieldName);
-        if ($columnType === PropertyInfo\Type::BUILTIN_TYPE_INT && !is_int($value[0] ?? null)) {
+
+        if ($columnType === PropertyInfo\Type::BUILTIN_TYPE_INT && !is_numeric($firstCollectionItem)) {
             throw new RestApiBundle\Exception\Mapper\Transformer\CollectionOfIntegersRequiredException();
-        } elseif ($columnType === PropertyInfo\Type::BUILTIN_TYPE_STRING && !is_string($value[0] ?? null)) {
+        } elseif ($columnType === PropertyInfo\Type::BUILTIN_TYPE_STRING && (!is_string($firstCollectionItem) && !is_numeric($firstCollectionItem))) {
             throw new RestApiBundle\Exception\Mapper\Transformer\CollectionOfStringsRequiredException();
         }