Bug Report

Summary

When using a Doctrine\ORM\Tools\Pagination\Paginator to iterate over a query that has entities with a custom DBAL type used in the identifier, then $id->__toString() is used implicitly by PDO, instead of being converted by the Doctrine\DBAL\Types system.

In order to reproduce this, you must have identifiers implementing #__toString() (to allow the UnitOfWork to hash them) and other accessors that are used by the custom DBAL type during DB/PHP conversions. If #__toString() and the DBAL type conversions are asymmetric, then the paginator will fail to find records.

Tricky situation, but this very much affects ramsey/uuid-doctrine and anyone relying on the uuid_binary.

Current behavior

class MyEntity {
    public TheId $id;
}

class TheId {
    public function toId() : string
    {
        return 'yep';
    }
    public function __toString() : string
    {
        return 'nope';
    }
}

class TheIdType extends StringType
{
    public function convertToDatabaseValue($value, $platform)
    {
        return $value->toId();
    }
}

$entity = new MyEntity();
$entity->id = new TheId();

$em->persist($entity);
$em->flush();

$paginator = (new Paginator($em->createQuery('SELECT f FROM ' . MyEntity::class . ' f'));

foreach ($paginator as $entry) {
    die('this won't be reached with current ORM state');
}

Expected behavior

The code above should loop over one found record.