Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Something in styles.module
changed causing problems. Patch coming.
Comment | File | Size | Author |
---|---|---|---|
#1 | undefined_property-1203252-1.patch | 1.02 KB | langworthy |
Comments
Comment #1
langworthy CreditAttribution: langworthy commentedThe second change fixed the problem for me. Assuming the first change should be made for consistency.
And this happened to me when using
media.module
HEAD,media_youtube.module
HEAD andfield_styles.module
7.x-2.0-alpha8Patch formatted for
drush_make
Comment #2
dddave CreditAttribution: dddave commentedApplied by hand and it works. Thanks!
Comment #3
Carlos Miranda Levy CreditAttribution: Carlos Miranda Levy commentedI confirm this works and fixes the problems created by latest versions of media, entity, styles, and all the patches applied to fix them.
Comment #4
aaron CreditAttribution: aaron commentedthanks. that didn't work for wysiwyg; applied a fix for both.
Comment #5
cajinx CreditAttribution: cajinx commentedwas wondering if you could please explain where to place the patch code, I need to insert by hand (not doing drush). Any explanation or directions would be greatly appreciated. ty.
Comment #6
dddave CreditAttribution: dddave commented@#5
Just wait for the next dev and download. aaron has committed the fix so there is no need to do it yourself.
Generally to apply a patch by hand you have to check which files to modify (usually in the top line of the patch but look out for further files mentioned within the patch). After that locate the line in the file which should be modified (the one with the - in front of it) and replace it with the line with the +. This is feasible for patches modifying only one line here and there (like this one) but in general not that fun.
Comment #8
BeaPower CreditAttribution: BeaPower commentedworked, thanks
Comment #9
cgonsalv CreditAttribution: cgonsalv commented•Notice: Undefined property: stdClass::$file in media_youtube_file_styles_filter() (line 65 of /home/badmin6/public_html/pollography.com/p002/sites/all/modules/media_youtube/includes/media_youtube.styles.inc).
•Notice: Trying to get property of non-object in media_youtube_file_styles_filter() (line 65 of /home/badmin6/public_html/pollography.com/p002/sites/all/modules/media_youtube/includes/media_youtube.styles.inc).
I get the above error messages even after applying the patches below
http://drupal.org/node/1186624
and
http://drupal.org/node/1203252
The error is occuring on the node that has a multimedia field which includes a youtube video.
However in the teaser view of the node a screen capture of the youtube video is shown which is the expected behaviour.
Thanks
Comment #10
jkaine CreditAttribution: jkaine commentedHowdy,
I had this same problem, but came to the discussion late. It turns out that a patch to the Styles module (per this reference: http://drupal.org/node/1186624 ) fixes this problem for Media: Youtube, plus others.
I'm running D7.8 + Media 7.x-1.0-beta5 + Media: Youtube 7.x-1.0-alpha5
Upgrading to Styles 7.x.2.x-dev (September 12) got rid of all these errors, and that was WITHOUT applying the patch from #1.
I haven't tested to see if the patch from #1 conflicts with the change to Styles-- I kinda doubt it-- but it might be a bit of cruft if they've actually fixed the problem with Styles.
Best,
JK
PS: Media: Youtube has saved me quite a bit of time. Thanks for the module.
Comment #11
damien_vancouver CreditAttribution: damien_vancouver commentedI also had this problem, whenever I tried to display a youtube media field in any kind of style. I was running styles-7.x-2.0-alpha8.
As per #10, I upgraded styles to 7.x-2.x-dev 2011-Sept-12, and the error went away without any patching.
Hopefully a new recommended release comes out soon for styles, which would stop people encountering this problem.