Delete current file at edit about cakephp-upload HOT 2 CLOSED

Can you create a fork with this in it, as well as the relevant tests? Thanks!

from cakephp-upload.

A note on why this was implemented my way and not quite the same as you.

Lets say for record 5, I upload a file named bob.jpg. Sometime later, I update the record with a different file, also named bob.jpg.

In your case, code, and if deleteOnUpdate is set to true in mine, the file bob.jpg is added to the deletion cue, the new file is uploaded, and then in the afterSave, the old file, bob.jpg, is deleted right at the end. That means we essentially did not upload a file.

Rather than rework this to use a complicated set of file renames that might potentially fail, I would rather let the developer decide what the tradeoffs be of allowing such a thing to occur. In the future, appending/prepending some hash to the filename, or even allowing the files to be renamed on the fly, may allow for this particular issue to be mitigated. Until then, I decided that implementing this in such a way as to not delete old files by default was the best way.

Please reopen if this is not satisfactory to your work, or if you have a better solution.

from cakephp-upload.