In function _filefield_file_delete
should

  module_invoke_all('filefield', 'delete', $node, $field, $file);

be

  module_invoke_all('filefield', 'file_delete', $node, $field, $file);

instead?

Comments

jpetso’s picture

jpetso commented

Hm, not sure how dopry intended that one... the 'delete' hook is called in _filefield_file_delete() unconditionally after the 'file_delete' hook is called in filefield_file_update(), and when deleted from filefield_field($op='delete'), the 'file_delete' hook is not called at all. I probably should have a look at how this is being done in imagefield's new field_file.inc.

dopry’s picture

dopry commented

strike the module_invoke_all in the update function, and make the op in _filefield_file_delete file_delete... nice catch magnus.

.darrel.

jpetso’s picture

jpetso commented
Version: 5.x-2.3-rc3 » 6.x-3.x-dev
Status: Active » Fixed

kthxfixed

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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