Throw exception if the value could not converted to database representation

Author: aaa2000

src/Doctrine/Orm/Filter/AbstractUuidFilter.php

@@ -122,30 +122,19 @@ private function convertValuesToTheDatabaseRepresentation(QueryBuilder $queryBui
         $doctrineType = Type::getType($doctrineFieldType);
         $platform = $queryBuilder->getEntityManager()->getConnection()->getDatabasePlatform();
 
-        if (\is_array($value)) {
-            $databaseValues = [];
-            foreach ($value as $val) {
-                try {
-                    $databaseValues[] = $doctrineType->convertToDatabaseValue($val, $platform);
-                } catch (ConversionException $e) {
-                    $this->logger->notice('Invalid value conversion to database representation', [
-                        'exception' => $e,
-                    ]);
-                }
+        $convertValue = static function (mixed $value) use ($doctrineType, $platform) {
+            try {
+                return $doctrineType->convertToDatabaseValue($value, $platform);
+            } catch (ConversionException $e) {
+                throw new InvalidArgumentException(\sprintf('The value "%s" could not be converted to database representation.', $value), previous: $e);
             }
+        };
 
-            return $databaseValues;
+        if (\is_array($value)) {
+            return array_map($convertValue, $value);
         }
 
-        try {
-            return $doctrineType->convertToDatabaseValue($value, $platform);
-        } catch (ConversionException $e) {
-            $this->logger->notice('Invalid value conversion to database representation', [
-                'exception' => $e,
-            ]);
-
-            return null;
-        }
+        return $convertValue($value);
     }
 
     /**

tests/Functional/Uuid/UuidFilterBaseTestCase.php

@@ -16,6 +16,7 @@
 use ApiPlatform\Symfony\Bundle\Test\ApiTestCase;
 use ApiPlatform\Tests\RecreateSchemaTrait;
 use ApiPlatform\Tests\SetupClassResourcesTrait;
+use Symfony\Component\HttpFoundation\Response;
 
 abstract class UuidFilterBaseTestCase extends ApiTestCase
 {
@@ -147,16 +148,7 @@ public function testSearchFilterByInvalidUuid(): void
             ],
         ]);
 
-        self::assertResponseIsSuccessful();
-        $json = $response->toArray();
-
-        self::assertArraySubset(['hydra:totalItems' => 0], $json);
-        self::assertArraySubset(
-            [
-                'hydra:member' => [],
-            ],
-            $json
-        );
+        $this->assertResponseStatusCodeSame(Response::HTTP_BAD_REQUEST);
     }
 
     public function testSearchFilterByManyInvalidUuid(): void
@@ -174,16 +166,7 @@ public function testSearchFilterByManyInvalidUuid(): void
             ],
         ]);
 
-        self::assertResponseIsSuccessful();
-        $json = $response->toArray();
-
-        self::assertArraySubset(['hydra:totalItems' => 0], $json);
-        self::assertArraySubset(
-            [
-                'hydra:member' => [],
-            ],
-            $json
-        );
+        $this->assertResponseStatusCodeSame(Response::HTTP_BAD_REQUEST);
     }
 
     public function testSearchFilterOnManyToOneRelationByUuid(): void
@@ -288,16 +271,7 @@ public function testSearchFilterOnManyToOneRelationByInvalidUuids(): void
             ],
         ]);
 
-        self::assertResponseIsSuccessful();
-        $json = $response->toArray();
-
-        self::assertArraySubset(['hydra:totalItems' => 0], $json);
-        self::assertArraySubset(
-            [
-                'hydra:member' => [],
-            ],
-            $json
-        );
+        $this->assertResponseStatusCodeSame(Response::HTTP_BAD_REQUEST);
     }
 
     public function testGetOpenApiDescription(): void