Comments

ghosts’s picture

I do also. I have not been able to figure out what is causing it.

I would not call myself a JS expert but I looked at the JS code in the module. The same function is called when the next button is clicked as is called when a song ends.

Drupal.jplayer.next(wrapper, player) which calls
Drupal.jplayer.setFiles(wrapper, player, index, play) which finally calls
$(player).jPlayer('play');

The song will automatically play when the next button is clicked, but not when the song ends, although a trace (Firebug) "looks" like everything should work. Since the JS correctly "changes" the song for both the next button and the song end, I looked at the player to see if it was passed correctly. It *looked* ok too. (Apologies if I overlooked something--I am a PHP person!)

ghosts’s picture

Update: It's not confined to views.

I have the same issue when I create a content type with a multi-value audio file field. When repeat is set to "All" a song will not automatically play after the previous song ends, although the next song will be set as the current song.

ghosts’s picture

continuous play works fine in

  • Windows Safari 5.1.7
  • Android Browser

continuous play does not work in

  • Windows Chrome 27.0
  • Windows Firefox 21.0
  • Windows IE 9
silurius’s picture

I'm seeing this in 7.x-2.0-beta1+10-dev as well. Anyone interested in picking this issue up?

silurius’s picture

Title: jPlayer Views-Plugin-Style does not continuous play the playlist » jPlayer Continuous play feature does not work in most browsers
Issue summary: View changes

Renaming the issue because it doesn't seem to be related to Views. (I am using a content type with a multi-value audio file field and am not using it with Views.)

tekket’s picture

I found that jplayer.theme.inc in 7.x-2.0-beta1+10-dev version of jPlayer module is missing one line in preprocess function for jplayer.tpl.php when displaying a view as a playlist.

Missing line is: 'continuous' => $vars['settings']['continuous'],

So player settings array in template_preprocess_jplayer_view_playlist should be:

$player = array(
    'jplayerInstances' => array(
      $vars['player_id'] => array(
        'files' => $player['files'],
        'solution' => $vars['settings']['solution'],
        'supplied' => $player['extensions'],
        'preload' => $vars['settings']['preload'],
        'volume' => $vars['settings']['volume'] / 100,
        'muted' => (boolean)$vars['settings']['muted'],
        'autoplay' => (boolean)$vars['settings']['autoplay'],
        'repeat' => $vars['settings']['repeat'],
        'backgroundColor' => $vars['settings']['backgroundColor'],
        'continuous' => $vars['settings']['continuous'],
      ),
    ),
  );
silurius’s picture

Issue summary: View changes

Huh, my version of that same .inc (same beta version I think) was not missing that line.

tekket’s picture

silurius: Don't forget it is missing in function template_preprocess_jplayer_view_playlist(which is why continuous play didn't work in views), not in template_preprocess_jplayer.

tr33m4n’s picture

That worked for me, cheers bud

Yogesh Kushwaha’s picture

#6 works for me
Thanks

markie’s picture

Hmmm I wonder how that got removed. I have added it back and will be committing it to dev today.

@tekket++

  • 669a36d committed on 7.x-2.x
    git commit -m 'Issue #1898810 by tekket, patrickroma: jPlayer Continuous...
markie’s picture

Status: Active » Fixed

Added continuous back to theme. Pushed to dev.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.