The check to see if the jplayer js files are installed in libraries/ fails, i.e., it always succeeds, even when not installed. The attached patch fixes that.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

berenddeboer’s picture

I made a few other changes also to fix #1133214: "Aggregate JavaScript files" in Performance breaks jPlayer, so I thought it best to provide a roll-up patch of all changes. This one also includes a link to jplayer settings to go to from within the module page.

deviantintegral’s picture

Status: Needs review » Fixed

I committed the fix for the library requirement in a6ad159. The other changes will be in separate issues.

Status: Fixed » Closed (fixed)

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