Closed (duplicate)
Project:
Media: YouTube
Version:
7.x-1.0-alpha4
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 May 2011 at 00:57 UTC
Updated:
17 Jun 2011 at 08:13 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
dema502 commentedHi Nigel
I think is it because Styles Module change API.
Comment #2
Carlos Miranda Levy commentedI confirm this patch fixes the issue for me.
Applied to alph4 version.
Just download patch to your sites/all/modules/media_youtube/includes directory and apply from there...
Comment #3
mattash commentedAfter applying this patch, Youtube embeds were properly rendered, but preview thumbnails now no longer work. The
imgtag is rendered withsrc points to the URL of the Youtube video.Comment #4
r0bm1lls commentedConfirm embedded video work now but no thumbnail and when editing using wysiwyg the image does not appear. Not through the woods yet... Thanks for interim patch though.
Comment #5
dema502 commentedPatch with fix thumbnail
Comment #6
dema502 commentedAnd remember to clear cache after patching!
Comment #7
mattash commentedI had a few other problems with the media_youtube.styles.inc file. Here is my modified patch. Note: This includes the changes from the patch in comment #1 and #5.
Comment #8
Anonymous (not verified) commentedpatch in #7 solves my problems, thanks !
Comment #9
Carlos Miranda Levy commentedPatch #7 fixes all.
Comment #10
tomfilepp commentedIt looks like if you just change all instances of lowercase 'youtube' to 'vimeo' and all capitalized instances of YouTube to Vimeo in the patch file, it also fixes this issue for the Vimeo style module.
Comment #11
aaron commentedfixed at http://drupalcode.org/project/media_youtube.git/commit/7358b9e
Comment #12
Carlos Miranda Levy commentedPardon my ignorance... but what do I do with http://drupalcode.org/project/media_youtube.git/commit/7358b9e ?
Should I download the patch link as a file and apply it as a patch?
Not too familiar with the git repository for drupal, sorry for asking.
Comment #13
aaron commentedah, no, sorry. you don't need to patch, you can update to alpha-5, as it's applied there. though you can download the patch and apply it from http://drupalcode.org/project/media_youtube.git/patch/7358b9e?hp=2e45bc3... if you really want.
Comment #14
Carlos Miranda Levy commentedThanks for quick reply. Is this patch also applied to the 7.x-1.x-dev version? I'm now battling other issues (styles related) and would like to know if eventually it would be "safe" to test the dev when dealing with further errors and not lose the applied patch.
Comment #15
cpelham commentedI have latest dev (June 1 I think) and I am getting an almost identical error (line number is different):
Fatal error: Call to undefined method MediaYouTubeStyles::getUri() in /home/cpelham/public_html/dev.crsny.org/public/sites/all/modules/media_youtube/includes/MediaYouTubeStyles.inc on line 37
I get the error when I try to edit any file style.