This issue is a followup to #2066275: file_field_load() overwites any field item properties with file entity properties to add tests for the bug fixed there.

CommentFileSizeAuthor
#2 2933564-2-WITH-ROLLBACK.patch4.42 KBDavid_Rothstein
#2 2933564-2.patch3.92 KBDavid_Rothstein
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

David_Rothstein created an issue. See original summary.

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein as a volunteer commented
Status: Active » Needs review
FileSize
2933564-2.patch3.92 KB
2933564-2-WITH-ROLLBACK.patch4.42 KB

Here is a patch, plus a patch with a rollback of the bugfix to show that the tests work.

Status: Needs review » Needs work

The last submitted patch, 2: 2933564-2-WITH-ROLLBACK.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein as a volunteer commented
Status: Needs work » Needs review

That was the expected failure (on the rollback patch only).