Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 296340

Fork Symfony's ContainerAwareTrait and ContainerAwareInterface into core

$
0
0

Problem/Motivation

In #3392616-5: Update to Symfony 6.4 we've found out that the Symfony\Component\DependencyInjection\ContainerAwareTrait and Symfony\Component\DependencyInjection\ContainerAwareInterface are being deprecated in Symfony 6.4 and removed in 7.0.

Proposed resolution

We can't get rid off all implementations and will need to fork those into core (#3392616-8: Update to Symfony 6.4 and #3392616-20: Update to Symfony 6.4)

However, we want to get rid of as many usage as possible, for that there is #3397519: [META] Reduce use of ContainerAware classes where possible.

MR!5181 is the actual MR, but since that also removes all the deprecation suppression it currently fails (hard...).

However if we put the MRs of the children of #3397519: [META] Reduce use of ContainerAware classes where possible on top of that MR, as done in MR!5182, we are fully green.

Remaining tasks

Postponed on #3397173: Convert both BookNavigationCacheContext and MenuActiveTrailsCacheContext to use lazy services
Review

User interface changes

API changes

Data model changes

Release notes snippet


Viewing all articles
Browse latest Browse all 296340

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>