The newest version of media module removes the media_load() function, which is going to make rotating banner whitescreen. Patch forthcoming.

CommentFileSizeAuthor
#1 1160786-1.file_entity.patch812 bytesksenzee
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ksenzee’s picture

ksenzee
she/her
Location Seattle area
CreditAttribution: ksenzee commented
Status: Active » Needs review
FileSize
1160786-1.file_entity.patch812 bytes

This patch should make rotating banner work with media both before and after the file_entity changes.

james.elliott’s picture

james.elliott CreditAttribution: james.elliott commented
Status: Needs review » Reviewed & tested by the community
kscott22’s picture

kscott22 CreditAttribution: kscott22 commented

Worked for me. Thanks!

gratefulsk’s picture

gratefulsk CreditAttribution: gratefulsk commented

Patch worked. Thank you!

mariano.barcia’s picture

mariano.barcia CreditAttribution: mariano.barcia commented
Status: Reviewed & tested by the community » Fixed

I'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.

Status: Fixed » Closed (fixed)

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

adamdicarlo’s picture

adamdicarlo CreditAttribution: adamdicarlo commented
Status: Closed (fixed) » Active

I'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.

I'm not seeing any commits newer than 8 months ago. Did you push the fix to Drupal.org?

effulgentsia’s picture

effulgentsia CreditAttribution: effulgentsia commented
Status: Active » Reviewed & tested by the community

Back 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.

katbailey’s picture

katbailey CreditAttribution: katbailey commented

I 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.

jjgar’s picture

jjgar CreditAttribution: jjgar commented

How 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

Jackinloadup’s picture

Jackinloadup CreditAttribution: Jackinloadup commented

#10 jjgar check out Drupal's documentation on Applying patches. Good luck.