You are welcome!

It is pleasure to assist.

Regards
Brajesh

Hi Tosin,
Code seems to be fine except for sanitizing url, wp_kses_post is overkill, esc_url_raw should be fine.

Also, your redirection callback is very inefficient, hitting database for user count on each page load.

Here is an updated version to avoid that.

// Check if registration is open or closed
function custom_bp_redirect_registration() {
	if ( ! bp_is_register_page() ) {
		return;
	}

	if ( get_option( 'bp-disable-site-registration', 'open' ) !== 'closed' ) {
		return;
	}

	$total_users                   = count_users();
	$total_users                   = $total_users['total_users'];
	$limit_user_registration_count = absint( get_option( 'bp-close-registration-at-users' ) );

	if ( $total_users >= $limit_user_registration_count ) {
		wp_redirect( get_option( 'bp-disable-site-registration-redirection-url' ) ); // wp_safe_redirect?
		exit;
	}
}

Regards
Brajesh

Hi Peter,
Can you please tell me on which site did you create the member types? Was it your main site? If yes, you can replace
BLOG_ID_CURRENT_SITE with 1. Otherwise, Please use the blog id on which you have created the member types.

PS:- BuddyPress and BuddyBoss both uses taxonomy terms as member type but they differ on the interface used for creation of the member types. BuddyBoss uses a post type for the creation interface while BuddyPress utilizes the taxonomy screen. It should not affect fetching of the terms(unless BuddyBoss fetches from post type instead of terms).

Regards
Brajesh

Hi Peter,
Thank you.
It’s good to know that it worked:)

Regards
Brajesh

Hi Nik,
Thank you for the question.
I see your point. We don’t have a none option for it.

I will suggest working with scoped group(only available to some groups). I will add the option “None” in next release.

Regards
Brajesh