After downloading JW Player and putting it inside libraries/ directory /admin/reports/status reports missing jwplayer.html5.js file. But such file doesn't exist in JW Player 7. It does exist in JW Player 6 but this player is no longer available for download from JW Player official webiste.

Issue is caused by incorrect check inside jw_player_requirements() function.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Peter Swietoslawski created an issue. See original summary.

Peter Swietoslawski’s picture

Version: 8.x-1.0-beta3 » 8.x-1.x-dev
Issue summary: View changes
Peter Swietoslawski’s picture

Issue summary: View changes
Berdir’s picture

Issue tags: +Novice

Confirmed. We have a setting for the version, so lets make this a bit more dynamic, always add the first two files, only add html5.js when we are on version 6.

bkhandruk’s picture

Added a check for JW Player version in implementation of hook_requirements.

Berdir’s picture

Status: Needs review » Needs work

Thanks!

I suppose that works too, I would have build an array variable and then looped over that. But this is fine. Lets add a comment that JW Player 7 doesn't have that file anymore.

bkhandruk’s picture

Added a comment.

  • Berdir committed 3d0e92e on 8.x-1.x authored by rocket.man
    Issue #2782953 by rocket.man: Incorrect requirements for jwplayer.html5....
Berdir’s picture

Status: Needs review » Fixed

Thanks, committed.

bkhandruk’s picture

Thanks, @Berdir.

Status: Fixed » Closed (fixed)

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