Как пофиксить «Only variables should be passed by reference»?

Question

[Иван Будаков]

Как исправить положение? Заранее благодарен!) (До переноса сайта с виртуального хостинга PHP5.6 на VestaCP всё работало, вот незадача...

Notice: Only variables should be passed by reference in /home/admin/web/shopspeed.ru/public_html/engine/include/constants.php on line 10

Файл constants.php

<?php


define('URL_HOME', 'http://'.$_SERVER['HTTP_HOST'].str_replace('index.php', '', $_SERVER['PHP_SELF']));
define('URL_THIS', 'http://'.$_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI']);
define('URL_APP', array_shift(explode('?',substr($_SERVER['REQUEST_URI'], strlen(str_replace('index.php', '', $_SERVER['PHP_SELF']))))));
define('URL_REFERER', @$_SERVER['HTTP_REFERER']);

Comments

[Сергей delphinpro]

Убрать собачку в последней строке

[Иван Будаков]

Сергей delphinpro, Ни на что не повлияло

Answer 1

[Daria Motorina]

Проблема в строчке 10, но вы привели больше :)
Видимо там настройки такие, чтобы явно выводились нотисы.
Исправьте:

define('URL_APP', array_shift(explode('?',substr($_SERVER['REQUEST_URI'], strlen(str_replace('index.php', '', $_SERVER['PHP_SELF']))))));

на

$php_self = $_SERVER['PHP_SELF'];
$request_uri = $_SERVER['REQUEST_URI'];
define('URL_APP', array_shift(explode('?',substr($request_uri, strlen(str_replace('index.php', '', $php_self))))));

Источники предположений:
https://stackoverflow.com/questions/5842366/php-va...
https://stackoverflow.com/a/30392112/5371978

Comments

[Иван Будаков]

Не помогло.
Похоже это не то. Возможно дело в том, что версии PHP разные?

[Сергей delphinpro]

не заметил array_shift… тогда так:

$arr = explode('?',substr(
  $_SERVER['REQUEST_URI'],
  strlen(str_replace('index.php', '', $_SERVER['PHP_SELF']))
));
define('URL_APP', array_shift($arr));

нафига такие вложенные конструкции писать-то? черт ногу сломит..

[Daria Motorina]

Иван Будаков, ошибка осталась та же? поменяйте еще

define('URL_HOME', 'http://'.$_SERVER['HTTP_HOST'].str_replace('index.php', '', $_SERVER['PHP_SELF']));

по аналогии, я не заметила, что str_replace встречается два раза в коде

[Иван Будаков]

Сергей delphinpro, Daria Motorina, Спасибо, думаю кому-то это однозначно пригодится, сейчас я точно уверен что дело в версии PHP.

Текущая версия PHP: 7.2.24-0ubuntu0.18.04.1
До миграции на VestaCP была 5.6, буду думать как сделать 5.6 на весте)

[Daria Motorina]

нафига такие вложенные конструкции писать-то? черт ногу сломит..

Сергей delphinpro, это будущий хайлоад)))

[Daria Motorina]

Иван Будаков, дело не в версии, дело в неправильном коде, просто раньше были отключены notice. VestaCP это просто панель, а не логика работы php. Можете отключить нотисы, если не хотите фиксить, но это неправильный подход

[Сергей delphinpro]

Иван Будаков, Честно говоря, я не помню, чтобы поведение было иным в других версиях. array_shift всегда принимала агрумент по ссылке, и туда нельзя было, передавать ничего кроме переменной.

Возможно у вас просто был отключен вывод ошибок. В данном контексте эта ошибка проблем не доставит, если ее просто подавить. Собственно это не ошибка, и даже не предупреждение, а так, уведомление о не совсем некорректном коде.

[Иван Будаков]

Daria Motorina, Сергей delphinpro, вывод ошибок я включил своими лапками, думал что проблема в этом, пока не увидел отличие версий PHP, которые скорее всего и являются причиной 500 ошибки

[Daria Motorina]

Иван Будаков, значит надо искать fatal error и exception в логах веб-сервера и приложения, ответ на изначальный вопрос дан правильно, другое дело что для вас он не актуален теперь

[Иван Будаков]

Daria Motorina, Да, не спорю. Спасибо)