For a sandbox module I am working on I needed to catch a couple of errors and hand off to file_save correctly. This patch attempts to do that.

CommentFileSizeAuthor
pdf_to_imagefield-file_entity-support-798128-0.patch3.02 KBqueenvictoria

Comments

dman’s picture

dman commented
Status: Active » Needs work

I had a look and tried it out with file_entity (by way of media.module)

I'm not seeing what this is trying to do here.
* If we were to do file_entity support (which I think would be bonza!) then it's not a case of adding a field field to a file entity - it already has one, and that's the one we want to extract the image from. This patch does not do that.
* The patch appears to refer to an attribute called $file->original - which I don't see in file_entity.module - at least not the one that comes with media.module (7.x-2.0-alpha3)
Could it be that the different projects are not as closely linked as I thought? https://www.drupal.org/project/file_entity

I'd need to clarify what variation of file_entity we were looking at here I think.

spanners’s picture

spanners commented

Following, Your Majesty.

digitgopher’s picture

digitgopher commented

Would be great if this module worked with the latest Media/file_entity!

hitchshock’s picture

hitchshock
Location Ukraine
commented
Status: Needs work » Closed (outdated)

We stopped supporting the D7 version, so the ticket will be closed, but thanks to everyone who was working on the ticket!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.