Почему не отрабатывает move_uploaded_file?

Question

[0906ddd]

$('.request__button').on('click', function(e) {
        var phone  = $('#c_phone-number').val();
        var email = $('#c_email').val();  

        if( typeof filesBlock == 'undefined' ) return;
        
        var data = new FormData();

        let action = 'request';

        data.append( 'action', action );
        data.append( 'phone', phone );
        data.append( 'email', email );
        
        for(let i = 0; i < filesCount; i++) {
          data.append(i, filesBlock[i]);
        }

        data.append('upload_files', 1);
        

            $.ajax({
                url: myajax.url,
                type: 'POST', 
                cache: false,
                dataType: 'json',
                processData: false,
                contentType: false,
                data: data, 
              
             }).done(function(data){


            });
     return false;
        
    }); 


function request_action()   {
 	$chat_id = 'свой номер ид';
	$token = 'свой номер токена'; 

	$phone = $_POST['phone'];
	$email = $_POST['email'];

	$text .= "<b>Сообщение с сайта</b>\n\n";
	$text .= "Телефон: ".$phone."\n";
	$text .= "Email: ".$email."\n";
	
	if( isset( $_POST['upload_files'] ) ) {  

		$site = $_SERVER['DOCUMENT_ROOT'];
		$path = '/wp-content/uploads/files';
		$uploaddir = $site . $path;

		// cоздадим папку если её нет
		if( ! is_dir( $uploaddir ) ) mkdir( $uploaddir, 0755 );

		$files  = $_FILES; // полученные файлы
		$done_files = array();

		// переместим файлы из временной директории в указанную
		foreach( $files as $file ){
			$file_name = $file['name'];

			if( move_uploaded_file( $file['tmp_name'], "$uploaddir/$file_name" ) ){
				$done_files[] = realpath( "$uploaddir/$file_name" );
				$text .= "Ссылка на файл: ".$done_files[]."\n";
			}
		}

		$data = $done_files ? array('files' => $done_files ) : array('error' => 'Ошибка загрузки файлов.');

		die( json_encode( $data ) );
	}

	$text = urlencode($text);

    $str = "https://api.telegram.org/bot".$token."/sendMessage?chat_id=".$chat_id."&parse_mode=html&text=".$text;

    $ch = curl_init();
    curl_setopt($ch, CURLOPT_URL, $str);
    curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
    $output = curl_exec($ch);
    curl_close($ch);

    wp_die();
}

Comments

[FanatPHP]

какое дублирование?

Answer 1

[zkrvndm]

Так на вскидку вижу ошибку при отправке файлов:

for(let i = 0; i < filesCount; i++) {
          data.append(i, filesBlock[i]);
        }

Comments

[0906ddd]

Надим Закиров, извините, а что поменяли? в коде ж есть такое

[zkrvndm]

0906ddd, я ничего не поменял, я просто указываю, где у вас ошибка. При прикреплении файлов в .append надо передавать 3 параметра, а не 2, плюс не понятно что там у вас лежит в filesBlock.

[0906ddd]

Надим Закиров, какой третий? в примерах всегда два
в filesBlock лежат элементы массива

[zkrvndm]

0906ddd, filesBlock у вас должен быть ссылкой на поле выбора файлов, а параметров нужно три, а не две.
https://learn.javascript.ru/formdata#metody-obekta...

Answer 2

[Сергей]

при условии что ошибка именно в move_uploaded_file
проверь существует ли конечная папка, проверь права на запись в эту папку, проверь регистр символов во всём пути к конечной папке, ну и конечно не должно быть кириллицы в пути к папке

ну и неплохо было бы продампить переменную $_FILES, может она пустая

Comments

[0906ddd]

Сергей. а как проверить права?

[Сергей]

да хотя бы с помощью filezilla

[0906ddd]

Сергей, не так проверяю?

[FanatPHP]

не надо ничего проверять

если не существует конечная папка, то move_uploaded_file выдаст ошибку
если не те права на запись в эту папку, то move_uploaded_file выдаст ошибку
если не тот регистр символов во всём пути к конечной папке, то move_uploaded_file выдаст ошибку
ну и конечно если быть кириллицы в пути к папке, то move_uploaded_file тоже выдаст ошибку

то есть надо не совершать случайным образом взятые с потолка суетливые телодвижения, а просто прочитать сообщение об ошибке

при условии, что дело в move_uploaded_file

[0906ddd]

FanatPHP, просто прочитать сообщение об ошибке
где именно? на странице ничего нет

[Сергей]

0906ddd, у тебя же WordPress?
Дебаг в WordPress

[Сергей]

0906ddd, и проверяешь права ты на локалке а не на сервере

[FanatPHP]

0906ddd, нету ножек - нету и варенья
если нет ошибки, значит дело не в move_uploaded_file
с чего вы вообще решили, что дело доходит до вызова этой функции?

[0906ddd]

FanatPHP, предположение.
а тут ошибки и не выводятся
код же в обработчике

[FanatPHP]

ну вот и надо проверять свои предположения.
если перед move_uploaded_file написать die('move') то выведет что-нибудь?

что вообще за манера писать сразу кучу кода, от отправки из яваскрипта до отправки в телеграм, не отладив ни один промежуточный этап?

[0906ddd]

FanatPHP, в js отладка была

пробую. ничего
тут разве можно проверить? в обработчике

[FanatPHP]

что значит "можно"? а кто мешает-то?

[Сергей delphinpro]

0906ddd, откройте в браузере консольку, перейдите на вкладку Сеть, отфильтруйте xhr запросы и повторите отправку формы.
Увидите новый выполненый запрос. Нажмете на него, увидите подробности. В том числе то, что вернул сервер, включая ошибки, если они есть.

[0906ddd]

FanatPHP, код в request_action()

add_action('wp_ajax_request', 'request_action');
add_action('wp_ajax_nopriv_request', 'request_action');

function request_action()   { 

}

[0906ddd]

Сергей delphinpro, та вроде ничего такого

[FanatPHP]

Это запрос
а смотреть надо ответ сервера

[0906ddd]

FanatPHP, Сергей delphinpro, насколько я понимаю тут ошибка
но на что заменить не понятно

$text .= "Ссылка на файл: ".$done_files[]."\n";

[FanatPHP]

на что угодно, только осмысленное.

у меня только просьба, не отмечать этот ответ решением, только потому что совершенно посторонние люди помогли в комментариях

[0906ddd]

FanatPHP, просто так будет дублирование

$done_files[] = realpath( "$uploaddir/$file_name" );
				$text .= "Ссылка на файл: ".realpath( "$uploaddir/$file_name" )."\n";

[Сергей delphinpro]

0906ddd, на всякий случай уточню: realpath даст вам путь в файловой системе сервера. Либо false в случае, если не сможет отрезолвить его.

А вашу строку кода замените например на это

$text .= "Ссылка на файл: ".implode("\n", $done_files)."\n";

implode

upd
только вынесите эту строку за пределы цикла foreach