I added a theme_video_filter_script() function and a new codec "Ooyala" so that I could use this [video:xxx] syntax to inject the proper <script src="url"></script> required to render Ooyala videos.

I would love for this patch to be included in the next release, but any feedback would be greatly appreciated!

Patch attached.

CommentFileSizeAuthor
#1 video_filter-62dev-2.patch1.9 KBjwhat
video_filter-62dev.patch2.82 KBjwhat

Comments

jwhat’s picture

jwhat commented
StatusFileSize
new video_filter-62dev-2.patch1.9 KB

Figured out how to embed Ooyala videos the SWF way as well, so I've made the code work for both JS and SWF links.

Also realized I didn't need the theme_video_filter_script() function that I had previously added, so I removed that. The only modified file now is video_filter.codecs.inc.

Updated patch attached.

blackdog’s picture

blackdog commented

Do you have a test video I can use?

jwhat’s picture

jwhat commented

The "sample_url" that I put in video_filter.codecs.inc should work, it's:

http://player.ooyala.com/player.swf?embedCode=d1Z29oMTqqKBoMP4m50UnEE5EQ...

You can also change that to the JS version to test that option by changing the URL to:

http://player.ooyala.com/player.js?embedCode=d1Z29oMTqqKBoMP4m50UnEE5EQb...

minnur’s picture

minnur commented
Issue summary: View changes
Status: Needs review » Closed (won't fix)

No activity since 2010. Closing this issue. Feel free to open a new one for either D7 or D8 versions.