Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[DependencyInjection] Use lazy-loading ghost object proxies out of the … #16976

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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
nicolas-grekas opened this issue Jul 12, 2022 · 3 comments
Closed

[DependencyInjection] Use lazy-loading ghost object proxies out of the … #16976

nicolas-grekas opened this issue Jul 12, 2022 · 3 comments
Labels
DependencyInjection hasPR A Pull Request has already been submitted for this issue.
Milestone
6.2

Comments

@nicolas-grekas
Copy link
Member

Q A
Feature PR symfony/symfony#46752
PR author(s) @nicolas-grekas
Merged in 6.2
@nicolas-grekas nicolas-grekas added this to the 6.2 milestone Jul 12, 2022
@javiereguiluz
Copy link
Member

Does this have any impact for developers? E.g. do they have to change anything in their apps? Can they remove some previously needed dependencies? etc. Thanks!

@nicolas-grekas
Copy link
Member Author

nicolas-grekas commented Jul 13, 2022 •
edited
Loading

Yes: they don't need proxy-manager-bridge anymore so we need to check where we tell about it and remove those mentions. It should just work out of the box now, and in the cases where it doesn't, it will throw an actionable error message.

@javiereguiluz javiereguiluz mentioned this issue Jul 18, 2022
Merged
@javiereguiluz javiereguiluz added the hasPR A Pull Request has already been submitted for this issue. label Jul 18, 2022
javiereguiluz added a commit that referenced this issue Jul 22, 2022
@javiereguiluz
minor #16995 [DependencyInjection] Use lazy-loading ghost object prox…
d4fb437 
…ies out of the box (javiereguiluz)

This PR was merged into the 6.2 branch.

Discussion
----------

[DependencyInjection] Use lazy-loading ghost object proxies out of the box

Fixes #16976.

Commits
-------

a6b283b [DependencyInjection] Use lazy-loading ghost object proxies out of the box
@javiereguiluz
Copy link
Member

Fixed in #16995.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
DependencyInjection hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Milestone
6.2
Development

No branches or pull requests
3 participants
@javiereguiluz @nicolas-grekas @carsonbot