- Issue created by @spokje
- Status changed to Fixed
about 1 year ago 7:45am 15 November 2023 - π¬π§United Kingdom alexpott πͺπΊπ
The child issues were resolved in 10.2.x
Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Active
11 months ago 9:48am 23 January 2024 - π¬π§United Kingdom longwave UK
Forgot this existed. This would be a better parent for many of the child issues I am creating than π Fork Symfony's ContainerAwareTrait and ContainerAwareInterface into core Needs work , so reopening.
- π«π·France andypost
Probably
\Drupal\Core\DependencyInjection\ClassResolver::getInstanceFromDefinition()
still should use deprecated interface for BCif ($instance instanceof ContainerAwareInterface) { $instance->setContainer($this->container); }
- π«π·France andypost
one more done π Remove ContainerAwareTrait from session middleware Fixed
- π¬π§United Kingdom longwave UK
Reparenting as the existing parent is closed.
π Notify downstream users that ContainerAware is going away Needs review proposes deprecating the code mentioned in #6.
- ππΊHungary GΓ‘bor Hojtsy Hungary
What else is left here? This the only issue left from the parent's issue summary. Also π [PP-2] Remove support for ContainerAwareInterface Postponed is linked from the parent, how are the two related?
- π¬π§United Kingdom longwave UK
Reparented the final removal under this issue, updated the IS as we can actually remove all implementations here.
- Status changed to RTBC
9 months ago 9:26am 21 March 2024 - π«π·France andypost
As fixed π [PP-2] Remove support for ContainerAwareInterface Postponed
The remaining one is good enough without meta - π Inject services into LoggerChannelFactory RTBC
- Status changed to Fixed
9 months ago 7:20pm 21 March 2024 - π¬π§United Kingdom catch
Just committed π Inject services into LoggerChannelFactory RTBC too. Nice to get all these done!
Automatically closed - issue fixed for 2 weeks with no activity.