Bart Kuijper

Sorry, I haven’t been able to respond earlier.

This is great, thank you very much for implementing this feature. I can’t believe you did it so quickly as well. Thanks again!

Alright one more addition, this time an alternative method. I created this because the previously posted method only worked to an extent. Certain bbp-style-pack options still didn’t work, probably because they rely on overruling parts of the bbpress.css which isn’t being loaded.

The code is again specific to the evolve theme, but it should be easy enough to adapt to a different theme.

How it works: the evolve theme hooks into wp_print_styles to deregister the bbp-default style. So we hook into that same action with a lower priority (PHP_INT_MIN) and unhook evolve’s function before it gets a chance to run. An important note about removing hooks: you don’t get an error message if removing the hook failed, and in order to succesfully remove a hook you need to specify the exact priority that was used to add the hook in the first place. Also in this case the theme’s hooked function is luckily not inside a class, so all I need is the function’s name.

My code then searches through the enqueued styles and injects some dependencies to make the bbp-default style load after the theme’s CSS.

add_action( 'wp_print_styles', 'style_dependency_fix', PHP_INT_MIN, 0 );

function style_dependency_fix() {
	remove_action( 'wp_print_styles', 'evolve_deregister_bbpress_styles', 15 );
	global $wp_styles;
	foreach ( $wp_styles->queue as $style ) :
		if ( 'bbp-default' === $wp_styles->registered[ $style ]->handle ) {
			$wp_styles->registered[ $style ]->deps = array_merge( $wp_styles->registered[ $style ]->deps, array( 'evolve-style', 'evolve-bootstrap' ) );
			break;
		}
	endforeach;
}

• This reply was modified 6 years, 1 month ago by Bart Kuijper. Reason: typo

Thank you very much for your reply, your code, your effort, and your time!

I couldn’t get bbp-default to register, not even when giving my wp_enqueue_scripts hook a priority of PHP_INT_MAX to make sure it goes last.

So then I had another look at when my theme deregisters bbp-default. Turns out the evolve theme deregisters the style using the wp_print_styles hook that fires later than wp_enqueue_scripts. Note: since WP 3.3 the wp_print_styles hook isn’t supposed to be used for enqueueing scripts and styles for the frontend. (source)

Since I’m not using a child theme currently, I decided to create a small Must-Use plugin that fixes the issue by registering the bbp-default style with its source set to false. This way the bbp-default style doesn’t actually get loaded and so doesn’t interfere with my theme, but it does satisfy this plugin’s bsp style dependency.

There’s of course some fluff in the mu-plugin (file header, comments) which I won’t all paste here. The actual magic happens in just these few lines of code anyway. Again I’m using a priority of PHP_INT_MAX, since the evolve theme uses this same hook to deregister the style and I want to make sure my hook fires last.

add_action( 'wp_print_styles', 'fix_bbp_dependency', PHP_INT_MAX, 0 );
function fix_bbp_dependency() {
	wp_register_style( 'bbp-default', false );
}

• This reply was modified 6 years, 1 month ago by Bart Kuijper. Reason: typo's & marked as resolved
• This reply was modified 6 years, 1 month ago by Bart Kuijper. Reason: corrected an ambiguous sentence

That makes perfect sense, I feel like I owe you an apology for even considering that you had broken backwards compatibility.

Thank you very much for your fast answer and thank you even more for the continued development of this excellent plugin!

You are most welcome! This plugin is essential to my website, it’s free and it’s never let me down. The least I could do was leave a review. Thank you and I wish you and yours a great 2020!