enqueue bp_core_confirmation_js

[bassjobsen]

./buddypress/bp-core/bp-core-cssjs.php sets add_action( 'wp_head', bp_core_confirmation_js', 100 ); where bp_core_confirmation_js add javascript to the head which depends on jquery. This code should be enqueued with wp_enqueue_script() to set the jquery dependency. Localization can be add with: wp_localize_script().

Thanks for your attention. I will need this fixes to get WP Defer Loading work with BuddyPress, see: ​https://github.com/bassjobsen/wp-defer-loading/issues/4

[boonebgorges]

I'm not totally sure why we do it this way (rather than proper enqueuing). I guess the idea is that there are places where we would want the .confirm JS but we don't want to load our entire script library. That makes sense, but in that case why not just have a separate .js file for confirm links? It'll get cached pretty heavily, so I can't imagine a huge performance issue.

Would like an opinion from another dev before moving forward with it.

[bassjobsen]

Thanks. Localization has been the reason maybe. wp_localize_script() will solve this

[boonebgorges]

In 8091:

Move .confirm javascript to a separate, properly enqueued file

This improves our compliance with WP standards, improves our integration with
caching systems, and allows plugins to manage the conditional loading of
scripts.

Fixes #5209