How to reproduce this bug:

  1. Find the current videojs library path, we call it path1, make sure we only have 1 videojs library in the site.
  2. Go to Videojs config page (admin/config/media/videojs).
  3. Change 'Video.js location' to 'specificed path', and filling path1 to 'Video.js file directory', hit 'Save Configuration'
  4. Variable videojs_directory is set to path1 as expected.
  5. Move videojs library to another place, we call it path2
  6. Change 'Video.js location' to 'Libraries API', hit 'Save Configuration'
  7. Variable videojs_directory is set to path1, that's not what we want.
CommentFileSizeAuthor
#1 videojs_directory_is_not_updated-2192515-1.patch1.9 KBtien.xuan.vo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tien.xuan.vo’s picture

tien.xuan.vo CreditAttribution: tien.xuan.vo commented
FileSize
videojs_directory_is_not_updated-2192515-1.patch1.9 KB

Add quick fix.

tien.xuan.vo’s picture

tien.xuan.vo CreditAttribution: tien.xuan.vo commented
Status: Needs work » Needs review

Update status.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Gaggle.net, Inc. commented
Assigned: tien.xuan.vo » Unassigned
Status: Needs review » Reviewed & tested by the community

This fixes Libraries integration.

Jorrit’s picture

Jorrit CreditAttribution: Jorrit as a volunteer commented
Status: Reviewed & tested by the community » Fixed

In change http://drupalcode.org/videojs/commit/?id=adef482 I make sure that videojs_get_version does not use videojs_directory when using the libraries API. The reason is that videojs_directory should only be used when videojs_location is 'path'. Can you test if the current -dev version works for you?

Status: Fixed » Closed (fixed)

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