Как записать в базу данных название загруженного изображения через форму?

Question

[Hey One]

Нужно реализовать запись в таблицу БД в столбец 'image' следующее:
images/post/['название картинки.jpg'];

Загрузку в папку я реализовал, всё работает, а вот почему-то запись в БД не хочет работать, var_dump показывает что массив $_FILES не содержит ничего, хотя загрузка файла в папку осуществляется. Может кто подсказать где я ошибся и как это реализовать?
Заранее спасибо! :)

Код:

function add_image($connect) {
	$path = '../../images/post/';
	$types = array('image/gif', 'image/png', 'image/jpeg');
	if($_SERVER['REQUEST_METHOD'] == 'POST'){
//Проверка файла на его тип
		if(!in_array($_FILES['image']['type'], $types))
			die();
//Перемещение файла в специальную директорию
		if(!@copy($_FILES['image']['tmp_name'], $path . $_FILES['image']['name'])){
			$image = ($_FILES['image']['name']);
			$sql = "INSERT INTO posts (`image`) VALUES ('$image')";
			mysql_query($connect, $sql);
		}
	}
	var_dump($_FILES);
}

Comments

[Daria Motorina]

Зачем глушить ошибки copy через "@"? Нельзя так делать, если хотите стабильный код

[Hey One]

Daria Motorina, пробывал и без этого, все равно не работает(

[Дмитрий]

Hey One, Вы содержимое массива выводите до того, как скопируете файл в нужную директорию.
Никогда, слышите, никогда не используйте "@", Вы сами себе оказываете медвежью услугу использованием "@"
И используйте move_uploaded_file() вместо copy()!!!

[Дмитрий]

Hey One, вот здесь Вы ошиблись.

if(!@copy($_FILES['image']['tmp_name'], $path . $_FILES['image']['name'])){

Звучит так: "Если не получилось копировать файл в директорию, то сделай мне запись в базу данных"

[Hey One]

Дмитрий, поменял на вот это:

if(move_uploaded_file($_FILES['image']['tmp_name'], $path . $_FILES['image']['name'])){

ничего не поменялось.

[Дмитрий]

Hey One, что именно не поменялось? Файл загружается? Запись в базу не идёт?

[Hey One]

Дмитрий, Запись в БД не идёт. Файл загружался и при !@copy :)

[Дмитрий]

Hey One, а почему у Вас нет проверок на ошибки при выполнении sql запроса?
В логи ещё загляните, тоже помогает найти причину.
По поводу !@copy.
Почитайте на досуге, что означает "!", почему не надо пользоваться "@" и чем отличается copy() от move_uploaded_file()

[Hey One]

Дмитрий, ! - оператор не.
@ - игнор ошибок
copy от move_upload_file - прочту, спасибо)

[Дмитрий]

Hey One, так если Вы знаете, что означает "@" - зачем используете?
Скорей всего причина в ошибке при выполнении sql запроса.

[Hey One]

Дмитрий, да это я с примера старый код выложил, он загружал файл на сервер, забыл подкоректировать текст этого вопроса, моя ошибка.

[Hey One]

Дмитрий, сейчас код выглядит так:

if($_SERVER['REQUEST_METHOD'] == 'POST'){
//Перемещение файла в специальную директорию
		if(move_uploaded_file($_FILES['image']['tmp_name'], $path . $_FILES['image']['name'])){
			$image = ($_FILES['image']['name']);
			$sql = "INSERT INTO posts (`image`) VALUES ('$image')";
			mysql_query($connect, $sql);
		}

[Дмитрий]

Hey One, ну так где же всё-таки проверки на ошибки при выполнении sql запроса?

[Дмитрий]

Hey One, у Вас включён вывод ошибок в браузер? error_reporting() на полную выставлен?

[Hey One]

Дмитрий, сейчас буду их прописывать)

[Hey One]

Дмитрий, нету этого

[Дмитрий]

Hey One, это надо делать не сейчас, а перед тем, как задавать тут вопрос)))

