Changes to entities with readonly identifier are not persisted #7585

New Issue

Closed

opened 2026-01-22 15:53:52 +01:00 by admin · 1 comment

admin commented 2026-01-22 15:53:52 +01:00

Owner

Copy Link

Originally created by @adrav on GitHub (Jan 1, 2026).

Problem when using readonly property for ID in Entity

Q	A
Version affected	3.2.2 (but also 3.1.1)
Works fine in	2.18.2

Summary

When using a readonly property for the identifier (id) in an entity, Doctrine fails to properly manage entity state. After flushing changes to a related entity (updating House::city through a HouseFloor reference), subsequent queries return stale data: the House entity is reloaded with its previous state instead of the updated one. Data is not persisted to the database.

This issue does not occur in doctrine-bundle 2.18.2 and disappears if the readonly modifier is removed from the entity ID property, indicating a regression in how Doctrine handles entities with readonly identifiers.

composer:

package	version
doctrine/collections	2.4.0
doctrine/data-fixtures	2.2.0
doctrine/dbal	4.4.1
doctrine/deprecations	1.1.5
doctrine/doctrine-bundle	3.2.2
doctrine/doctrine-fixtures-bundle	4.3.1
doctrine/doctrine-migrations-bundle	3.7.0
doctrine/event-manager	2.0.1
doctrine/inflector	2.1.0
doctrine/instantiator	2.0.0
doctrine/lexer	3.0.1
doctrine/migrations	3.9.5
doctrine/orm	3.6.0
doctrine/persistence	4.1.1
doctrine/sql-formatter	1.5.3
phpstan/phpstan-doctrine	2.0.12
ramsey/uuid-doctrine	2.1.0
symfony/doctrine-bridge	6.4.26
symfony/doctrine-messenger	6.4.31

How to reproduce

I have two entites:

House.php

<?php declare(strict_types=1);

namespace App\Entity;

use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\Common\Collections\Collection;
use Doctrine\DBAL\Types\Types;
use Doctrine\ORM\Mapping as ORM;
use Ramsey\Uuid\Uuid;

#[ORM\Table(name: 'house')]
#[ORM\Entity()]
class House
{

    #[ORM\Id]
    #[ORM\Column(type: Types::GUID, unique: true)]
    private readonly string $id;

    #[ORM\Column()]
    private string $city;

    /**
     * @var Collection<int, HouseFloor>
     */
    #[ORM\OneToMany(targetEntity: HouseFloor::class, mappedBy: 'house')]
    private Collection $floors;

    public function __construct(
    ) {
        $this->id = Uuid::uuid4()->toString();
        $this->floors = new ArrayCollection();
    }
    public function getId(): string
    {
        return $this->id;
    }
    public function getCity(): string
    {
        return $this->city;
    }
    public function setCity(string $city): void
    {
        $this->city = $city;
    }
    public function getFloors(): Collection
    {
        return $this->floors;
    }
    public function setFloors(Collection $floors): void
    {
        $this->floors = $floors;
    }
}

HouseFloor.php

<?php declare(strict_types=1);

namespace App\Entity;

use App\Repository\HouseFloorRepository;
use Doctrine\DBAL\Types\Types;
use Doctrine\ORM\Mapping as ORM;
use Ramsey\Uuid\Uuid;

#[ORM\Table(name: 'house_floor')]
#[ORM\Entity(repositoryClass: HouseFloorRepository::class)]
class HouseFloor
{

    #[ORM\Id]
    #[ORM\Column(type: Types::GUID, unique: true)]
    private readonly string $id;

    #[ORM\Column(type: Types::INTEGER)]
    private int $number;

    #[ORM\ManyToOne(inversedBy: 'floors')]
    #[ORM\JoinColumn(nullable: false)]
    private House $house;

    public function __construct(House $house, int $number)
    {
        $this->id = Uuid::uuid4()->toString();
        $this->house = $house;
        $this->number = $number;
    }
    public function getNumber(): int
    {
        return $this->number;
    }
    public function setNumber(int $number): void
    {
        $this->number = $number;
    }
    public function getId(): string
    {
        return $this->id;
    }
    public function getHouse(): House
    {
        return $this->house;
    }
}

