Как устранить ошибку Only variables can be passed by reference?

Question

[Денис Московский]

[Error]
Only variables can be passed by reference (0)
/var/www/www-root/data/www/xn--b1aftlcheoc0h.xn--p1ai/bitrix/modules/main/classes/general/php_parser.php:117
#0: PHPParser::GetParamsRec(string, array, string)
/var/www/www-root/data/www/xn--b1aftlcheoc0h.xn--p1ai/bitrix/modules/main/classes/general/php_parser.php:117
#1: PHPParser::GetParamsRec(string, array, array)
/var/www/www-root/data/www/xn--b1aftlcheoc0h.xn--p1ai/bitrix/modules/main/classes/general/php_parser.php:285
#2: PHPParser::GetComponentParams(string, array)
/var/www/www-root/data/www/xn--b1aftlcheoc0h.xn--p1ai/bitrix/modules/main/classes/general/php_parser.php:361
#3: PHPParser::ParseScript(string)
/var/www/www-root/data/www/xn--b1aftlcheoc0h.xn--p1ai/bitrix/modules/main/classes/general/php_parser.php:780
#4: PHPParser::FindComponent(string, string, integer)
/var/www/www-root/data/www/xn--b1aftlcheoc0h.xn--p1ai/bitrix/modules/main/public/component_props2.php:101
#5: require(string)
/var/www/www-root/data/www/xn--b1aftlcheoc0h.xn--p1ai/bitrix/admin/component_props.php:1
----------
Не силен в PHP, но тут вроде по ссылкам и передаются переменные...

public static function GetParamsRec($params, &$arAllStr, &$arResult)
	{
		$found = false;
		$paramsList = "";
		if (mb_strtolower(substr($params, 0, 6)) == 'array(')
		{
			$found = true;
			$paramsList = substr($params, 6);
		}
		elseif(str_starts_with($params, "["))
		{
			$found = true;
			$paramsList = substr($params, 1);
		}
		if($found)
		{
			$arParams = PHPParser::GetParams($paramsList);
			foreach ($arParams as $i => $el)
			{
				$p = mb_strpos($el, "=>");
				if ($p === false)
				{
					if(is_string($arResult))
					{
						$arResult = PHPParser::ReplString($el, $arAllStr);
					}
					else
					{
						PHPParser::GetParamsRec($el, $arAllStr, $arResult[$i]);
					}
				}
				else
				{
					$el_ind = PHPParser::ReplString(mb_substr($el, 0, $p), $arAllStr);
					$el_val = mb_substr($el, $p + 2);
					PHPParser::GetParamsRec($el_val, $arAllStr, $arResult[$el_ind]);
				}
			}
		}
		else
		{
			$arResult = PHPParser::ReplString($params, $arAllStr);
		}
	}

117 строка:

PHPParser::GetParamsRec($el_val, $arAllStr, $arResult[$el_ind]);

Comments

[Ипатьев]

и какая из этих строк 117, а какая 285?

[Денис Московский]

Ипатьев,

PHPParser::GetParamsRec($el_val, $arAllStr, $arResult[$el_ind]);

[ThunderCat]

...
else
	{
		var_dump($el);
		var_dump($arAllStr);
		var_dump($arResult[$i]);
		PHPParser::GetParamsRec($el, $arAllStr, $arResult[$i]);
	}
...

[Денис Московский]

ThunderCat,

$arParams = PHPParser::GetParams($paramsList);
			foreach ($arParams as $i => $el)
			{
				$p = mb_strpos($el, "=>");
				if ($p === false)
				{
					if(is_string($arResult))
					{
						$arResult = PHPParser::ReplString($el, $arAllStr);
					}
					else
					{
						var_dump($el);
						var_dump($arAllStr);
						var_dump($arResult[$i]);
						PHPParser::GetParamsRec($el, $arAllStr, $arResult[$i]);
					}
				}
				else
				{
					$el_ind = PHPParser::ReplString(mb_substr($el, 0, $p), $arAllStr);
					$el_val = mb_substr($el, $p + 2);
					PHPParser::GetParamsRec($el_val, $arAllStr, $arResult[$el_ind]);
				}
				
			}
		}
		else
		{
			$arResult = PHPParser::ReplString($params, $arAllStr);
		}

не работает, если конечно я правильно Вас понял. Ошибка таже самая.

[galaxy]

Денис Московский, со строкой ошибки точно ничего не напутали?
У меня есть подозрения насчет этой:

$el_ind = PHPParser::ReplString(mb_substr($el, 0, $p), $arAllStr);

[Денис Московский]

galaxy, этот файл и данную строку я не правил. Но могу по вашему совету, попробовать исправить и проверить. Подскажете, в чем возможная ошибка?

[Ипатьев]

galaxy, со стэк трейсом сложно спорить

[galaxy]

Денис Московский, можно попробовать

$subst = mb_substr($el, 0, $p);
$el_ind = PHPParser::ReplString($subst, $arAllStr);

[galaxy]

Ипатьев, а я чет не могу понять, он показывает строку с ошибкой или строку с последним вызовом по call stack...

[Денис Московский]

galaxy, к сожалению, без изменений...

[Денис Московский]

galaxy,
вот ошибка:

