Opened 19 months ago
Closed 7 weeks ago
#3538 closed defect (bug) (fixed)
blacklist_keys deprecated
Reported by: | Robin W | Owned by: | |
---|---|---|---|
Milestone: | 2.6.11 | Priority: | normal |
Severity: | normal | Version: | |
Component: | General | Keywords: | |
Cc: |
Description
bbpress/includes/common/functions.php line 821 states
<?php $option_name = 'blacklist_keys';
this is deprecated and needs changing to
<?php disallowed_keys
Change History (5)
#2
follow-up:
↓ 3
@
2 months ago
It appears to me that this is fixed in 2.6.11 (but still "new defect" here). Am I confused, or should this be marked done?
#3
in reply to:
↑ 2
;
follow-up:
↓ 4
@
7 weeks ago
Replying to mike80222:
It appears to me that this is fixed in 2.6.11 (but still "new defect" here). Am I confused, or should this be marked done?
It looks like it's still there:
https://github.com/bbpress/bbPress/blob/trunk/src/includes/common/functions.php#L866-L870
#4
in reply to:
↑ 3
@
7 weeks ago
Replying to dd32:
I don't actually understand how to view the source on github, but after updating I see this code on my site, around line 866:
// Strict mode uses WordPress "blacklist" settings if ( true === $strict ) { $hook_name = 'blacklist'; $option_name = 'disallow_keys';
#5
@
7 weeks ago
- Milestone changed from Awaiting Review to 2.6.11
- Resolution set to fixed
- Status changed from new to closed
Oh.. well, that's interesting :) turns out the GitHub mirror has ceased syncing.
It looks like this was indeed resolved in 2.6.11 via [7270] which may have mentioned the wrong ticket, or just not known about this one.
In [7270]:
Common: update blacklist_keys to disallow_keys
This change prevents a deprecated option notice in WordPress versions greater than 5.5.
See: #3601
In branches/2.6, for 2.6.11.
Marking this as resolved.
I tried that change and it does appear to work.