And simple controller:

    #[Route('/test/house/add', name: 'test_house_add')]
    public function testAddAction(): Response
    {
        $house = new House();
        $house->setCity('Krakow');
        $this->entityManager->persist($house);

        $houseFloor1 = new HouseFloor($house, 1);
        $this->entityManager->persist($houseFloor1);

        $houseFloor2 = new HouseFloor($house, 2);
        $this->entityManager->persist($houseFloor2);

        $this->entityManager->flush();

        return new Response('OK.');
    }

    #[Route('/test/house/listFloors', name: 'test_house_list')]
    public function testListAction(): Response
    {
        $list = $this->houseFloorRepository->findAll();

        foreach ($list as $floor) {
            $floor->getHouse()->getCity();
        }
        dump($list);

        die;

    }

    #[Route('/test/house/testFloor/{houseFloor}', name: 'test_house_test_floor')]
    public function testFloorAction(HouseFloor $houseFloor): Response
    {
        $houseFloor->getHouse()->setCity('Paris');
        $this->entityManager->flush();

        return new Response('OK. Flushed. New city:'.$houseFloor->getHouse()->getCity());
    }

Then reproduce as below:

1. https://localhost/test/house/add
2. https://localhost/test/house/listFloors
3. Get one housefloor GUID
4. Use it here: https://localhost/test/house/testFloor/{houseFloorGUID}
5. New "City name" will be displayed.
6. The new "city name" in House object is expected, but the old one is still here: https://localhost/test/house/listFloors

If you remove readonly property from House entity it works fine.

Expected behavior:
Modifying a managed entity and calling flush() should result in an UPDATE statement and persisted changes in the database.

Actual behavior:
With a readonly identifier, Doctrine does not detect the entity as dirty and silently skips persisting the change.

Workaround

This issue does not occur in doctrine-bundle 2.18.2 and disappears if the readonly modifier is removed from the ID property, indicating a regression in change-tracking when entities use PHP readonly identifiers.

Downgrade the doctrine-bundle package:

composer require doctrine/doctrine-bundle:2.18.2

Then it works fine with the readonly property on $id field.

