Cookbook - File Uploads with Doctrine - removeUpload at PreRemove

[sricardo]

In the section "Using the id as the filename", it seems that the callback of the removeUpload() function should be at PreRemove instead of PostRemove, because we will not be able to retrieve the id.

[craue]

Indeed, $this->id is already null when removeUpload() is called for the PostRemove event and thus the generated path is invalid. But deleting the file on PreRemove doesn't seem to be a good solution for me because the file would have been deleted even if an error occurs while removing the entity from the database, right?

[craue]

One option could be to add a method for PreRemove temporarily saving the path (using the id still available) into a private variable which is then used to delete the file on PostRemove.

[weaverryan]

Hi guys!

@craue you're absolutely right with your concern - only in PostRemove can we safely delete the file. It's not pretty, but the only responsible way to work around it is as you suggested - to store the path in PreRemove and then use it in PostRemove. I've made this change.

Thanks!