http://flowplayer.org/documentation/version-history.html

CommentFileSizeAuthor
#2 flowplayer320.patch652 bytesjbrown

Comments

jbrown’s picture

jbrown commented

http://drupal.org/cvs?commit=365288

The SWFs and js have been updated to the latest versions from flowplayer.org.

jbrown’s picture

jbrown commented
Status: Active » Needs review
StatusFileSize
new flowplayer320.patch652 bytes

Minor javascript fixup.

msmollin’s picture

msmollin commented

Has this patch been merged into the CVS?
-EDIT-
And can it be merged into the 5.x dev?

rickmanelius’s picture

rickmanelius commented

subscribing

dellis’s picture

dellis commented

subscribing...

WoozyDuck’s picture

WoozyDuck commented

Would fixing the JS file fix the problem?

yan’s picture

yan commented

Seems to work with Flowplayer 3.2.5 without the javascript patch.

yan’s picture

yan commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

ñull’s picture

ñull commented
Category: task » bug
Status: Closed (fixed) » Needs work

The patch was never merged. I applied the patch and still get an error:

300: Player initialization failed: ReferenceError: Error #1069

mustanggb’s picture

mustanggb commented
Title: Upgrade to Flowplayer 3.2.0 » Upgrade to Flowplayer 3.2.x

Dropping the 3.2.7 files (flowplayer.swf and flowplayer.controls.swf) into 7.x-1.0-alpha1 results in a black box

rickmanelius’s picture

rickmanelius commented

Even applying said patch doesn't help

rickmanelius’s picture

rickmanelius commented
Status: Needs work » Reviewed & tested by the community

I can now confirm that this patch does indeed work. It was a browser caching issue and when I reset it and drupal, it started working again with flowplayer 3.2.7.

BTW, the patch no longer applies within the flowplayer.admin.js file. Instead, you have to replace that line in the flowplayer.js file.

Now we just need to ping Rob Loach to get this applied and we're good to go.

rickmanelius’s picture

rickmanelius commented

Any chance we can get this committed? Acquia dev cloud is containing this module is out of sync with the official release :)

arisaves’s picture

arisaves commented

Would be awesome to outfit the above patch for 7.x

arisaves’s picture

arisaves commented
Version: 6.x-1.x-dev » 7.x-1.0-alpha1

Oops. Herp-derp. This totally works for D7 as is, manually replacing the line in the js file and swapping out the swf files.

If anybody else needs specific instructions,

- Follow the patch instructions, patching flowplayer.js instead of flowplayer.admin.

- Download the latest flowplayer.swf and flowplayer.controls.swf files and swap them with the ones contained in the module.

Thank you, jbrown, for the patch and rickmanelius for confirming.

g_miric’s picture

g_miric commented

This doesn't work for latest flowplayer (3.2.16)

kreynen’s picture

kreynen commented

FlowPlayer has now jumped from version 3.2.17 -> 5.4.6 (http://flowplayer.org/news/). I wasn't able to get RTMP working from a Wowza EC2 instance with either and had to resort to a custom JWPlayer implementation since RTMP doesn't work w/ https://drupal.org/project/jw_player either.

#1402424: Streaming rtmp

While I'd prefer to use an open solution, I'm also aware that Flowplayer is GPLv3 and should be in Drupal.org's git repo.

#722554: flowplayer module includes Flash player protected with GPL 3 licence

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Issue summary: View changes
Related issues: +#1805580: New Flowplayer 5.0 HTML5 Player

This should be deprecated in favor of #1805580: New Flowplayer 5.0 HTML5 Player.