Problem/Motivation

Since Drupal 10.2, core has supported automatic autowiring of controller and form dependencies: https://www.drupal.org/node/3395716

Since our oldest supported core version is 10.4, we should adopt this to greatly reduce dependency injection boilerplate.

Issue fork project_browser-3539425

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

phenaproxima created an issue. See original summary.

phenaproxima opened merge request !847

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Active » Needs review
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Assigned: Unassigned » tim.plunkett

Assigning this one to @tim.plunkett for review, commit, and backport since this is the kind of code clean-up that is firmly in his wheelhouse.

phenaproxima opened merge request !854

tim.plunkett made their first commit to this issue’s fork.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Version: 2.1.x-dev » 2.0.x-dev
Assigned: tim.plunkett » Unassigned
Status: Needs review » Fixed

Merged !847 🎉 (2.1.x)
Merged !854 🎉 (2.0.x)

Love this clean-up/modernization

Status: Fixed » Closed (fixed)

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