2.13.2 breaks all queries with object hydration for entities using Enum fields #7044

New Issue

admin · 2026-01-22T15:43:36+01:00

admin commented

2026-01-22 15:43:36 +01:00

Originally created by @gndk on GitHub (Sep 22, 2022).

BC Break Report

Q	A
BC Break	yes
Version	2.13.2

Summary

This change in #10041 breaks all queries with object hydration for entities with an Enum field.

Previous behavior

It worked.

Current behavior

Exception with stack trace:

TypeError:
App\Domain\Channel\ChannelId::from(): Argument #1 ($value) must be of type string, App\Domain\Channel\ChannelId given

  at vendor/doctrine/orm/lib/Doctrine/ORM/Mapping/ReflectionEnumProperty.php:93
  at App\Domain\Channel\ChannelId::from()
     (vendor/doctrine/orm/lib/Doctrine/ORM/Mapping/ReflectionEnumProperty.php:93)
  at Doctrine\ORM\Mapping\ReflectionEnumProperty->initializeEnumValue()
     (vendor/doctrine/orm/lib/Doctrine/ORM/Mapping/ReflectionEnumProperty.php:77)
  at Doctrine\ORM\Mapping\ReflectionEnumProperty->setValue()
     (vendor/doctrine/orm/lib/Doctrine/ORM/UnitOfWork.php:2745)
  at Doctrine\ORM\UnitOfWork->createEntity()
     (vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php:266)
  at Doctrine\ORM\Internal\Hydration\ObjectHydrator->getEntity()
     (vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php:492)
  at Doctrine\ORM\Internal\Hydration\ObjectHydrator->hydrateRowData()
     (vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php:148)
  at Doctrine\ORM\Internal\Hydration\ObjectHydrator->hydrateAllData()
     (vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/AbstractHydrator.php:270)
  at Doctrine\ORM\Internal\Hydration\AbstractHydrator->hydrateAll()
     (vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php:1224)
  at Doctrine\ORM\AbstractQuery->executeIgnoreQueryCache()
     (vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php:1165)
  at Doctrine\ORM\AbstractQuery->execute()
     (vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php:953)
  at Doctrine\ORM\AbstractQuery->getOneOrNullResult()
     (src/Infrastructure/Doctrine/ORM/Repository/OrderRepository.php:51)
  at App\Infrastructure\Doctrine\ORM\Repository\OrderRepository->get()
     (src/Infrastructure/Symfony/Controller/Account/OrderDetailAction.php:43)
  at App\Infrastructure\Symfony\Controller\Account\OrderDetailAction->__invoke()
     (vendor/symfony/http-kernel/HttpKernel.php:153)
  at Symfony\Component\HttpKernel\HttpKernel->handleRaw()
     (vendor/symfony/http-kernel/HttpKernel.php:75)
  at Symfony\Component\HttpKernel\HttpKernel->handle()
     (vendor/symfony/http-kernel/Kernel.php:202)
  at Symfony\Component\HttpKernel\Kernel->handle()
     (vendor/symfony/runtime/Runner/Symfony/HttpKernelRunner.php:35)
  at Symfony\Component\Runtime\Runner\Symfony\HttpKernelRunner->run()
     (vendor/autoload_runtime.php:29)
  at require_once('/srv/share/vendor/autoload_runtime.php')
     (public/index.php:7)

How to reproduce

I have an Order entity with a ChannelId field that looks like this (simplified):

/** @final */
#[ORM\Entity]
#[ORM\Table(name: '`order`')]
class Order
{
    /** @psalm-immutable */
    #[ORM\Id]
    #[ORM\Column(type: 'order_id')]
    private OrderId $id

    /** @psalm-immutable */
    #[ORM\Column(type: 'string', enumType: ChannelId::class)]
    private ChannelId $channelId;
}

And a repository (simplified) with this method:

/** @throws OrderWithIdDoesNotExist */
public function get(OrderId $orderId): Order
{
    $manager = $this->registry->getManagerForClass(Order::class);
    $repository = $manager->getRepository(Order::class);
        
    $queryBuilder = $repository->createQueryBuilder('o');
    $queryBuilder->where('o.id = :id');
    $queryBuilder->setParameter('id', $orderId->toString());

    $order = $queryBuilder->getQuery()->getOneOrNullResult();

    if (! $order instanceof Order) {
        throw new OrderWithIdDoesNotExist($orderId);
    }

    return $order;
}

I'll see if I can come up with reproducing test case.

