Let's take advantage of multi loading.

CommentFileSizeAuthor
#3 media-prepare_view-followup.patch561 byteseffulgentsia
media-prepare_view.patch2.8 KBeffulgentsia
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, media-prepare_view.patch, failed testing.

effulgentsia’s picture

effulgentsia CreditAttribution: effulgentsia commented
Status: Needs work » Fixed

The patch failed to apply, because I already committed it to CVS before the test ran.

effulgentsia’s picture

effulgentsia CreditAttribution: effulgentsia commented
FileSize
media-prepare_view-followup.patch561 bytes 
+++ media.module	11 Aug 2010 00:53:19 -0000
@@ -540,12 +540,32 @@ function media_load($fid) {
-  if ($files = entity_load('media', $fids)) {
+  if ($files = media_load_multiple('media', $fids)) {

Passing 'media' to media_load_multiple() was wrong. I committed this patch to fix it.

Status: Fixed » Closed (fixed)

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