feat(doctrine): uuid filter #7628

aaa2000 · 2025-12-22T16:53:51Z

Q	A
Branch?	main for features / current stable version branch for bug fixes
Tickets	Closes #5618 , #7465, #4313, #4656, #2838, #3880
License	MIT
Doc PR	api-platform/docs#...

Add Symfony uuid, Ramsey uuid and Ramsey uuid binary filters.

The PR avoids modifying the search filter to prevent regressions. Furthermore, supporting partial strategies for UUID requires more work. In PostgreSQL, I think we need to cast the UUID to a string in the SQL query.

Possible improvements:

filter by IRI
Add ODM support
Maybe implements \ApiPlatform\Metadata\JsonSchemaFilterInterface

src/Doctrine/Orm/Filter/AbstractUuidFilter.php

soyuka

Nice, also note I'm working on a range filter that can be used with many different types (soyuka@c74675d)

src/Doctrine/Orm/composer.json

aaa2000 · 2025-12-30T12:37:29Z

src/Doctrine/Orm/Filter/AbstractUuidFilter.php

+
+    public function getSchema(Parameter $parameter): array
+    {
+        return self::UUID_SCHEMA;


API Platform not automatically adds a Symfony Uuid/Ulid constraint, I could create another PR for that

at https://github.com/api-platform/core/blob/main/src/Validator/Util/ParameterValidationConstraints.php

src/Doctrine/Orm/Filter/AbstractUuidFilter.php

soyuka · 2025-12-31T09:34:54Z

src/Doctrine/Orm/Filter/AbstractUuidFilter.php

+            return $databaseValues;
+        }
+
+        return $values;


we should not need this this is done by Doctrine when applying parameters no? If this is really needed could you tell me why? Also you could have a guard clause in the callee and reduce the function complexity.

Doctrine handles type conversion automatically when you set the third argument in setParameter(). However, if the type is not explicitly set, UUIDs are treated as strings by default. Depending on the database platform, this can cause issues because if the platform natively supports UUIDs, the value should be converted to a binary representation.

Doctrine Documentation:

Calling setParameter() automatically infers which type you are setting as value.
This works for integers, arrays of strings/integers, DateTime instances, and for managed entities. If you want to set a type explicitly, you can call the third argument to setParameter() explicitly. It accepts either a DBAL Doctrine\DBAL\ParameterType::* or a DBAL Type name for conversion.

Symfony Documentation:

When using built-in Doctrine repository methods (e.g. findOneBy()), Doctrine knows how to convert these UUID types to build the SQL query. However, when using DQL queries or building the query yourself, you'll need to set uuid as the type of the UUID parameters.

In the application of fixture:

We have the configuration in config_common.yml:
https://github.com/aaa2000/core/blob/b8397aeec444d94d34e3631961e693560622a933/tests/Fixtures/app/config/config_common.yml#L12

doctrine: dbal: driver: 'pdo_sqlite' charset: 'UTF8' types: uuid: Ramsey\Uuid\Doctrine\UuidType uuid_binary: Ramsey\Uuid\Doctrine\UuidBinaryType symfony_uuid: Symfony\Bridge\Doctrine\Types\UuidType

So, the third argument should be uuid or uuid_binary or symfony_uuid, and the array of uuid strings should be transformed into an array of objects (Symfony uuid, Ramsey uuid...). Note: we can use the convertToPHPValue method of the Doctrine type for this https://github.com/symfony/symfony/blob/8.0/src/Symfony/Bridge/Doctrine/Types/AbstractUidType.php#L43

For WHERE ... IN() clauses, we cannot set the parameter type with uuid or uuid_binary or symfony_uuid. ArrayParameterType only supports INTEGER, STRING, ASCII, and BINARY.

UUIDs are stored as CHAR(36) by default, or in a platform-specific datatype if the database supports native UUIDs. PostgreSQL and SQL Server have native UUID support: PostgreSQLPlatform, SQLServerPlatform See also Doctrine DBAL: Mapping Matrix

For this reason, it seemed simpler to convert the UUIDs into their database representation.

Note that Symfony does the same thing in ORMQueryBuilderLoader https://github.com/symfony/symfony/blob/f31baa789d552e6cd3b349ea35aad6b67508a599/src/Symfony/Bridge/Doctrine/Form/ChoiceList/ORMQueryBuilderLoader.php#L70

src/Doctrine/Orm/Filter/AbstractUuidFilter.php

soyuka · 2025-12-31T09:37:44Z

src/Doctrine/Orm/Filter/AbstractUuidFilter.php

