Closed (fixed)
Project:
Flowplayer API
Version:
6.x-1.0-beta1
Component:
Code
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
24 Nov 2010 at 21:06 UTC
Updated:
24 Feb 2011 at 19:03 UTC
I bought a commercial license for FlowPlayer to remove the logo/branding. I didn't realize the module uses a much older version. Any suggestions for upgrading the flowplayer? It seems that the files inside the flowplayer folder are much different..
Contents of old folder:
README.txt
flowplayer.captions.swf
flowplayer.min.js
flowplayer.swf
flowplayer.audio.swf
flowplayer.controls.swf
flowplayer.rtmp.swf
Contents of new folder:
LICENSE_COMMERCIAL.txt
flowplayer.commercial-3.2.5.swf
README.txt
flowplayer.controls-3.2.3.swf
example
Comments
Comment #1
bcobin commentedsubscribing
Comment #2
RavenHursT commentedReally? Nothing on this since Nov??
Comment #3
yan commentedI think you just need to copy the .swf files and rename them like
flowplayer.controls-3.2.3.swf --> flowplayer.controls.swf
Comment #4
RavenHursT commentedYan.. Have you tried this and verified that this is an answer for this module? If not, I very much doubt that this would be a viable solution since it's obvious that Flowplayer has made some drastic changes to their over-all code structure. I would highly suggest that the Flowplayer API be modified to utilize the Libraries API module to better layer the dependencies of this outside code-base. Would be great is Rob Loach, or one of the other maintainers of this project could chime in here.
Comment #5
yan commentedYes I have and it worked fine..
Comment #6
bkat commentedworks for me too
Comment #7
RavenHursT commentedI still think the external libraries should be housed somewhere outside of Drupal's code-base.
http://engineeredweb.com/blog/10/5/3-tips-using-external-libraries-drupal
I'm not sure, but having external libraries included in a Drupal module in CVS may even be in violation of Drupal's standards policy...
Comment #8
yan commentedI guess this is a feature request now.
Comment #9
yan commentedSorry, I just noticed that there already exists a feature request:
#1036954: Implement libraries
Comment #10
RavenHursT commentedYes... Thanks.