Skip to:
Content

bbPress.org

Opened 19 months ago

Closed 7 weeks ago

#3538 closed defect (bug) (fixed)

blacklist_keys deprecated

Reported by: robin-w's profile 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)

#1 @mike80222
19 months ago

I tried that change and it does appear to work.

#2 follow-up: @mike80222
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: @dd32
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 @mike80222
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 @dd32
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.

Note: See TracTickets for help on using tickets.