+
+        $values = (array) $value;
+        $associations = [];
+        if ($this->isPropertyNested($property, $resourceClass)) {


At some point I'd like to share the relation filtering logic into other filters, I'm still wondering how we'd make this good enough.

I'm really interested by the nested property filters soyuka so I would happy to help.
cf #7554 (comment) Currently I have a hack NestedFilter used this way

'target' => new QueryParameter(filter: new NestedFilter(new ExactFilter()), property: 'languagePair.target'),

but I think it would be awesome to support it natively.

Looking at your comments

Filtering on Nested Properties does not work with QueryParameter #7526 (comment)

feat(doctrine): new search filters #7121 (comment)

I thought that you didn't want to support nested properties. So I'd like to understand what changed in this AbstractUuidFilter ? If it's supported for AbstractUuidFilter can it be for ExactFilter and PartialSearchFilter ? If not, why the bug you wanted to avoid on those filter cannot happen on this filter ?

the bug is about the nested property not being of the type of the filter (main issue with the SearchFilter). We're working with @Maxcastel on providing the update for relations and nested props on all the filters. Can you show your NestedFilter ?

We're working with @Maxcastel on providing the update for relations and nested props on all the filters.

I'm happy to hear it, don't hesitate to ping me on your discussion if I can help :)

Can you show your NestedFilter ?

I'm not sure my NestedFilter will help cause I think it's kinda hacky but I can still share it

use OrmPropertyHelperTrait; use PropertyHelperTrait; public function __construct( private readonly FilterInterface $filter, ) { } /** * @param array<string, mixed> $context */ public function apply(QueryBuilder $queryBuilder, QueryNameGeneratorInterface $queryNameGenerator, string $resourceClass, ?Operation $operation = null, array $context = []): void { if ($this->filter instanceof ManagerRegistryAwareInterface) { $this->filter->setManagerRegistry($this->getManagerRegistry()); } if ($this->filter instanceof LoggerAwareInterface) { $this->filter->setLogger($this->getLogger()); } /** @var Parameter $parameter */ $parameter = $context['parameter']; $property = $parameter->getProperty(); if (null === $property) { throw new \InvalidArgumentException('Nested filter only supports single property.'); } [$alias, $property] = $this->resolveNestedJoins( $queryBuilder, $queryNameGenerator, $property, $resourceClass, ); $qb = clone $queryBuilder; $qb->resetDQLPart('from'); $qb->from(self::class, $alias); // Override the root alias of the queryBuilder $qb->resetDQLPart('where'); $qb->setParameters(new ArrayCollection()); $newParameter = $parameter->withProperty($property)->withProperties([$property]); $this->filter->apply( $qb, $queryNameGenerator, $resourceClass, $operation, ['parameter' => $newParameter] + $context ); $queryBuilder->andWhere($qb->getDQLPart('where')); $parameters = $queryBuilder->getParameters(); foreach ($qb->getParameters() as $p) { $parameters->add($p); } $queryBuilder->setParameters($parameters); }

and then I'm using it with Filter which does not supports nested property, this way

'target' => new QueryParameter(filter: new NestedFilter(new ExactFilter()), property: 'languagePair.target'),

src/Doctrine/Orm/Filter/AbstractUuidFilter.php

soyuka

Could you target main ?

src/Doctrine/Orm/Filter/AbstractUuidFilter.php

soyuka · 2026-01-03T08:50:27Z

src/Doctrine/Orm/Filter/AbstractUuidFilter.php

+        }
+
+        $values = $this->convertValuesToTheDatabaseRepresentationIfNecessary($queryBuilder, $doctrineTypeField, $values);
+        $this->addWhere($queryBuilder, $queryNameGenerator, $associationAlias, $associationField, $values);


Suggested change

$this->addWhere($queryBuilder, $queryNameGenerator, $associationAlias, $associationField, $values);

if ($values) {

$this->addWhere($queryBuilder, $queryNameGenerator, $associationAlias, $associationField, $values);

}

This cannot be done because if there are multiple invalid UUIDs, there is no filter and all resources are fetched. So that doesn't seem okay to me.

If I add automatically a Symfony Uuid/Ulid constraint in ParameterValidationConstraints or if I rethrow the ConversionException, I could add it.

@see testSearchFilterOnManyToOneRelationByInvalidUuids

aaa2000 · 2026-01-05T13:58:46Z

composer.json

        "illuminate/support": "^11.0 || ^12.0",
        "jangregor/phpstan-prophecy": "^2.1.11",
-        "justinrainbow/json-schema": "^5.2.11",
+        "justinrainbow/json-schema": "^6.5.2",


cherry pick of #7619 to fix the behat tests on main branch

I've merged 4.2 onto main just now

soyuka

Nice work! a few small comments, let me know if you think its fine like this, I'm concerned about the addWhere being called with a null value.

src/Doctrine/Orm/Filter/AbstractUuidFilter.php

aaa2000 · 2026-01-06T18:10:56Z

