Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.6k
[VarExporter] Fix lazy objects with hooked properties#59761
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
88649d9
intosymfony:6.4Uh oh!
There was an error while loading.Please reload this page.
stof commentedFeb 13, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I think this should have more tests to cover more cases:
|
That'd be nice indeed, help wanted! |
rixafy commentedFeb 22, 2025
@nicolas-grekas was this somewhere reverted? I see $hookedProperties but it's not used later in codehttps://github.com/symfony/symfony/blame/7.3/src/Symfony/Component/VarExporter/Internal/LazyObjectRegistry.php#L69 The part with foreach remained unchanged, unlike in this commit, and I can't figure out why. I'm getting |
rixafy commentedFeb 22, 2025
I see that in 6.4 the commit is OK and
|
Please open a new issue if you want this to be tracked. |
Uh oh!
There was an error while loading.Please reload this page.
With hook proxies when needed so that lazy objects are fully compatible with non-final hooks.