search

silverstripe / silverstripe-framework

Silverstripe Framework, the MVC framework that powers Silverstripe CMS
https://www.silverstripe.org
BSD 3-Clause "New" or "Revised" License
720 stars 823 forks source link

TODO: Injector::inject improvements #10988

Closed sabina-talipova closed 10 months ago

sabina-talipova commented 10 months ago

Description

Track all the existing objects that have had a service bound into them, so we can update that binding at a later point if need be (ie if the managed service changes).

See: https://github.com/silverstripe/silverstripe-framework/blob/cf78a9d785b9959a92dfb52c2a456bea80a4c567/src/Core/Injector/Injector.php#L669

GuySartorelli commented 10 months ago

Gonna close this one - it honestly seems like it'd be more confusing than useful.