@soyuka I updated the PR, I throw an InvalidArgumentException if there is a conversion exception. When the uuid is invalid, the response is now a bad request response instead of a success response with 0 items.
And the open api parameters depends on the definition of the user parameter by checking its schema. (I hope I understood the comment correctly.)

soyuka · 2026-01-08T14:11:55Z

Thanks! I should've been more specific about the schema/openapi, I took the liberty of adding a new commit for this at 9f982d1 (#7628)

Could you tell me if you're fine with my changes?
The rest LGTM!

…tation

aaa2000 · 2026-01-09T09:20:36Z

@soyuka Yes, everything looks good to me. Thanks for addressing this! The jod [CI / PHPUnit api-platform/serializer (PHP 8.5 lowest) has failed, to fix it, I think it is necessary to require "sebastian/exporter": "^6.3.1" @see https://github.com/api-platform/core/pull/7585/changes#diff-d286be18d4ad1911898492160512bc18e46b2699ec8e7e48b15d1d4545abb712R42

soyuka · 2026-01-09T09:24:11Z

thanks a lot for tackling this issue! thats lots of PR actually to close :D

next big work on filters is continuing soyuka@c74675d that way we'll have uuid ranges available :) (feel free to grab the work if you're interested)

aaa2000 force-pushed the fix-7465-uuid-filter branch from f80b922 to 7f6964d Compare December 22, 2025 20:04

soyuka reviewed Dec 26, 2025

View reviewed changes

src/Doctrine/Orm/Filter/AbstractUuidFilter.php Outdated Show resolved Hide resolved

soyuka requested changes Dec 26, 2025

View reviewed changes

aaa2000 force-pushed the fix-7465-uuid-filter branch 2 times, most recently from b7e3778 to 100a022 Compare December 29, 2025 16:24

aaa2000 commented Dec 30, 2025

View reviewed changes

src/Doctrine/Orm/composer.json Show resolved Hide resolved

aaa2000 commented Dec 30, 2025

View reviewed changes

aaa2000 marked this pull request as ready for review December 30, 2025 12:37

soyuka requested changes Dec 31, 2025

View reviewed changes

aaa2000 force-pushed the fix-7465-uuid-filter branch from b8397ae to 5cb992d Compare January 2, 2026 20:35

soyuka requested changes Jan 3, 2026

View reviewed changes

aaa2000 force-pushed the fix-7465-uuid-filter branch from 5cb992d to 80c9d45 Compare January 5, 2026 12:04

aaa2000 changed the base branch from 4.2 to main January 5, 2026 12:15

aaa2000 force-pushed the fix-7465-uuid-filter branch from 80c9d45 to a6b7796 Compare January 5, 2026 12:19

aaa2000 commented Jan 5, 2026

View reviewed changes

aaa2000 force-pushed the fix-7465-uuid-filter branch from 4733b7d to 3719b08 Compare January 5, 2026 14:49

soyuka changed the title ~~fix #7465 Add uuid filter~~ feat(doctrine): uuid filter Jan 5, 2026

soyuka requested changes Jan 5, 2026

View reviewed changes

aaa2000 force-pushed the fix-7465-uuid-filter branch from 110cbaf to 860bbed Compare January 6, 2026 17:44

soyuka force-pushed the fix-7465-uuid-filter branch from a2b33c1 to 9f982d1 Compare January 8, 2026 14:11

soyuka force-pushed the fix-7465-uuid-filter branch 2 times, most recently from 177fc5f to b4fd07a Compare January 8, 2026 15:12

aaa2000 and others added 4 commits January 8, 2026 16:17

fix(metadata): add uuid filter (api-platform#7465)

061b581

Throw exception if the value could not converted to database represen…

6eef77c

…tation

Returns open api parameters according to schema

0405c88

schema/openapi parameter

1b68192

soyuka force-pushed the fix-7465-uuid-filter branch from b4fd07a to 1b68192 Compare January 8, 2026 15:17

soyuka mentioned this pull request Jan 8, 2026

OpenApiFactory auto-generates string schema type for foo[] filters. #7657

Closed

soyuka approved these changes Jan 8, 2026

View reviewed changes

soyuka merged commit d640d10 into api-platform:main Jan 9, 2026
148 of 149 checks passed

Uh oh!

feat(doctrine): uuid filter #7628

feat(doctrine): uuid filter #7628

Conversation

aaa2000 commented Dec 22, 2025

Uh oh!

Uh oh!

soyuka left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

soyuka Jan 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

soyuka left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

soyuka left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

aaa2000 commented Jan 6, 2026

Uh oh!

soyuka commented Jan 8, 2026

Uh oh!

aaa2000 commented Jan 9, 2026

Uh oh!

Uh oh!

soyuka commented Jan 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

soyuka Jan 9, 2026 •

edited

Loading

soyuka commented Jan 9, 2026 •

edited

Loading