Hi!

When Views is showing an image field but there's no image set for that node, the above warning is generated.

The fix is to have file_view_multiple() verify that it was actually passed files (which would also be proper code design).
On line 80 of file_entity.file_api.inc add the following:

  if (empty($files)) {
    return $build;
  }

Sorry, don't have time to generate a patchfile for this. I'm busy tracking down a views-generated SegFault (11).

CommentFileSizeAuthor
#3 file_entity-warning_uasort_fix_4_file_view_multiple-2643122-3.patch622 bytesaDarkling
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aDarkling created an issue. See original summary.

aDarkling’s picture

aDarkling CreditAttribution: aDarkling as a volunteer commented
Issue summary: View changes
aDarkling’s picture

aDarkling CreditAttribution: aDarkling as a volunteer commented
FileSize
file_entity-warning_uasort_fix_4_file_view_multiple-2643122-3.patch622 bytes

Huh. I already had git pulled & ready to go on this module. Here y'go!

froboy’s picture

froboy
he/him
Primary language English
Location Chicago, IL
CreditAttribution: froboy commented
Status: Active » Reviewed & tested by the community

Ran into this issue using entity_embed. Patch applies successfully and fixes the problem. I'll call it RTBC.

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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