Notice: Trying to get property of non-object in file_entity_edit() (line 718 of /www/html/sites/all/modules/contrib/file_entity/file_entity.pages.inc).

https://mysite.com/media/1614/edit/ajax

CommentFileSizeAuthor
#4 file_entity-missing-user-2549487-4.patch665 bytesjoelstein
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

StephenRobinson’s picture

StephenRobinson CreditAttribution: StephenRobinson commented

removed

tasc’s picture

tasc
Primary language German
Location Berlin
CreditAttribution: tasc commented

Just had the same problem when migrating a lot of files.
In my case, the uid of the file was pointing to a non-existing user. After fixing it, no errors shown anymore.

StephenRobinson’s picture

StephenRobinson CreditAttribution: StephenRobinson commented
Issue summary: View changes
joelstein’s picture

joelstein CreditAttribution: joelstein at On Fire Media commented
Version: 7.x-2.0-beta2 » 7.x-2.x-dev
Status: Active » Needs review
FileSize
file_entity-missing-user-2549487-4.patch665 bytes

Here's a patch which attempts to load the user instead of simply assuming the user exists.

Spokje’s picture

Spokje
Primary language Dutch
CreditAttribution: Spokje at Annertech commented
Status: Needs review » Reviewed & tested by the community

Patch applies cleanly and solves the problem.
All 11 coding standards messages found by TestBot are not related to this patch.
RTBC for me.

joseph.olstad’s picture

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

fixed in dev

Status: Fixed » Closed (fixed)

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