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
Issue fork reroute_email-3177642
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:
- 3177642-whitelist-v3
changes, plain diff MR !15
- 3177642-whitelist-v2
changes, plain diff MR !9
- 3177642-whitelist
changes, plain diff MR !7
Comments
Comment #2
Pascal- commentedComment #7
bohartComment #12
bohart@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!