Problem/Motivation

Running PHPUnit tests with  --display-deprecations  on Drupal 11 shows:

token_requirements without a #[LegacyRequirementsHook] attribute is deprecated in drupal:11.3.0 and removed in drupal:13.0.0.

Steps to reproduce

Run PHPUnit tests with  --display-deprecations  on Drupal 11.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork token-3618758

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:

Comments

stinis87 created an issue. See original summary.

ankit_kumar’s picture

Assigned: Unassigned » ankit_kumar

ankit_kumar changed the visibility of the branch 3618758-tokenrequirements-triggers-deprecation to hidden.

ankit_kumar changed the visibility of the branch 3618758-tokenrequirements-triggers-deprecation to active.

berdir’s picture

Status: Active » Closed (duplicate)

This already happened in the dev version and will be in the next release.

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.

ankit_kumar’s picture

Assigned: ankit_kumar » Unassigned
Status: Closed (duplicate) » Active

I checked the current Token module code and found that the `#[LegacyRequirementsHook]` attribute is already present in `token.install`.

The attribute appears to have been added in commit `e0953a5687d3cdeaecd45dd46d9d6bc0be61a71e`, which references issue #3586686.

Could someone confirm whether this existing change also resolves this issue, or whether any additional changes are required?

Thanks!

ankit_kumar’s picture

Status: Active » Closed (duplicate)

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.