Clearing the em before accessing an uninitialized relation results in "typed property must not be accessed before initialization..." error #7083

New Issue

admin · 2026-01-22T15:44:15+01:00

admin commented

2026-01-22 15:44:15 +01:00

Originally created by @kbond on GitHub (Dec 22, 2022).

BC Break Report

Q	A
BC Break	yes
Version	2.14.0

Summary

Consider the following code:

$results = $repo->createQueryBuilder('e')
    ->innerJoin('e.relation', 'r')
    ->getQuery()
    ->execute()
;

$em->clear();

echo $results[0]->relation->value;

Previous behavior

In 2.13.5, would echo the relation's value property.

Current behavior

In 2.14.0:

Error: Typed property Relation::$value must not be accessed before initialization

@nicolas-grekas, could this be related to #10187?

Reproducer: #10337

Originally created by @kbond on GitHub (Dec 22, 2022). ### BC Break Report | Q | A |------------ | ------ | BC Break | yes | Version | 2.14.0 #### Summary Consider the following code: ```php $results = $repo->createQueryBuilder('e') ->innerJoin('e.relation', 'r') ->getQuery() ->execute() ; $em->clear(); echo $results[0]->relation->value; ``` #### Previous behavior In 2.13.5, would echo the relation's value property. #### Current behavior In 2.14.0: > Error: Typed property Relation::$value must not be accessed before initialization @nicolas-grekas, could this be related to #10187? Reproducer: #10337

admin closed this issue

2026-01-22 15:44:15 +01:00

admin commented

2026-01-22 15:44:16 +01:00

@darthf1 commented on GitHub (Dec 22, 2022):

I have the same exception, but thrown in my tests when a method is called on a proxied object. The proxied object is returned via the referenceRepository of LiipTestFixturesBundle

1) Tests\Functional\UserInterface\DataCube\Rest\DataCubeProjectionTest::testGetDataCubes
Error: Typed property Domain\DataCube\DataCube::$analysisType must not be accessed before initialization
/builds/xxx/api/src/Domain/DataCube/DataCube.php:101
/builds/xxx/api/var/cache/test/doctrine/orm/Proxies/__CG__DomainDataCubeDataCube.php:210
/builds/xxx/tests/Functional/UserInterface/DataCube/Rest/DataCubeProjectionTest.php:59

@darthf1 commented on GitHub (Dec 22, 2022): I have the same exception, but thrown in my tests when a method is called on a proxied object. The proxied object is returned via the [referenceRepository](https://github.com/liip/LiipTestFixturesBundle/blob/2.x/doc/database.md#referencing-fixtures-in-tests-) of [LiipTestFixturesBundle](https://github.com/liip/LiipTestFixturesBundle) ``` 1) Tests\Functional\UserInterface\DataCube\Rest\DataCubeProjectionTest::testGetDataCubes Error: Typed property Domain\DataCube\DataCube::$analysisType must not be accessed before initialization /builds/xxx/api/src/Domain/DataCube/DataCube.php:101 /builds/xxx/api/var/cache/test/doctrine/orm/Proxies/__CG__DomainDataCubeDataCube.php:210 /builds/xxx/tests/Functional/UserInterface/DataCube/Rest/DataCubeProjectionTest.php:59 ```

admin commented

2026-01-22 15:44:17 +01:00

@brilsergei commented on GitHub (Dec 23, 2022):

I have probably similar problem in tests using the referenceRepository of LiipTestFixturesBundle. I call method which should return a string (field of an entity) but it returns null. While debugging found that in older version of this lib in method \Doctrine\ORM\UnitOfWork::createEntity the condition if (isset($this->identityMap[$class->rootEntityName][$idHash])) { gets true while in 2.14 this condition gets false. identityMap is empty in UnitOfWork object.

@brilsergei commented on GitHub (Dec 23, 2022): I have probably similar problem in tests using the [referenceRepository](https://github.com/liip/LiipTestFixturesBundle/blob/2.x/doc/database.md#referencing-fixtures-in-tests-) of [LiipTestFixturesBundle](https://github.com/liip/LiipTestFixturesBundle). I call method which should return a string (field of an entity) but it returns null. While debugging found that in older version of this lib in method \Doctrine\ORM\UnitOfWork::createEntity the condition `if (isset($this->identityMap[$class->rootEntityName][$idHash])) {` gets true while in 2.14 this condition gets false. identityMap is empty in UnitOfWork object.

admin commented

2026-01-22 15:44:17 +01:00

@mathieudz commented on GitHub (Dec 24, 2022):

Same problem here since 2.14: createEntity ignores the existing proxy object (specified in the hint) and returns a new one instead. As result, the proxy object stays uninitialized. Results eventually in same error message as the title of this issue.

@mathieudz commented on GitHub (Dec 24, 2022): Same problem here since 2.14: createEntity ignores the existing proxy object (specified in the hint) and returns a new one instead. As result, the proxy object stays uninitialized. Results eventually in same error message as the title of this issue.

admin commented

2026-01-22 15:44:18 +01:00

@nicolas-grekas commented on GitHub (Dec 29, 2022):

Thanks for the reproducer. I will definitely have a look, just not before the 9th.

@nicolas-grekas commented on GitHub (Dec 29, 2022): Thanks for the reproducer. I will definitely have a look, just not before the 9th.

admin commented

2026-01-22 15:44:18 +01:00

@simonberger commented on GitHub (Jan 2, 2023):

The bug is caused by the removal of those three lines.
https://github.com/doctrine/orm/pull/10187/files#diff-c9c1faa5750ed5a13ec97a922d181d5b26a9f74fd79345cef5264725054cef2bL151-L153

After adding them again the test succeeds and none fails (in PHP 7.4 environment).
Yet there might be a good reason to do this change. My understanding is not good enough. The removed lines are looking rather isolated tho and all other usages of the constant are still in place.

@simonberger commented on GitHub (Jan 2, 2023): The bug is caused by the removal of those three lines. https://github.com/doctrine/orm/pull/10187/files#diff-c9c1faa5750ed5a13ec97a922d181d5b26a9f74fd79345cef5264725054cef2bL151-L153 After adding them again the test succeeds and none fails (in PHP 7.4 environment). Yet there might be a good reason to do this change. My understanding is not good enough. The removed lines are looking rather isolated tho and all other usages of the constant are still in place.

admin commented

2026-01-22 15:44:19 +01:00

@jetrodn commented on GitHub (Jan 3, 2023):

The bug is caused by the removal of those three lines. https://github.com/doctrine/orm/pull/10187/files#diff-c9c1faa5750ed5a13ec97a922d181d5b26a9f74fd79345cef5264725054cef2bL151-L153

After adding them again the test succeeds and none fails (in PHP 7.4 environment). Yet there might be a good reason to do this change. My understanding is not good enough. The removed lines are looking rather isolated tho and all other usages of the constant are still in place.

I have faced with the same issue and can confirm that reverting those lines fixed the issue in my case.

@jetrodn commented on GitHub (Jan 3, 2023): > The bug is caused by the removal of those three lines. https://github.com/doctrine/orm/pull/10187/files#diff-c9c1faa5750ed5a13ec97a922d181d5b26a9f74fd79345cef5264725054cef2bL151-L153 > > After adding them again the test succeeds and none fails (in PHP 7.4 environment). Yet there might be a good reason to do this change. My understanding is not good enough. The removed lines are looking rather isolated tho and all other usages of the constant are still in place. I have faced with the same issue and can confirm that reverting those lines fixed the issue in my case.

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/archived-orm#7083