string not converted to array on empty check

[Cybr]

Function bbp_pre_get_posts_normalize_forum_visibility() at includes/forums/functions.php has an empty check:

// Get any existing post status
$post_stati = $posts_query->get( 'post_status' );

// Default to public status
if ( empty( $post_stati ) ) {
	$post_stati[] = bbp_get_public_status_id();

// Split the status string
} elseif ( is_string( $post_stati ) ) {
	$post_stati = explode( ',', $post_stati );
}

When empty, the value could be a string. This will lead into a fatal PHP error:
Uncaught Error: [] operator not supported for strings in /.../wp-content/plugins/bbpress/includes/forums/functions.php:1800

This causes a white screen of death at /wp-admin/edit.php?post_type=forum on PHP 7.1 and 4.8-alpha-39901.

Suggestion:

// Get any existing post status
$post_stati = $posts_query->get( 'post_status' );

// Default to public status
if ( empty( $post_stati ) ) {
	// -- We know it's empty. Just reset the array.
	$post_stati = array();

	$post_stati[] = bbp_get_public_status_id();

// Split the status string
} elseif ( is_string( $post_stati ) ) {
	$post_stati = explode( ',', $post_stati );
}

Cheers!

[johnjamesjacoby]

Hey there! I believe we've fixed this in 2.6, and I'll be putting out Beta 2 today.

Would you be able to test & confirm?

[johnjamesjacoby]

Coming back to confirm this is fixed in 2.6.