[Hey One]

Дмитрий, стоял только var_dump на _$FILES

[Hey One]

Hey One, он показывал пустоту. то есть в него ничего не прилетало.

[Hey One]

Hey One, но мне кажется что в него ничего прилетать и не будет при обновлении страницы, при клике на кнопку загрузить файл страница обновляется у меня с REQUEST

[Hey One]

Дмитрий, спасибо, учту)

[Дмитрий]

Hey One, а зачем тут круглые скобки?
$image = ($_FILES['image']['name']);

[Hey One]

Дмитрий, моя невнимательность))) несмотря на это, все равно не работает запись в БД.
Может быть дело в том, что что-то мешает в другой функции, которая присутствует в этой форме?

function edit_post($connect, $post_id){
	$post_id = $_GET['post_id'];
	if(isset($_POST['submit'])){
		$title = $_POST['title'];
		$text = $_POST['text'];
		$category = $_POST['category'];
		$update = "UPDATE posts SET `title` = '$title', `text` = '$text', `category_id` = '$category' WHERE id = ".$post_id;
		$result = mysqli_query($connect, $update);
		header("Location: ". $_SERVER["REQUEST_URI"]);
	}
}

[Hey One]

Hey One, хотя здесь image не затронут...

[Дмитрий]

Hey One, где в этом коде проверки на ошибки для sql?

[Hey One]

Дмитрий, они здесь не требуются, так как код рабочий, вопрос в том, может что-то здесь мешает другой функции (которая добавляет изображение на сервер) правильно работать, так как эта функция и функция загрузки изображения подключены к одной форме.

[Дмитрий]

они здесь не требуются

Hey One, так почему же не записывается в базу информация?

[Hey One]

Дмитрий, понял, ушёл проверять, спасибо)

[Дмитрий]

Hey One, Вы в логи смотрели?

[Hey One]

Дмитрий, нет(

[Дмитрий]

Hey One, смотрите, вопрос задали вчера, а в лог Вы собрались только сейчас смотреть... Я Вам два раза писал о проверке ошибок... Вам не кажется это странным?)))

[Hey One]

Дмитрий, мне бы казалось это странным, если бы у меня было полно времени на занятия по PHP, но к сожалению, я имею очень мало времени на данный момент для этого :)

[Дмитрий]

Hey One, ну да, по количеству комментариев и по тому, что только сейчас Вы собрались заглянуть в лог у Вас действительно "мало" времени)))
Первым делом надо заглядывать во все логи, писать проверки на ошибки, включать вывод ошибок и выставлять уровень этих ошибок на полную.
Тогда действительно надо будет мало времени на поиск ошибки.

[Hey One]

Дмитрий, :DDD
я пытался выкрутиться как мог)
Спасибо за совет:)

[Дмитрий]

Hey One, это не совет - это рекомендация.)

Answer 1

[Daria Motorina]

Всегда надо смотреть, есть ли ненулевое значение в поле$_FILES['image']['error'].
Я начала воспроизводить код, сделала самую простую форму загрузки, и проблема отсутствия имени в $_FILES показала себя как ошибка номер 2 в $_FILES['image']['error'] - список ошибок.
Решается указанием в поле hidden значения value, кратным двойке, или вовсе удалением этого поля, валидация размера файла все равно достоверна только на сервере.

<input type="hidden" name="MAX_FILE_SIZE" value="524288" />

Comments

[Hey One]

то есть должна присутствовать проверка файла на его размер?

[Daria Motorina]

Hey One, в идеале - да, должна быть проверка на размер на стороне php

[Hey One]

Daria Motorina, сделал проверку, все равно значение не попадает в _FILES, тут в чем то другом проблема походу...

[Daria Motorina]

Hey One, Что показывает $_FILES['image']['error']? Валидация размера файла - это "джентельменский набор", а не решение конкретно вашей проблемы