Hi Aaron,

The module is outputting a simplified flv from Blip.tv and not leveraging their advertising or the rest of the options offered by their player. I wasn't sure if this was a limitation of Blip's API or not, but I got some sample code from them and then spent some time tweaking things. I think I've got things purring nicely now!

This patch does 2 things:

1) It collects the proper 'post_id' value and stores it in the $data array collected via the Blip API. Migrating users will need to edit and re-submit existing nodes in order to update this value.

2) It replaces the theme_video_cck_bliptv_flash() function with the latest/greatest player from Blip. This player allows advertising, full screen, and all of the wiz-bang stuff that Blip offers.

Enjoy!

CommentFileSizeAuthor
blip-proper-video.patch3.11 KBjjeff

Comments

aaron’s picture

aaron commented
Assigned: Unassigned » aaron

thanks, i'll review & roll this in.

aaron’s picture

aaron commented

bump. not sure if this was put in or not :(

alex ua’s picture

alex ua commented
Status: Needs review » Needs work

I tried this, but $item['data']['post_id'] doesn't seem to return the correct value (it's returning the entire url instead of just the post id).

Also, we're moving all of these files (as much as is possible) away from the deprecated "embed" tag.

bflora’s picture

bflora commented

YOU ARE A GENIUS. Oh man, this works like a dream.

alex ua’s picture

alex ua commented
Status: Needs work » Fixed

The latest Dev version is now using the new Blip player, so this should be addressed.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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