Problem/Motivation

Drupal supports autowire and autoconfigure for services.

Autoconfigure: https://www.drupal.org/node/3357408
Autowire: https://www.drupal.org/node/3218156

Proposed resolution

Autowire / autoconfigure services

Remaining tasks

  1. Write a patch
  2. Review
  3. Commit

User interface changes

None.

API changes

None.

Data model changes

None.

Issue fork stage_file_proxy-3463085

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:

About issue forks

Comments

idebr created an issue. See original summary.

idebr opened merge request !79

idebr’s picture

idebr commented
Status: Active » Needs review

smustgrave’s picture

smustgrave commented
Status: Needs review » Fixed

Nice update.

smustgrave’s picture

smustgrave commented

Removing credit from myself

Status: Fixed » Closed (fixed)

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

smustgrave’s picture

smustgrave commented
Status: Closed (fixed) » Needs work

Ended up having to revert this as it caused a fatal error on a few sites #3468809: AutowiringFailedException breaking site

idebr opened merge request !84

idebr’s picture

idebr commented
Status: Needs work » Needs review

Fixed the autowring for the lock service. For reference see #3429527: Automated Drupal 11 compatibility fixes for content_lock

smustgrave’s picture

smustgrave commented

Thanks for the quick response, will try and test this week.

smustgrave closed merge request !84

  • smustgrave committed 7abad8e0 on 3.1.x 
    Issue #3463085 by idebr: Autowire / autoconfigure services
smustgrave’s picture

smustgrave commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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