UuidBinary (or array of uuid) as parameter for QueryBuilder

[grachevko]

When doctrine make query with array of uuids as parameter, every uuid cast to string.
Result match nothing, because need cast to Bytes.
Is only to extend Uuid class and replace behavior of __toString() method?

[NavyCoat]

I think It can have something with this issue in Doctrine doctrine/orm#4632.

I also found some other issue that can be connected to this:
doctrine/orm#2991
doctrine/orm#4632
doctrine/orm#1339
doctrine/orm#574

@grachevko Did you found any solution for this?

[grachevko]

@NavyCoat i just override method __toString() to make it return Bytes.

[oniric85]

I'm having a similar problem when using Doctrine Paginator. I think binary UUID should not be recommended as long as we have these major issues.

[grachevko]

@oniric85 this is a doctrine issue, because it can't handle array of objects. Only array of int or strings.
I guess this issue not actual for PostgreSQL database. Because it has uuid field type, which can get uuid as string and seamlessly convert it to binary.

[oniric85]

I guess I'm going to convert my mapping information to string UUID for the time beeing, running MariaDB here :)

[grachevko]

@oniric85 try to set $fetchJoinCollection as false
https://github.com/doctrine/doctrine2/blob/master/lib/Doctrine/ORM/Tools/Pagination/Paginator.php#L63

[oniric85]

@grachevko that's what I'm doing in the meantime :) But this won't work for more complex queries I guess.

[ruslankovalov]

The same thing for me with EntityType in Symfony. The entity is not found since Symfony uses IN condition in its query and doctrine turns them to string UUIDs

[fritz-gerneth]

I run into the same issue when removing entities that use table inheritance. The JoinTablePersister does not use proper type-mapping either. So I went ahead and replaced the OrderedTimeCode's encode function to return the binary value. Also solved all issues with querying for an array of Uuids and such.

Pretty effective but not the cleanest way, rather would have Doctrine handle this correctly. But given how slow-moving Doctrine is we certainly didn't want to wait for them to fix this. Only downside is that now all other code has to convert the UUIDs into readable format on their end. Fortunately for us this is only on 2 places.

[jaikdean]

Doctrine has an undocumented and more explicit way of setting parameters now, where you may specify a DBAL type for each one. That solves this issue.

Example:

use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\ORM\Query\Parameter;

$em
    ->createQuery('SELECT b FROM \Foo\Bar b WHERE b.id = :id')
    ->setParameters(new ArrayCollection([
        new Parameter('id', $uuid, 'uuid_binary'),
    ]))
;

[grachevko]

@jaikdean try 'SELECT b FROM \Foo\Bar b WHERE b.id IN (:ids)'

[jaikdean]

Touché. I've not seen a clean way to do that. This should work?

->setParameters([
    'ids' => array_map(function ($uuid) {
        return $uuid->getBytes();
    }, $uuids),
])

[grachevko]

This should work, but you can't do that in every cases. In vendor files for example.