I am wondering if that is the right way to use wp_enqueue_script() function in php. Do I need to place wp_enqueue_script() somewhere else ?
Not really, here’s what the official docs say in the notes:
https://developer.wordpress.org/reference/functions/wp_enqueue_script/
The function should be called using the wp_enqueue_scripts action hook if you want to call it on the front-end of the site, like in the examples above. To call it on the administration screens, use the admin_enqueue_scripts action hook. For the login screen, use the login_enqueue_scripts action hook. Calling it outside of an action hook can lead to problems, see the ticket #11526 for details.
The way you’re doing it might work as it will try to print scripts out in the footer, assuming your theme is built correctly and makes all the right calls, e.g. wp_head, wp_footer, etc
But further than that, the approach is wrong.
Instead of:
- When Brightcove player HTML is printed
- enqueue the script
- Hide a logo using a
hscLogofunction in the script that’s called by doing ????
The code should:
- Always enqueue the script
- on the DOM ready, check for a brightcove player
- if a brightcove player is found, remove the logo
Or better yet, instead of using javascript, put this CSS in your stylesheet:
.hsc-tv-logo {
display: none;
}