Originally created by @gndk on GitHub (Sep 22, 2022).  ### BC Break Report  | Q | A |------------ | ------ | BC Break | yes | Version | 2.13.2 #### Summary [This change](https://github.com/doctrine/orm/pull/10041/files#diff-f1936bc3177c94ebd06906a54a439e496b410cfee881d6740ac344f03382e0ddR468) in #10041 breaks all queries with object hydration for entities with an Enum field. #### Previous behavior It worked. #### Current behavior Exception with stack trace: ``` TypeError: App\Domain\Channel\ChannelId::from(): Argument #1 ($value) must be of type string, App\Domain\Channel\ChannelId given at vendor/doctrine/orm/lib/Doctrine/ORM/Mapping/ReflectionEnumProperty.php:93 at App\Domain\Channel\ChannelId::from() (vendor/doctrine/orm/lib/Doctrine/ORM/Mapping/ReflectionEnumProperty.php:93) at Doctrine\ORM\Mapping\ReflectionEnumProperty->initializeEnumValue() (vendor/doctrine/orm/lib/Doctrine/ORM/Mapping/ReflectionEnumProperty.php:77) at Doctrine\ORM\Mapping\ReflectionEnumProperty->setValue() (vendor/doctrine/orm/lib/Doctrine/ORM/UnitOfWork.php:2745) at Doctrine\ORM\UnitOfWork->createEntity() (vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php:266) at Doctrine\ORM\Internal\Hydration\ObjectHydrator->getEntity() (vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php:492) at Doctrine\ORM\Internal\Hydration\ObjectHydrator->hydrateRowData() (vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php:148) at Doctrine\ORM\Internal\Hydration\ObjectHydrator->hydrateAllData() (vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/AbstractHydrator.php:270) at Doctrine\ORM\Internal\Hydration\AbstractHydrator->hydrateAll() (vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php:1224) at Doctrine\ORM\AbstractQuery->executeIgnoreQueryCache() (vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php:1165) at Doctrine\ORM\AbstractQuery->execute() (vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php:953) at Doctrine\ORM\AbstractQuery->getOneOrNullResult() (src/Infrastructure/Doctrine/ORM/Repository/OrderRepository.php:51) at App\Infrastructure\Doctrine\ORM\Repository\OrderRepository->get() (src/Infrastructure/Symfony/Controller/Account/OrderDetailAction.php:43) at App\Infrastructure\Symfony\Controller\Account\OrderDetailAction->__invoke() (vendor/symfony/http-kernel/HttpKernel.php:153) at Symfony\Component\HttpKernel\HttpKernel->handleRaw() (vendor/symfony/http-kernel/HttpKernel.php:75) at Symfony\Component\HttpKernel\HttpKernel->handle() (vendor/symfony/http-kernel/Kernel.php:202) at Symfony\Component\HttpKernel\Kernel->handle() (vendor/symfony/runtime/Runner/Symfony/HttpKernelRunner.php:35) at Symfony\Component\Runtime\Runner\Symfony\HttpKernelRunner->run() (vendor/autoload_runtime.php:29) at require_once('/srv/share/vendor/autoload_runtime.php') (public/index.php:7) ``` #### How to reproduce I have an `Order` entity with a `ChannelId` field that looks like this (simplified): ``` /** @final */ #[ORM\Entity] #[ORM\Table(name: '`order`')] class Order { /** @psalm-immutable */ #[ORM\Id] #[ORM\Column(type: 'order_id')] private OrderId $id /** @psalm-immutable */ #[ORM\Column(type: 'string', enumType: ChannelId::class)] private ChannelId $channelId; } ``` And a repository (simplified) with this method: ``` /** @throws OrderWithIdDoesNotExist */ public function get(OrderId $orderId): Order { $manager = $this->registry->getManagerForClass(Order::class); $repository = $manager->getRepository(Order::class); $queryBuilder = $repository->createQueryBuilder('o'); $queryBuilder->where('o.id = :id'); $queryBuilder->setParameter('id', $orderId->toString()); $order = $queryBuilder->getQuery()->getOneOrNullResult(); if (! $order instanceof Order) { throw new OrderWithIdDoesNotExist($orderId); } return $order; } ``` I'll see if I can come up with reproducing test case.

admin added the Bug Regression labels 2026-01-22 15:43:36 +01:00

admin closed this issue

2026-01-22 15:43:36 +01:00

admin commented

2026-01-22 15:43:37 +01:00

@gndk commented on GitHub (Sep 22, 2022):

Here is a new test case that works on 2.13.1 and fails on 2.13.2:

public function testEnumHydrationObjectHydrator(): void
{
    $this->setUpEntitySchema([Card::class, CardWithNullable::class]);

    $card       = new Card();
    $card->suit = Suit::Clubs;

    $this->_em->persist($card);
    $this->_em->flush();
    $this->_em->clear();

    $result = $this->_em->createQueryBuilder()
            ->from(Card::class, 'c')
            ->select('c')
            ->getQuery()
            ->getResult();

    $this->assertInstanceOf(Suit::class, $result[0]->suit);
    $this->assertEquals(Suit::Clubs, $result[0]->suit);
}

@gndk commented on GitHub (Sep 22, 2022): Here is a new test case that works on `2.13.1` and fails on `2.13.2`: ``` public function testEnumHydrationObjectHydrator(): void { $this->setUpEntitySchema([Card::class, CardWithNullable::class]); $card = new Card(); $card->suit = Suit::Clubs; $this->_em->persist($card); $this->_em->flush(); $this->_em->clear(); $result = $this->_em->createQueryBuilder() ->from(Card::class, 'c') ->select('c') ->getQuery() ->getResult(); $this->assertInstanceOf(Suit::class, $result[0]->suit); $this->assertEquals(Suit::Clubs, $result[0]->suit); } ```

admin commented

2026-01-22 15:43:37 +01:00

@webda2l commented on GitHub (Sep 22, 2022):

I confirm indeed. Better to keep 2.13.1 for now.

@webda2l commented on GitHub (Sep 22, 2022): I confirm indeed. Better to keep 2.13.1 for now.

admin commented

2026-01-22 15:43:37 +01:00

@XhmikosR commented on GitHub (Sep 25, 2022):

Would it be possible to cut a new patch release with this fix please?

@XhmikosR commented on GitHub (Sep 25, 2022): Would it be possible to cut a new patch release with this fix please?

admin commented

2026-01-22 15:43:38 +01:00