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

Question

[leni_m]

Есть страничка html, подключил к ней js плагин на загрузку нескольких файлов перетаскиванием.
Думаю при загрузке файлов на сервер запомнить, какие файлы загрузил пользователь (далее будет отправка их на мыло).
Плагин закидывает по одному файлу на php обработчик, и я вот думал, что можно через сессию их все записать в один массив, но почему-то массив перезаписывается, а не добавляются новые пути.
вот код, который сейчас есть в обработчике:

<?php
function save_file($path) {
    if (empty($_SESSION['files'])) {
        session_start();
        $_SESSION['files'] = array();
    }
    array_push($_SESSION['files'], $path);
}
if($_SERVER['REQUEST_METHOD'] == 'POST') {
	if(!empty($_FILES['newfile'])) {
            $path = dirname(__FILE__).'/uploads/'.$_FILES['newfile']['name'];
		if(move_uploaded_file($_FILES['newfile']['tmp_name'],$path)){
                    save_file($path);
                    echo "OK";
		}
		else {
			echo 'error';
		}
	}
}
print_r ($_SESSION['files']);

из браузера скрин, кидаю 5 файлов и везде один элемент возвращает.

Comments

[Павел Корнилов]

На зачем проверку

if (empty($_SESSION['files'])) {
        session_start();
        $_SESSION['files'] = array();
    }

делать внутри тела цикла? Лучше снаружи.

[leni_m]

KorniloFF, Страничка HTML и я не знаю как до обработчика открыть сессию. Или куда запоминать файлы?

[Павел Корнилов]

1. Проверку на наличие сессии с созданием массива нужно делать ДО цикла.
2. Запоминать пути файлов можно как в сессию (думаю, предпочтительно, поскольку они будут разные у пользователей), так и в файл, скажем, как вариант.

[leni_m]

KorniloFF,
js:

$(".upload").upload({
                    action: 'obr.php',
                    label: 'Перетащите файлы в блок загрузки (до 10 Мб)',
                    postKey: 'newfile',
                    maxQueue: 7,
                    maxSize: 10485760
                })

и как тут до цикла сделать проверку?

[Павел Корнилов]

leni_m, пожалуй, да. Точнее, тут нужно вникать в систему передачи клиент - сервер - клиент, чтобы иметь возможность её оптимизировать.
Навскидку не скажу.

[Павел Корнилов]

Как-то так я бы попробовал, но нужно тестировать:

function save_file($path) {
    if (empty($_SESSION)) {
        session_start();
    }

   $_SESSION['files'] = $_SESSION['files'] ?? [];
   $_SESSION['files'][] = $path
}

Answer 1

[Merzley]

Перенесите session_start() в начало файла. Дело в том, что до вызова session_start() переменной $_SESSION вообще не существует. Поэтому выполнение функции save_file() всегда выглядит так:

• Пустая ли переменная $_SESSION['files'] - всегда да, т.к. переменной $_SESSION не существует
  
• запускаем сессию - появляется переменная $_SESSION, в которой есть элемент ['files'], хранящий данные из прошлого запроса
  
• очищаем элемент ['files'] - стирается всё, что приходило в прошлых запросах
  
• помещаем туда данные текущего запроса
  

При последующих запросах всё повторяется, поэтому и получается, что в сессии только одна запись сохраняется

Comments

[leni_m]

function save_file($path) {
   // первый раз выполнится условие, т.к. она пуста.
   // второй и последующие разы не выполнится
    if (empty($_SESSION['files'])) {
    // а раз выполнится, то создаст переменную $_SESSION['files']
        session_start();
        $_SESSION['files'] = array();
    }
    // и запишет первый файл в наш массив
    array_push($_SESSION['files'], $path);
}

разве не так?

[Merzley]

Увы, но нет. До вызова session_start() переменной $_SESSION ВООБЩЕ НЕ СУЩЕСТВУЕТ. Поэтому empty(#не существующая переменная#) всегда будет выдавать true. Да и вообще любое обращение к переменной $_SESSION до вызова session_start() не имеет никакого смысла. Поэтому самое минимальное, куда можно поместить session_start() это непосредственно перед if (empty($_SESSION['files'])) {

[leni_m]

Merzley, действительно session_start() в начале файла решило проблему, спасибо громное!