[wp-trac] [WordPress Trac] #50413: Update code and comments to remove "blacklist" and "whitelist"
WordPress Trac
noreply at wordpress.org
Wed Jul 22 17:01:34 UTC 2020
#50413: Update code and comments to remove "blacklist" and "whitelist"
---------------------------------------------------+-----------------------
Reporter: strangerstudios | Owner: desrosj
Type: defect (bug) | Status: reopened
Priority: normal | Milestone: 5.5
Component: General | Version:
Severity: normal | Resolution:
Keywords: has-patch needs-dev-note dev-feedback | Focuses:
---------------------------------------------------+-----------------------
Comment (by johnjamesjacoby):
@desrosj Huge thanks for patching this so quickly.
approach-2.diff is awesome. It is exactly what WordPress needs for these
rare occurrences where option keys are deprecated. It seems worthwhile to
consider this for the 5.5 release, pending approval from relevant
stakeholders. It is complete, and elegant.
Re: direct database queries and direct cache access, I think it's OK not
to catch those cases. Specifically, it would make the `550` upgrade
routine difficult, requiring some kind of core bypass to allow it to do
its direct query.
approach-1.diff works for me. It is enough to circumvent the problem as
I've identified it. It is not ideal, but it is enough.
🙏
--
Ticket URL: <https://core.trac.wordpress.org/ticket/50413#comment:40>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list