[Error] 
Only variables can be passed by reference (0)
/var/www/www-root/data/www/xn--b1aftlcheoc0h.xn--p1ai/bitrix/modules/main/classes/general/php_parser.php:117
#0: PHPParser::GetParamsRec(string, array, string)
	/var/www/www-root/data/www/xn--b1aftlcheoc0h.xn--p1ai/bitrix/modules/main/classes/general/php_parser.php:117
#1: PHPParser::GetParamsRec(string, array, array)
	/var/www/www-root/data/www/xn--b1aftlcheoc0h.xn--p1ai/bitrix/modules/main/classes/general/php_parser.php:285
#2: PHPParser::GetComponentParams(string, array)
	/var/www/www-root/data/www/xn--b1aftlcheoc0h.xn--p1ai/bitrix/modules/main/classes/general/php_parser.php:361
#3: PHPParser::ParseScript(string)
	/var/www/www-root/data/www/xn--b1aftlcheoc0h.xn--p1ai/bitrix/modules/main/classes/general/php_parser.php:780
#4: PHPParser::FindComponent(string, string, integer)
	/var/www/www-root/data/www/xn--b1aftlcheoc0h.xn--p1ai/bitrix/modules/main/public/component_props2.php:101
#5: require(string)
	/var/www/www-root/data/www/xn--b1aftlcheoc0h.xn--p1ai/bitrix/admin/component_props.php:1
----------

[Денис Московский]

galaxy, есть похожая проблема, но я не силен в php и смотря на решение, которое там предоставили, не понимаю подходит ли оно мне и как его правильно применить.
Знал бы не написал сюда)

[ThunderCat]

Денис Московский, Не что не работает, а что выдают дампы переменных?

[Денис Московский]

ThunderCat, Вы простите, но я не программист... Программист уже сделал что мог и оставил эту ошибку нам...
страница ошибки Если это можно посмотреть в консоли, то буду благодарен Вам. Правки которые Вы прислали внес.

[Денис Московский]

Денис Московский, правда, возможно видно ее только из под админки...

[ThunderCat]

Денис Московский,

Правки которые Вы прислали внес.

Это не "правки", это код отладки, который должен выводить содержимое переменных. По нему можно понять что происходит в коде на этапе выполнения.

страница ошибки

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

Вы простите, но я не программист.

«Хабр Q&A» — это узкоспециализированный сервис вопросов и ответов, предназначенный для IT-специалистов и всех тех, кто отчаянно пытается ими стать.

Выдержка из правил сервиса.

Программист уже сделал что мог и оставил эту ошибку нам...

Этот программист
явно неправильный, найдите нормального, или бейте этого до решения ситуации (косяк явно программный, не понятно что значит "оставил нам"?).

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

[Денис Московский]

ThunderCat, Я это все понимаю. Я уже указал выше, что нашел похожую проблему и там она просто решалась, естественно, я запостил свой вопрос в надеже, что и моя проблема решиться схожим образом.

[Денис Московский]

Я 90% ошибок исправил за один день, прочитав ответы на схожие проблемы на форумах и тут. И две недели мне их пытались исправить разные программисты за деньги...

[Vitsliputsli]

Денис Московский,

тут вроде по ссылкам и передаются переменные...

Тут, 3 аргумент - получение значения по индексу массива, а он просит переменную.

PHPParser::GetParamsRec($el_val, $arAllStr, $arResult[$el_ind]);

[Денис Московский]

Vitsliputsli, Спасибо! Проблема решена, напишите в решения (я отмечу решением).

[galaxy]

Vitsliputsli, да ну, в какой версии это так работает? Code

[Vitsliputsli]

galaxy, спасибо, вы правы, это работает.
Хотя не понимаю почему, в мануале указано, что только:
Variables, i.e. foo($a)
References returned from functions, i.e.:
можно передать по ссылке. Если есть информация почему это работает, буду благодарен.

[Adamos]

galaxy, это так работает, например, для $arResult === 'error'.
Битрикс же. Эти загадочные массивы (или нет) приходят в шаблоны из глубин его трясин, что в них оказывается и почему - можно выяснять часами и все равно не быть до конца уверенным...

[galaxy]

Adamos, да, спасибо, наконец, воспроизвелось.

Тогда у меня для ТС плохие новости: там не синтаксическая, а существенная логическая ошибка где-то в коде, просто вынос $arResult[$el_ind] в отдельную переменную не спасет.

[Денис Московский]

galaxy, но сработало. Вынос $arResult[$el_ind] в отдельную переменную помог мне. Ошибок нет и модуль в котором воспроизводилась ошибка корректно заработал.

[galaxy]

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

[Vitsliputsli]

Интересно, получается, что передача по ссылке элемента массива полученного по ключу - это недокументированная возможность. Причем это не побочный эффект, php не просто создает новую переменную, а в зависимости от функции, если она требует значение или требует ссылку, будет использоваться разный код - FETCH_DIM_R или FETCH_DIM_W соответственно.
В общем, оно логично. Нет смысла передавать что-либо по ссылке, если у него нет имени и к нему никак потом не обратиться. А к элементу массива обратиться можем, а значит такое поведение нужно.
Со строкой тоже самое, но для строк запрещен FETCH_DIM_W, по понятным причинам.

Answer 1

[samizdam]

Сигнатура метода ожидает второй и третий аргументы переменными по ссылке.
$arResult[$el_ind] - не переменная, а элемент массива. Попробуйте в переменную это значение присвоить и поменять вызов. Переменная сможет быть изменена внутри функции, но на элемент массива это может и не повлиять.