Hi Aaron
I had trouble playing BlipTV hosted videos, even with the patch submitted by JJeff. It seems that when a video has just been uploaded to blip and not yet converted to Flash, the data sent by blip about this video is different than when it has been converted to Flash. So I add code to supportboth answers.

I removed BlipTV data caching mechanism, and used another flash player. I added support for .wmv playback.

Patrick

CommentFileSizeAuthor
#4 emfield-bliptv.inc-1.6.patch15.28 KBpfournier
bliptv.inc_.patch14.34 KBpfournier

Comments

dgtlmoon’s picture

dgtlmoon commented

Hmm i had trouble applying your patch for my version of bliptv

// $Id: bliptv.inc,v 1.6 2008/01/04 23:53:47 aaron Exp $

yours?

pfournier’s picture

pfournier commented

// $Id: bliptv.inc,v 1.4 2007/09/12 15:08:47 aaron Exp $

alex ua’s picture

alex ua commented
Status: Needs review » Needs work

This patch failed for me as well. Can you try to patch against the latest version of the module?

pfournier’s picture

pfournier commented
Status: Needs work » Needs review
StatusFileSize
new emfield-bliptv.inc-1.6.patch15.28 KB

Here is the patch for version 1.6 of the file.

aaron’s picture

aaron commented
Priority: Normal » Critical

we need to make sure blip.tv works for the release (if it's not currently). bump to look at this patch.

aaron’s picture

aaron commented
Priority: Critical » Normal
Status: Needs review » Needs work

I don't like the idea of not caching the data. The way this patch works right now, the blip.tv api will be called for every thumbnail. Some sites I work on, that could easily be 500k+ times per day. Not good.

On another note, there is a feature request to wget the thumbnail and store it, in which case we wouldn't need to call data for tn's. Maybe that would solve the problem.

Thanks for the start though. We'll just need to see how we can tweak that.

Aaron

pfournier’s picture

pfournier commented

I understand the point.

I removed caching because immediately after upload, BlipTV sends back a generic thumbnail. It takes a while before it computes the tn.

Maybe we could add an expiration date to emfield_request_xml, that we could pass to cache_set? That would solve the problem.

Patrick

mlncn’s picture

mlncn commented

Subscribing. This would actually be a 'starting' date-- the idea here would be to not cache for the first day or something, but once BlipTV has made the custom thumbnail, it can be cached?

benjamin, Agaric Design Collective

alex ua’s picture

alex ua commented
Project: Embedded Media Field » Media: BlipTV
Version: 5.x-1.x-dev » 6.x-1.x-dev
Component: Providers » Code

Not sure if this is still an issue, but if it is it should be addressed in Media: BlipTV

aaron’s picture

aaron commented

emfield has long since had a caching mechanism for providers to use. bumping this to remind me to look at it.

aaron’s picture

aaron commented

i moved the metadata fetching function to the main module code, so this will no longer apply. a bit tricky to test for, and i have more pressing things. patches are still welcome!