Support from Acquia helps fund testing for Drupal Acquia logo

Comments

skwashd’s picture

Status: Active » Needs review
FileSize
868 bytes

Here is the patch.

Status: Needs review » Needs work

The last submitted patch, entity-1741028-file-creation-callback.patch, failed testing.

lucascaro’s picture

As an alternative, the patch from #1778484: UUID Services for file should not use the update method. for uuid adds support for creating files.

fmizzell’s picture

Issue summary: View changes

This patch works, at least in conjunction with media and file entity. The failing test has something to do with file deletion (did not look at it closely), but manually deleting a file through the media dashboard works fine.

joseph.olstad’s picture

Status: Needs work » Closed (outdated)

This was already committed to the entity.module

7.x-1.6 already has this patch applied.

joseph.olstad’s picture

Thanks ! I'm going to do this.

dasha_v’s picture

Status: Closed (outdated) » Needs work

No, it is not included in 1.6. See http://cgit.drupalcode.org/entity/tree/entity.module?h=7.x-1.6#n1505
Only nodes and users have creation callback. File entity is missing it.

joseph.olstad’s picture

Oops I was pulling long hours, might have mixed up my issues. Follow up asap

Status: Needs review » Needs work

The last submitted patch, 9: entity-1741028-1-file-creation-callback.patch, failed testing.

The last submitted patch, 9: entity-1741028-1-file-creation-callback.patch, failed testing.