Question 347072 in PHP Question posted in
Official manual can be found here.

Can't delete images from folder with PHP using Unlink function

I am trying to execute a php process file, in a folder where there’s this php file and other files (can be images, text document, pdf, other code files, etc.)

I need to generate a process where detects main image formats and deletes it automatically.

I have this code:

$dirPath = '.';
$files = scandir($dirPath);  
foreach ($files as $file) {
    $filePath = $dirPath . '/' . $file;
    if (is_file($filePath)) {
        //echo $file . "<br>";
        $file_parts = explode(".", $file);
        $file_extension = $file_parts[1];
        
        $image_extensions = array( "jpg", "png", "jpeg", "tiff", "webm", "jpeg", "gif" );
        
        echo "<br />Found this file: ".$file;
        foreach($image_extensions as $i_e){
            if (strcmp($file_extension, strval($i_e)) === 0) {
                chmod ($file, 0777);
                unlink($file);
                echo 'Deleted for being an image.';
            }
        }

    }
}

But never deletes it. Files have a permissions level of 0644.
Is this a problem? Can I change permissions?
Could be another reason.

All files are in the same folder.
Why files isn’t being deleted?

Thanks in advance.

I was trying to delete all images in a folder and isn’t working.

Tags:

2

Answers


  1. 0

    Having files with permissions set to 644 should not be a problem since the owner has both read and write permissions in PHP. However, check if the file is owned by the user under which your PHP script is running.

    Here are some adjustments in your code that might help!

    $dirPath = '.';

$files = scandir($dirPath);

$image_extensions = array("jpg", "png", "jpeg", "tiff", "webm", "jpeg", "gif");

foreach ($files as $file) {

    $filePath = $dirPath . '/' . $file;

    if (is_file($filePath)) {

        $file_parts = explode(".", $file);

        //The function strtolower() ensures case-insensitive comparison of file extensions
        //The function end() moves the internal pointer to the last element in the array
        $file_extension = strtolower(end($file_parts));
        
        if (in_array($file_extension, $image_extensions)) {
            echo "<br />Found this image file: " . $file;
            if (unlink($filePath)) {
                echo '$file has been deleted';
            } else {
                echo '$file has not been deleted';
            }
        }
    }
}
    Login or Signup to reply.
  2. 0

    Privileges of 644 means that the owner user has read and write permissions, but no execute, whereas group users and others have read permissions only.

    Now, in order to remove a file, you need X (execute) to cd into the file and W (write) to remove them. Hence, the user needs to have either 7 or 5 as a privilege in order to be able to delete the file, see https://www.quora.com/What-exact-permission-is-required-to-delete-a-file-and-directory-in-Linux-What-exact-permission-is-required-to-copy-or-move-a-file-dir-in-Linux

    So the permissions by themselves prevent your program from successfully running, which is the technical reason of your problem.

    But the real reason is that you either did not set PHP to properly log the errors. You will need to

    • set the folder permissions so that the user with whom you want to issue the delete command has write and execute permission on the folder
    • look at the files you see in $files and whether your code actually tries to remove the files you wanted to remove
    • which files passed the if (strcmp($file_extension, strval($i_e)) === 0) { and whether the files you intend to remove successfully pass this condition
    • what does unlink return. Is it returning true, claiming the file was removed, or is it false?
    • what errors if any do you find in the logs related to this script

    Looking into these questions you should be able to solve your problem.

    Login or Signup to reply.
Please signup or login to give your own answer.

Back To Top
Search