Replace use of whitelist/blacklist in the path alias module

Created on 12 June 2020, about 4 years ago
Updated 26 April 2024, 2 months ago

Problem/Motivation

Lets remove usage of "blacklist" and "whitelist" in the path alias module.

They are:

  • An historic bad labelling of people
  • Provide no context: "what is listed in them"?

Proposed resolution

TBD

Will involve some BC layer(s):

  • 'path_alias.whitelist' service
  • All of AliasWhitelistInterface
  • AliasManager::$whitelist protected member?
  • AliasWhite cid parameter
  • ...

Remaining tasks

  1. Agree on exactly what needs BC layers + deprecations and what can be changed as protected implementation details.
  2. Fix everything:
    • core/modules/path/tests/src/Functional/PathAliasTest.php
    • core/modules/path_alias/tests/src/Unit/AliasManagerTest.php
    • core/modules/path_alias/tests/src/Kernel/AliasTest.php
    • core/modules/path_alias/path_alias.services.yml
    • core/modules/path_alias/src/AliasManager.php
    • core/modules/path_alias/src/AliasWhitelist.php
    • core/modules/path_alias/src/AliasWhitelistInterface.php
  3. Add tests for new deprecations, etc.
  4. Reviews, refinements.
  5. RTBC.
  6. Commit.

User interface changes

None

API changes

@todo

Data model changes

@todo

Release notes snippet

@todo

📌 Task
Status

Needs work

Version

11.0 🔥

Component
Other 

Last updated about 5 hours ago

Created by

🇬🇧United Kingdom alexpott 🇪🇺🌍

Live updates comments and jobs are added and updated live.
  • Novice

    It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.69.0 2024