Originally created by @adrav on GitHub (Jan 1, 2026). ### Problem when using readonly property for ID in Entity | Q | A |-------------------------------------------- | ------ | Version affected | 3.2.2 (but also 3.1.1) | Works fine in | 2.18.2 #### Summary When using a `readonly` property for the identifier (`id`) in an entity, Doctrine fails to properly manage entity state. After flushing changes to a related entity (updating `House::city` through a `HouseFloor` reference), subsequent queries return stale data: the `House` entity is reloaded with its previous state instead of the updated one. Data is not persisted to the database. This issue does not occur in `doctrine-bundle` 2.18.2 and disappears if the `readonly` modifier is removed from the entity ID property, indicating a regression in how Doctrine handles entities with `readonly` identifiers. #### composer: | package | version |-------------------------------------------- | ------ | doctrine/collections | 2.4.0 | doctrine/data-fixtures | 2.2.0 | doctrine/dbal | 4.4.1 | doctrine/deprecations | 1.1.5 | doctrine/doctrine-bundle | 3.2.2 | doctrine/doctrine-fixtures-bundle | 4.3.1 | doctrine/doctrine-migrations-bundle| 3.7.0 | doctrine/event-manager | 2.0.1 | doctrine/inflector | 2.1.0 | doctrine/instantiator | 2.0.0 | doctrine/lexer | 3.0.1 | doctrine/migrations | 3.9.5 | doctrine/orm | 3.6.0 | doctrine/persistence | 4.1.1 | doctrine/sql-formatter | 1.5.3 | phpstan/phpstan-doctrine | 2.0.12 | ramsey/uuid-doctrine | 2.1.0 | symfony/doctrine-bridge | 6.4.26 | symfony/doctrine-messenger | 6.4.31 #### How to reproduce I have two entites: House.php ``` <?php declare(strict_types=1); namespace App\Entity; use Doctrine\Common\Collections\ArrayCollection; use Doctrine\Common\Collections\Collection; use Doctrine\DBAL\Types\Types; use Doctrine\ORM\Mapping as ORM; use Ramsey\Uuid\Uuid; #[ORM\Table(name: 'house')] #[ORM\Entity()] class House { #[ORM\Id] #[ORM\Column(type: Types::GUID, unique: true)] private readonly string $id; #[ORM\Column()] private string $city; /** * @var Collection<int, HouseFloor> */ #[ORM\OneToMany(targetEntity: HouseFloor::class, mappedBy: 'house')] private Collection $floors; public function __construct( ) { $this->id = Uuid::uuid4()->toString(); $this->floors = new ArrayCollection(); } public function getId(): string { return $this->id; } public function getCity(): string { return $this->city; } public function setCity(string $city): void { $this->city = $city; } public function getFloors(): Collection { return $this->floors; } public function setFloors(Collection $floors): void { $this->floors = $floors; } } ``` HouseFloor.php ``` <?php declare(strict_types=1); namespace App\Entity; use App\Repository\HouseFloorRepository; use Doctrine\DBAL\Types\Types; use Doctrine\ORM\Mapping as ORM; use Ramsey\Uuid\Uuid; #[ORM\Table(name: 'house_floor')] #[ORM\Entity(repositoryClass: HouseFloorRepository::class)] class HouseFloor { #[ORM\Id] #[ORM\Column(type: Types::GUID, unique: true)] private readonly string $id; #[ORM\Column(type: Types::INTEGER)] private int $number; #[ORM\ManyToOne(inversedBy: 'floors')] #[ORM\JoinColumn(nullable: false)] private House $house; public function __construct(House $house, int $number) { $this->id = Uuid::uuid4()->toString(); $this->house = $house; $this->number = $number; } public function getNumber(): int { return $this->number; } public function setNumber(int $number): void { $this->number = $number; } public function getId(): string { return $this->id; } public function getHouse(): House { return $this->house; } } ``` And simple controller: ``` #[Route('/test/house/add', name: 'test_house_add')] public function testAddAction(): Response { $house = new House(); $house->setCity('Krakow'); $this->entityManager->persist($house); $houseFloor1 = new HouseFloor($house, 1); $this->entityManager->persist($houseFloor1); $houseFloor2 = new HouseFloor($house, 2); $this->entityManager->persist($houseFloor2); $this->entityManager->flush(); return new Response('OK.'); } #[Route('/test/house/listFloors', name: 'test_house_list')] public function testListAction(): Response { $list = $this->houseFloorRepository->findAll(); foreach ($list as $floor) { $floor->getHouse()->getCity(); } dump($list); die; } #[Route('/test/house/testFloor/{houseFloor}', name: 'test_house_test_floor')] public function testFloorAction(HouseFloor $houseFloor): Response { $houseFloor->getHouse()->setCity('Paris'); $this->entityManager->flush(); return new Response('OK. Flushed. New city:'.$houseFloor->getHouse()->getCity()); } ``` Then reproduce as below: 1. https://localhost/test/house/add 2. https://localhost/test/house/listFloors 3. Get one housefloor GUID 4. Use it here: https://localhost/test/house/testFloor/{houseFloorGUID} 5. New "City name" will be displayed. 6. The new "city name" in House object is expected, but the old one is still here: https://localhost/test/house/listFloors If you remove **readonly** property from **House** entity it works fine. **Expected behavior:** Modifying a managed entity and calling `flush()` should result in an UPDATE statement and persisted changes in the database. **Actual behavior:** With a `readonly` identifier, Doctrine does not detect the entity as dirty and silently skips persisting the change. #### Workaround This issue does not occur in `doctrine-bundle` 2.18.2 and disappears if the `readonly` modifier is removed from the ID property, indicating a regression in change-tracking when entities use PHP `readonly` identifiers. Downgrade the **doctrine-bundle** package: ``` composer require doctrine/doctrine-bundle:2.18.2 ``` Then it works fine with the **readonly** property on $id field.

admin closed this issue 2026-01-22 15:53:52 +01:00

admin commented 2026-01-22 15:53:53 +01:00

Author

Owner

Copy Link

@stof commented on GitHub (Jan 3, 2026):

This is much more likely to be related to versions of the orm than to the version of the bundle (downgrading the bundle might force a change of version of the ORM as well)

@stof commented on GitHub (Jan 3, 2026): This is much more likely to be related to versions of the orm than to the version of the bundle (downgrading the bundle might force a change of version of the ORM as well)

admin referenced this issue 2026-01-22 16:07:57 +01:00

