On line 315 of bootstrap_carousel.module,

if (isset($slide['carousel_video'])) {
      drupal_add_js(variable_get('bootstrap_carousel_youtube_player_api', 'https://www.youtube.com/player_api'), 'external');
}

We should use empty function instead of isset like below.

if (!empty($slide['carousel_video'])) {
      drupal_add_js(variable_get('bootstrap_carousel_youtube_player_api', 'https://www.youtube.com/player_api'), 'external');
}
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gabrielmachadosantos’s picture

Version: 7.x-1.0 » 7.x-1.x-dev
Status: Active » Needs review
FileSize
540 bytes

It was also loading the JS for each slide. Tested and patched.

rhormens’s picture

Status: Needs review » Reviewed & tested by the community

+1 RTBC

gabrielmachadosantos’s picture

Assigned: Unassigned » gabrielmachadosantos
Status: Reviewed & tested by the community » Closed (fixed)

Patched and committed against 7.x-1.x-dev.