Embedding a YouTube video such as…
<object width="480" height="385"><param name="movie" value="http://www.youtube.com/v/OQSNhk5ICTI?fs=1&amp;hl=en_US"></param><param name="allowFullScreen" value="true"></param><param name="allowscriptaccess" value="always"></param><embed src="http://www.youtube.com/v/OQSNhk5ICTI?fs=1&amp;hl=en_US" type="application/x-shockwave-flash" allowscriptaccess="always" allowfullscreen="true" width="480" height="385"></embed></object>

… results in JW Player not playing the video due to the ?fs=1 not being stripped from the URL. Standard YouTube player not affected.

The attached patch fixes this, but not sure if emvideo_youtube_extract() is the right place to do it.

CommentFileSizeAuthor
emfield-youtube_embed.patch556 bytesximo

Comments

joestewart’s picture

joestewart commented

patch worked great. thanks.

#881646: youtube thumbnail doesn't work anymore a duplicate?

mckeen_greg’s picture

mckeen_greg commented

This patch works. Thank you.

whimsy’s picture

whimsy commented

Patch works. Thank you!

inky@inky3d.com’s picture

inky@inky3d.com commented

Worked for me too, thanks!

joestewart’s picture

joestewart commented
Status: Needs review » Reviewed & tested by the community

I guess OK to mark RTBC? maintainer?

rokit88’s picture

rokit88 commented

I had a different problem, but this also solved it. Looks like the ?fs=1 is being saved as part of the video id/value for youtube (and it shouldn't be). Thanks!

horrorview’s picture

horrorview commented

Hey guys, where do I put this patch? Having the same problem. Which file in Emfield do I alter? :)

chezwel’s picture

chezwel commented

Patched worked for me with 6.x-1.x-dev. Installed patch in sites\all\modules\media_youtube\providers\emvideo\youtube.inc

Anonymous’s picture

Anonymous (not verified) commented

Updated to 6.x-1.24 then applied patch. Fixed thumbnails for YouTube.

toemaz’s picture

toemaz commented

Successfully applied

alex ua’s picture

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

Thanks for the patch! I committed this to emfield 6.x-1.x-dev. I'm assigning to Media: YouTube so that it gets ported upstream.

bartidris’s picture

bartidris commented

The patch fixed a related problem: If I paste an entire embed code in the emfield and save the node, the generated thumbnail URL end up becoming something like http://img.youtube.com/vi/mxMDFWUjBrk?fs=1/0.jpg instead of the correct http://img.youtube.com/vi/mxMDFWUjBrk/0.jpg.

fabsor’s picture

fabsor commented

This seems to be fixed in the new version, since applying the patch gives you the option to revert the changes. If so, this issue should be set to fixed.

aaron’s picture

aaron commented
Status: Reviewed & tested by the community » Fixed

got this committed. thanks!

Status: Fixed » Closed (fixed)

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