Problem/Motivation

In Drupal 12, it will be mandatory that plugin managers have an Attribute class instead of/as well as an Annotation Class. This is currently deprecated in Drupal 11.2.x and in some case, like PHPunit tests, will output an unwanted HTTP header which can, when there are many of them (not limited to Pathauto) can stop the tests from working and producing a 502 error.

Steps to reproduce

Not currently a problem, but will be for a Drupal 12 version

Proposed resolution

Create a AliasType Attribute class for AliasTypeManager to sit along side the Annotation class so we're ready for Drupal 12 and to aid tests in the present.

Remaining tasks

Create Attribute AliasType class
Update AliasTypeManager to include the new Attribute class
Update any module provided plugins to include the new Attribute.

User interface changes

N/A

API changes

N/A

Data model changes

N/A

I will start work on an MR right away.

Issue fork pathauto-3591344

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

christiansanders created an issue. See original summary.

christiansanders’s picture

christiansanders commented
Title: Not supporting attribute discovery in AliasTypeManager » Support attribute discovery in AliasTypeManager
christiansanders’s picture

christiansanders commented

Looks like this code is already in the dev branch.

christiansanders’s picture

christiansanders commented
Status: Active » Closed (outdated)

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.