Flowplayer Module

CommentFileSizeAuthor
#2 flowplayer_patch.txt2.17 KBjspayne

Comments

winston’s picture

winston commented
Version: » 6.x-1.x-dev
jspayne’s picture

jspayne commented
Status: Active » Needs review
StatusFileSize
new flowplayer_patch.txt2.17 KB

Here is a patch to support the Flowplayer module for audio & video. I took the code from the custom URL provider and tweaked it to work with archive.org.

I think it would be good to implement a configuration option so one can choose the player used (important when other modules are supported). It currently will always use the Flowplayer module if available.

jspayne’s picture

jspayne commented
Assigned: winston » Unassigned
winston’s picture

winston commented

Thanks will test and apply this weekend.

I'm comfortable without the configuration option. Probably better to work towards the new media module. And over-riding the theme function is also always possible for those who want to do something different.

winston’s picture

winston commented
Status: Needs review » Postponed

So I tested your patch - looks like it works fine. Thanks!

However, there will be a slight delay in applying it as I need to sort out an issue related to the base emfield so I can roll a proper release.

See http://drupal.org/node/741826

Once that is resolved I'll roll a release of this module that works with the recommended emfield and maybe another release that works along with the new 2.0 branch of emfield depending on how that shakes out.

winston’s picture

winston commented
Assigned: Unassigned » winston
Status: Postponed » Active

Now that I've put out a 2.x release that explicitly includes the video provider I'll get to this shortly.

asb’s picture

asb commented

Any updates on this issue?

Thank you!