Reviewed & tested by the community
Project:
Rotating Banner
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
17 May 2011 at 23:42 UTC
Updated:
13 Aug 2012 at 06:14 UTC
Jump to comment: Most recent file
Comments
Comment #1
ksenzeeThis patch should make rotating banner work with media both before and after the file_entity changes.
Comment #2
james.elliott commentedComment #3
kscott22 commentedWorked for me. Thanks!
Comment #4
gratefulsk commentedPatch worked. Thank you!
Comment #5
mariano.barcia commentedI'm experiencing a similar issue, and I checked my code, and turns out this issue is already committed in HEAD, so marking it as fixed for now.
Comment #7
adamdicarlo commentedI'm not seeing any commits newer than 8 months ago. Did you push the fix to Drupal.org?
Comment #8
effulgentsia commentedBack to RTBC as in #2. The #1 patch is good, but has not yet been committed.
Also, cross-linking #1145092: Notice: Undefined index: #file in theme_media_thumbnail() (line 288 of .../sites/all/modules/media/includes/media.theme.inc) which is mostly a duplicate, but contains additional code.
Comment #9
katbailey commentedI went ahead and marked the other issue as a duplicate as I'm fairly confident the additional code mentioned above was because it was not the correct fix, whereas the fix here works perfectly.
Comment #10
jjgar commentedHow do I apply this patch? I was trying to find out if there was a file with the same name that needed to be replaced but seems that the code in this file needs to be added to another file. Any insight on the process to follow will be appreciated.
Thanks
Comment #11
Jackinloadup commented#10 jjgar check out Drupal's documentation on Applying patches. Good luck.