I have an hopefully easy to fix issue. In upload.php, is_writable($current_file) should be is_writable(dirname($current_file)). unlink requires write access on the directory and doesn't even look at the file permissions.
A few sites have complicated permissions where the web-server doesn't have write access to the attachment files, but can remove them and add new attachments.
mansj
commented
7 years ago
Hello and thank you for Enable Media Replace,
I have an hopefully easy to fix issue. In upload.php, is_writable($current_file) should be is_writable(dirname($current_file)). unlink requires write access on the directory and doesn't even look at the file permissions.
A few sites have complicated permissions where the web-server doesn't have write access to the attachment files, but can remove them and add new attachments.
Thanks!