[PR #7585] #7583 fix #10560

Sign in to join this conversation.

No Branch/Tag Specified

Branches Tags

3.6.x

2.20.x

dependabot/github_actions/2.20.x/doctrine/dot-github/dot-github/workflows/composer-lint.yml-14.0.0

3.7.x

4.0.x

2.21.x

old-prototype-3.x

3.6.2

3.6.1

3.6.0

2.20.9

3.5.8

3.5.7

3.5.6

3.5.5

2.20.8

3.5.4

3.5.3

2.20.7

3.5.2

2.20.6

3.5.1

3.5.0

3.4.4

3.4.3

3.4.2

2.20.5

3.4.1

3.4.0

3.3.4

2.20.4

3.3.3

2.20.3

3.3.2

2.20.2

3.3.1

2.20.1

3.3.0

3.2.3

2.20.0

2.19.8

3.2.2

2.19.7

3.2.1

2.19.6

3.2.0

3.1.4

3.1.3

2.19.5

3.1.2

2.19.4

3.1.1

2.19.3

2.19.2

2.19.1

3.1.0

2.19.0

3.0.3

2.18.3

3.0.2

2.18.2

3.0.1

2.18.1

3.0.0

2.18.0

2.17.5

3.0.0-RC1

2.17.4

2.17.3

2.17.2

2.17.1

3.0.0-beta2

2.17.0

2.16.3

3.0.0-beta1

2.16.2

2.16.1

2.16.0

2.15.5

2.15.4

2.15.3

2.15.2

2.15.1

2.15.0

2.14.3

2.14.2

2.14.1

2.14.0

2.13.5

2.13.4

2.13.3

2.13.2

2.13.1

2.13.0

2.12.4

2.12.3

2.12.2

2.12.1

2.12.0

2.11.3

2.11.2

2.11.1

2.11.0

2.10.5

2.10.4

2.10.3

2.10.2

2.10.1

2.10.0

2.9.6

2.9.5

2.9.4

2.9.3

2.9.2

2.9.1

2.9.0

2.8.5

2.8.4

2.8.3

2.8.2

2.8.1

2.8.0

2.7.5

2.7.4

v2.7.3

v2.7.2

v2.7.1

v2.7.0

v2.6.6

v2.6.5

v2.6.4

v2.6.3

v2.6.2

v2.6.1

v2.6.0

v2.5.14

v2.5.13

v2.5.12

v2.5.11

v2.5.10

v2.5.9

v2.5.8

v2.5.7

v2.5.6

v2.5.5

v2.5.4

v2.5.3

v2.5.2

v2.4.8

v2.5.1

v2.5.0

v2.5.0-RC2

v2.5.0-RC1

v2.5.0-beta1

v2.5.0-alpha2

v2.5.0-alpha1

v2.4.7

v2.4.6

v2.4.5

v2.4.4

v2.4.3

v2.3.6

v2.4.2

2.3.5

v2.4.1

v2.4.0

2.4.0-RC2

2.4.0-RC1

2.4.0-BETA2

2.3.4

2.3.3

2.4.0-BETA1

2.3.2

2.3.1

2.3.0

2.3.0-RC4

2.3.0-RC3

2.3.0-RC2

2.2.3

2.3.0-RC1

2.3.0-BETA1

2.1.7

2.2.2

2.2.1

2.1.6

2.2.0

2.2.0-RC1

2.2.0-BETA2

2.2.0-BETA1

2.1.5

2.1.4

2.1.3

2.1.2

2.1.1

2.0.7

2.1.0

2.1.0RC3

2.1.0RC2

2.1.0RC1

2.0.6

2.1.0BETA1

2.0.5

2.0.4

2.0.3

2.0.2

2.0.1

2.0.0

2.0.0RC2

2.0.0RC1

2.0.0-BETA4

2.0.0-BETA3

2.0.0-BETA2

2.0.0-BETA1

Labels

Clear labels

BC Break

Bug

Can't Fix

Deprecation

Documentation

DQL

Duplicate

Failing Test

Hacktoberfest

Improvement

Improvement

Incomplete

Invalid

Missing Tests

New Feature

PR Created

pull-request

Mirrored from GitHub Pull Request

Question

Regression

Status: Waiting feedback

Waiting feedback

WIP

Won't Fix

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

admin

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/archived-orm#7585