Problem/Motivation

In #3187857: [Symfony 6] Service deprecation messages require more fields we added code to support two different method signatures for Symfony\Component\DependencyInjection\Definition - this allows Drupal 9 to run on Symfony 5 (after a manual compuser update) without changes.

In Drupal 10 we'll require Symfony 5 or higher, so can drop the compatibility layer for Symfony 4.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3197729

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch created an issue. See original summary.

andypost’s picture

andypost’s picture

Spokje’s picture

Assigned: Unassigned » Spokje
Status: Postponed » Needs work

Unpostponing since there's now a Drupal 10.0.x-dev branch

Spokje’s picture

Unpostponing since there's now a Drupal 10.0.x-dev branch

Spokje’s picture

Assigned: Spokje » Unassigned
Status: Needs work » Postponed

No, re-postponing, since the current 10.0.x-dev is on

        {
            "name": "symfony/dependency-injection",
            "version": "v4.4.34",
            "source": {
                "type": "git",
                "url": "https://github.com/symfony/dependency-injection.git",
                "reference": "117d7f132ed7efbd535ec947709d49bec1b9d24b"
            },
daffie’s picture

Spokje’s picture

Status: Active » Needs review
longwave’s picture

Status: Needs review » Reviewed & tested by the community

Looks OK to me.

Could do with a followup to remove the cache.null service as per https://www.drupal.org/node/3066407 if an issue for this doesn't exist already.

Spokje’s picture

Thanks @longwave, couldn't find an issue for the removal of the cache.null service, so created a follow-up: #3258918: Remove deprecated 'cache.null' service

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 6923e5e and pushed to 10.0.x. Thanks!

  • alexpott committed 6923e5e on 10.0.x
    Issue #3197729 by Spokje: Remove forward/bc layer for Symfony Definition...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.