how to Add current file to "delete list" except file with mp3 and mp4 extension?

Question

i want to make delete-list but except mp3 and mp4 file

my code is

error Fatal error: Call to undefined function endsWith() in E:\xampp\htdocs\tes\index.php on line 39

    $files = new RecursiveIteratorIterator(
    new RecursiveDirectoryIterator($rootPath),
    RecursiveIteratorIterator::LEAVES_ONLY
);

foreach ($files as $name => $file)
{
    // Skip directories (they would be added automatically)
    if (!$file->isDir())
    {
        // Get real and relative path for current file
        $filePath = $file->getRealPath();
        $relativePath = substr($filePath, strlen($rootPath) + 1);

        // Add current file to archive
        $zip->addFile($filePath, $relativePath);
        $zip->close();

        // Add current file to "delete list"
        // delete it later cause ZipArchive create archive only after calling close function and ZipArchive lock files until archive created)
        if ($file->getFilename().endsWith(".mp3")) error at this code
        {
            $filesToDelete[] = $filePath;
        }
    }
}

anyone help me please

check the mime-type of the file, file-extensions (e.g. .mp3) are unreliable — Sepultura, Jun 08 '17 at 15:56
I notice you've specified the `php-5.3` tag. Please be aware that this php version is now very old and unsupported. You should be urgently considering upgrading to at least version 5.6 to avoid security issues. — Simba, Jun 09 '17 at 10:28

Sepultura · Answer 1 · 2017-06-11T15:58:31.583

1

Again, checking the file-extension is unreliable. You should check the mime-type of the file with: mime_content_type.

In your code, you can do something like this:

// Add current file to "delete list"
        // delete it later cause ZipArchive create archive only after calling close function and ZipArchive lock files until archive created)
        if (mime_content_type($filePath) == "audio/mp3"))
        {
            $filesToDelete[] = $filePath;
        }

But be carefull because there're a lot of different mime-types for mp3, as you can see here.

edit: If you wan't to check for multiple mime_types you can use in_array:

if(in_array(mime_content_type($filePath), ['audio/mp3','audio/mpeg']))

edited Jun 11 '17 at 15:58

answered Jun 09 '17 at 07:51

Sepultura

997
1
9
28

Even checking mime type can be unreliable and dangerous, but at least it's better than file extensions. – Simba Jun 09 '17 at 10:31
how to add two type?? mean audio/mp3 and video/mp4 or anymore – Si Aji Reza Jun 11 '17 at 15:20

score 0 · Answer 2 · answered Jun 08 '17 at 16:02

0

The is no endsWith function. You need to create your own. Check this example.

startsWith() and endsWith() functions in PHP

answered Jun 08 '17 at 16:02

Kirill Staruhin

137
6

Then what should i do? – Si Aji Reza Jun 08 '17 at 16:06
Check the link in my comment there is example, how to implement this functionality – Kirill Staruhin Jun 08 '17 at 16:08

how to Add current file to "delete list" except file with mp3 and mp4 extension?

2 Answers2