Download & Extend

Doesn't properly check if jplayer library is installed

Project:jPlayer
Version:7.x-2.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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.

AttachmentSize
jplayer_requirements.patch364 bytes

Comments

#1

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.

AttachmentSize
jplayer_js_aggregation.patch 1.04 KB

#2

Status:needs review» fixed

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

#3

Status:fixed» closed (fixed)

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

nobody click here