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.

CommentFileSizeAuthor
#1 jplayer_js_aggregation.patch1.04 KBberenddeboer
jplayer_requirements.patch364 bytesberenddeboer

Comments

berenddeboer’s picture

berenddeboer commented
StatusFileSize
new jplayer_js_aggregation.patch1.04 KB

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

deviantintegral commented
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.