Some bliptv videos have non-alphanumeric chars in their video_id. This one contains the "+" character. http://blip.tv/indy-tv/divest-from-chase-bank-6094146.
The issue comes when media_bliptv creates the id attribute for the wrapper div. I don't believe that "+" is a legal character for ids (http://stackoverflow.com/questions/70579/what-are-valid-values-for-the-i...) and furthormore jquery doesn't work properly with them in selectors. This means the iframe never gets inserted and the video doesn't work.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | wrapper_id.patch | 866 bytes | bdk |
Comments
Comment #1
bdk commentedI'm attaching a patch that fixes the issue for me. It just removes the video_id part from the wrapper div. I think just the id variable should be enough to ensure uniqueness.
Another way to do it would be to strip out the non-alphanumeric chars from video_id before using it for wrapper_id, but I think this way is cleaner.