Почему PHP не даёт ошибки?

Question

[rensly]

код:

<?php
include "../../include/lib/connection.php";
require_once "../../include/lib/exploitPatch.php";

$log = "";


if ($_FILES && $_FILES['filename']['error'] == UPLOAD_ERR_OK) {
    if (isset($_POST['authorname'] ) && isset($_POST['songname'])) {
        $file_type = $_FILES['filename']['type'];
        $maxsize = 10485760;
        if ($_FILES['filename']['size'] >= $maxsize) {
            $log = "Max file size is 10mb";
        } else {
            $hash = "";

            $string = $_FILES['filename']['name']; // song_name.mp3
            $songname = str_replace('.mp3', '', $string); //song_name
            $song_name = str_replace('_', ' ', $songname); // song name
            $name = str_replace(' ', '%20', $string); // song%20name

            move_uploaded_file($_FILES['filename']['tmp_name'], "song/$songname.mp3");
            $size = round($_FILES['filename']['size'] / 1024 / 1024, 2);


            $song = "http://".$_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI']."song/";
            $cur = str_replace('upload.php', '', $song) . $name;

            $query = $db->prepare("INSERT INTO songs (name, authorID, authorName, size, download, hash)
            VALUES (:name, '9', :author, :size, :download, :hash)");
            $query->execute([':name' => $song_name, ':download' => $cur, ':author' => "", ':size' => $size, ':hash' => $hash]);
            $log = "Success! ID: <b>". $db->lastInsertId() ."</b>";
        }
    } else {
        $log = "Provide a song name and an author name.";
    }    
}
?>

в папку загружать файлы можно, права есть.
скрипт должен загружать файл в папку song, т.е:

storage/
├─ songs/
│  ├─ song/
│  │  ├─ загруженные файлы с помощью upload.php
│  ├─ upload.php

ошибок скрипт не даёт, они включены и в настройках php и в самом скрипте.
если поможет, вот хтмл форма:

<form class="form" method="post"  action="upload.php" enctype='multipart/form-data'>
    <input type='text' style="margin-top: 5%;" placeholder="Song Name" name='songname'><br>
    <input type='text' placeholder="Author" name='authorname'><br>
    <input type='file' name='filename' size='10' accept="audio/*"><br>
    <input class="button" type="submit" value="Upload">
</form>

ОС стоящая на моем ВДС: CentOS 7

Comments

[Сергей delphinpro]

А какие ошибки вы хотите увидеть?

Если ошибки загрузки файла, то ваш код просто не выполняется, если они есть
Сами же написали: if $_FILES['filename']['error'] == UPLOAD_ERR_OK

[Сергей delphinpro]

Допишите ветку else в этом условии, и проверяйте какие там ошибки возникли
https://www.php.net/manual/ru/features.file-upload...

Answer 1

[ThunderCat]

учимся базовой отладке (ну и элементарной логике):

var_dump($_FILES); exit(); //проверяем есть ли вообще файл, и что там с ним происходит
if ($_FILES && $_FILES['filename']['error'] == UPLOAD_ERR_OK) { //где else к этому ифу???
// по уму надо отдельно проверять $_FILES['filename']['error'], так как скорее всего лажа где-то в нем
// вангую что размер upload_max_filesize или max_post_size слишком мал
    if (isset($_POST['authorname'] ) && isset($_POST['songname'])) { //здесь больше подходит !empty, 
//и надо бы как-то проверить подходят ли поля под стандарты, длина, символы левые...
// дальше в принципе нет смысла пояснять, так как первый if скорее всего тупо не работает
        $file_type = $_FILES['filename']['type'];
        $maxsize = 10485760;
        if ($_FILES['filename']['size'] >= $maxsize) {
            $log = "Max file size is 10mb";
        } else {
            $hash = "";
            $string = $_FILES['filename']['name']; // song_name.mp3
            $songname = str_replace('.mp3', '', $string); //song_name
            $song_name = str_replace('_', ' ', $songname); // song name
            $name = str_replace(' ', '%20', $string); // song%20name
            move_uploaded_file($_FILES['filename']['tmp_name'], "song/$songname.mp3");
            $size = round($_FILES['filename']['size'] / 1024 / 1024, 2);
            $song = "http://".$_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI']."song/";
            $cur = str_replace('upload.php', '', $song) . $name;
            $query = $db->prepare("INSERT INTO songs (name, authorID, authorName, size, download, hash)
            VALUES (:name, '9', :author, :size, :download, :hash)");
            $query->execute([':name' => $song_name, ':download' => $cur, ':author' => "", ':size' => $size, ':hash' => $hash]);
            $log = "Success! ID: <b>". $db->lastInsertId() ."</b>";
        }
    } else {
        $log = "Provide a song name and an author name.";
    }    
}

Comments

[rensly]

все конечно хорошо да, но у меня хтмл и пхп в одном .php файле, если преписать else к первому условию у меня будет при просто загрузке он будет срабатывать

[rensly]

var_dump() выводит - array(1) { ["filename"]=> array(5) { ["name"]=> string(19) "EL$E - NEAR2DIE.mp3" ["type"]=> string(0) "" ["tmp_name"]=> string(0) "" ["error"]=> int(1) ["size"]=> int(0) } }

[ThunderCat]

wz, для проверки срабатывания формы есть отдельная переменная $_SERVER['REQUEST_METHOD'] == 'POST', а у вас код тупо вообще не будет работать если файла нет. И после срабатывания формы в любом случае нужно перенаправить заголовком на страничку, чтобы при рефреше страницы данные не отправлялись еще раз.

[rensly]

ThunderCat,

var_dump($_FILES);
if ($_SERVER['REQUEST_METHOD'] == 'POST') {
    if ($_FILES && $_FILES['filename']['error'] == UPLOAD_ERR_OK) {
        if (isset($_POST['authorname'] ) && isset($_POST['songname'])) {
            if ($_FILES['filename']['size'] >= 10485760) {
                $log = "Max file size is 10mb";
            } else {
                $author_name = $_POST['authorname'];
                $song_name = $_POST['songname'];

                $songName = $author_name . " - " . $song_name;
                $url = str_replace(" ", "", $songName);

                move_uploaded_file($_FILES['filename']['tmp_name'], "$url.mp3");

                $size = round($_FILES['filename']['size'] / 1024 / 1024, 2);
                $hash = hash_file('sha256', "song/$url.mp3");

                $song = "http://".$_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI']."song/";
                $cur = str_replace('upload.php', '', $song) . $url;

                $query = $db->prepare("INSERT INTO songs (name, authorID, authorName, size, download, hash)
                VALUES (:name, '9', :author, :size, :download, :hash)");
                $query->execute([':name' => $songName, ':download' => $cur, ':author' => $author_name, ':size' => $size, ':hash' => $hash]);
                
                $log = "Success! ID: <b>". $db->lastInsertId() ."</b>";
            }
        } else {
            $log = "Provide a song name and an author name.";
        }    
    } else {
        $log = "Error uploading file.";
    }
}

?

[ThunderCat]

wz, ну, не гениально, но для начала пойдет. Еще

else {
        $log = "Error uploading file: ".$_FILES['filename']['error'];
    }

и var_dump($_FILES); потом уберите, это чисто для отладки и понимания что там вообще пришло.

[ThunderCat]

["error"]=> int(1)

[rensly]

ThunderCat, Error uploading file: 1

[ThunderCat]

wz, выше дал ссылку, и перечитайте комментарии в коде в моем ответе ВНИМАТЕЛЬНО.

[ThunderCat]

wz,

move_uploaded_file($_FILES['filename']['tmp_name'], "song/$songname.mp3");

Во первых, где проверка? move_uploaded_file выдает результат операции. Во вторых должно выдавать варнинг что путь не найден, так как это дебильный путь, не привязанный ни к чему. Путь при загрузке должен быть абсолютным, от корня драйва. Используйте соответствующие переменные и константы, __FILE__, __DIR__, и сервер документ_рут. Ну и права должны быть настроены на папку.