[DependencyInjection] Add #[Lazy] attribute
#19506
Conversation
OskarStark
changed the title
#[Lazy] attribute
service_container/lazy_services.rst
Outdated
| // ... | ||
| } | ||
|
|
||
| This attribute can be used on a class or on a parameter which should be lazy-loaded, and has a ``lazy`` parameter |
There was a problem hiding this comment.
I don't understand this part:
and has a ``lazy`` parameter that also supports defining interfaces to proxy and intersection types::
What's the purpose of the lazy option? (which, by the way, looks like a bad option name to me because the attribute is also called lazy)_. Is "type intersection" related to lazy option in any way? Thanks!
There was a problem hiding this comment.
It's a compatibility issue, because the same param exists in both #[Autoconfigure] and #[Autowire], and that way the parameter handling is the same.
I admit I'm not sure how to phrase this. But since this is the only parameter, maybe the name can be dropped in the example? Like:
and has a parameter that also supports defining interfaces to proxy and intersection types::
|
Upping this PR. I made modifications, is there anything else I can do to improve it? @javiereguiluz @alexandre-daubois |
javiereguiluz
force-pushed
the
add_lazy_attribute
branch
from
b17bd5f to
6f05767
Compare
|
Thanks Benjamin! This is now merged. |
Fixes #19486
Documentation for the new
#[Lazy]attribute.