Whitelist is not an accurate term for an allow_list, it's also racial stereotyping.

Let's find a new name and change it.
I suggest: allowlist

Pretty sure blacklist is not used in this module?
If it is, change that to denylist as well.

I'll try creating a patch for this if I find the time in the coming weeks.

Remaining tasks

- Update the code base (maintain backwards compatibility with whitelist?)
- Add update hook to preserve functionality (if required?)
- Update translations where necessary (if required?)

Here's an example of another module that recently made the change:
https://www.drupal.org/project/eu_cookie_compliance/issues/3150156

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

Pascal- created an issue. See original summary.

Pascal-’s picture

Title: Rename whitelist to something more accurate like allow_list » Rename whitelist to something more accurate like allowlist
Issue summary: View changes

VEoz made their first commit to this issue’s fork.

bohart made their first commit to this issue’s fork.

bohart’s picture

Issue tags: +LutskCodeSprint2021

  • bohart committed 6e504e2 on 2.x
    Issue #3177642 by VEoz, bohart: Updated tests class names and test...
  • bohart committed 9493e21 on 2.x authored by VEoz
    Issue #3177642 by VEoz, bohart: Rename whitelist to allowlist.
    
  • bohart committed d95ac4f on 2.x
    Issue #3177642: Renamed configuration properties from whitelist to...
bohart’s picture

Version: 2.0.0 » 2.x-dev
Status: Active » Fixed

@Pascal-, thanks for paying attention to that!
@VEoz, thanks for your contribution!

Those changes were tested and reviewed by our team at #LutskCodeSprint2021.
Committed to 2.x-dev branch. There is no backport planned to 8.x-1.x branch.

So, marking this one as fixed.
Thanks!

Status: Fixed » Closed (fixed)

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