- Issue created by @tim bozeman
- Merge request !8528Issue #3456871 by Tim Bozeman: Support NULL services โ (Open) created by tim bozeman
- ๐บ๐ธUnited States tim bozeman
Oh thank you. That made the tests pass! ๐
- Status changed to Needs review
7 months ago 11:06pm 24 June 2024 - Status changed to Needs work
7 months ago 1:53pm 25 June 2024 - ๐บ๐ธUnited States smustgrave
can the MR be updated for 11.x please
Also feels like something that should have some test coverage maybe?
- ๐ฎ๐ณIndia samit.310@gmail.com
samit.310@gmail.com โ made their first commit to this issueโs fork.
- ๐ฎ๐ณIndia samit.310@gmail.com
samit.310@gmail.com โ changed the visibility of the branch 3456871-support-null-services to hidden.
- Status changed to Needs review
6 months ago 7:20am 18 July 2024 - ๐ฎ๐ณIndia samit.310@gmail.com
MR added with new branch
3456871-support-null-services-fix
against11.x
Thanks
Samit K. - Status changed to Needs work
6 months ago 1:25pm 18 July 2024 - ๐ฌ๐งUnited Kingdom longwave UK
Agree this needs tests. What is the use case? The Symfony link implies that if you use the
@?
syntax then NULL is passed if a service is not found, but I don't see what this has to do with the reverse container. - Status changed to Closed: works as designed
6 months ago 2:53pm 18 July 2024 - ๐บ๐ธUnited States tim bozeman
Thank you longwave. The only thing it has to do with the reverse container is that it throws an error when a service is null. I'm not totally sure that null services was even the intention of the
@?
syntax though. After digging through it a bit more it seems like the best way forward in general is not to rely on core supporting null services and just handle whether a service is viable in a service. ๐