Closed (fixed)
Project:
Media: YouTube
Version:
6.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Jul 2012 at 06:50 UTC
Updated:
7 Sep 2013 at 20:01 UTC
Jump to comment: Most recent file
Comments
Comment #1
traviscarden commentedLooks like this is fixed in commit 2dfaf7e. You can get it as a patch file. Note: It only seems to fix the feature for the HTML5 player. (Since that's what I use, I didn't take the time to find out if the Flash player is fixed in another commit.)
Comment #2
imclean commentedThe URL for the Flash player wasn't constructed correctly. There should be a question mark rather than ampersand before the first parameter.
Patch is against the latest git.
Comment #3
imclean commentedcheck_plain() on the URL was preventing the other options from functioning (autohide etc.).
I'm not sure it's required on the URL as a whole as most elements seem to go through check_plain individually.
Comment #4
daveX99 commentedThanks for this patch. It looks like its working.
What's weird is that my client only asked about fixing this today (6/5/2013), and you only posted the patch yesterday. If he'd asked any earlier, I wouldn't have found it probably.
Is there a way to urge that this be rolled into the next update?
-dave.
Comment #5
shawngo commentedThe patch in #3 works for me although I ended up applying it manually against 6.x-1.3.
Comment #6
aaron commentedCommitted to http://drupalcode.org/project/media_youtube.git/commit/3bac33f