Hey there,

if you transliterate files over the backend (admin/config/media/file-system/transliteration) you will get the following error message:

Fatal error: Unsupported operand types in /path/to/drupal/sites/all/modules/contrib/file_entity/file_entity.file.inc on line 103

I know that this error is coming from another module, but it mainly occurs, because the hook is used in a wrong manner.

If an update hook is called an "original" is always used in all of Drupals internals and should be provided here, too.
Since natural methods of doing so are not used (avoiding file_save()) it needs to be done manually. Also to follow the file_save() function's behavior "module_invoke_all('entity_update', $file, 'file');" should be called, too.

Anyways, I think the best way to this issue is to just call file_save().
The comment saying "Don't use file_save() as it modifies the timestamp." is not a real reason, because the file itself is modified. The date should be therefore update.

Please consider this change back to the way it should be.

func0der

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

func0der created an issue. See original summary.

func0der’s picture

Here comes da patch, ready or not, here comes da...how you like me now?

func0der’s picture

Status: Active » Needs review
ShaneOnABike’s picture

Status: Needs review » Reviewed & tested by the community

I can confirm that this patch works. I was also receiving the same error and I was getting teh WSOD

joep.hendrix’s picture

Status: Reviewed & tested by the community » Needs review
FileSize
613 bytes

Removed the paths in the patch file.

cassien’s picture

#5 works like a charm

Chris Matthews’s picture

Status: Needs review » Reviewed & tested by the community

The 2 year old patch in #5 applied cleanly to the latest 7.x-3.x-dev snapshot and works for me as well so changing the status to RTBC.

renatog’s picture

Thank you all for the contribution

  • RenatoG committed d0e015f on 7.x-3.x authored by func0der
    Issue #2602512 by func0der, joep.hendrix, Chris Matthews, ShaneOnABike,...
renatog’s picture

Status: Reviewed & tested by the community » Fixed

Committed to the dev branch.

Best

Status: Fixed » Closed (fixed)

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