1

Выдает ошибку:

Only variables should be passed by reference

Ругается на строку где получаем расширение:

$ext = array_pop(explode('.',$_FILES['picture']['name']));

И только на php 5.3 работает нормально

Сам код:

if (isset($_POST['add'])) {

$path = 'i/';

$ext = array_pop(explode('.',$_FILES['picture']['name'])); 

$new_name = time().'.'.$ext; 

$full_path = $path.$new_name;

if($_FILES['picture']['error'] == 0){

 if(move_uploaded_file($_FILES['picture']['tmp_name'], $full_path)){

}
}
else{

echo "Ошибка";
}

как убрать эту ошибку на php 7.1 ?

2 ответа 2

4

Тоже натыкался на такое при переходе на PHP7+, проблема в том что нужно вынести результат explode в отдельную переменную:

$name = explode('.',$_FILES['picture']['name']);
$ext = array_pop($name);
1
  • 1
    спасибо большое, добрый человек! 23 мая 2017 в 13:26
1

Дело в том, что функция array_pop в качестве аргумента принимает ссылку на массив. В PHP7 внесено много изменений в механизм передачи по ссылке и, в частности, попытка передать по ссылке результат выполнения функции теперь вызывает ошибку.

Соответственно, как правильно сказал Ярослав, делать нужно так:

$name = explode('.',$_FILES['picture']['name']);
$ext = array_pop($name);
1
  • и вам спасибо!) 23 мая 2017 в 13:26

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.