Как исправить ошибку при переходе на php8 Битрикс [TypeError] Argument #5 ($day) must be of type ?int, string given (0)?

Question

[barsik22]

После перехода выдает такую ошибку

[TypeError] 
mktime(): Argument #5 ($day) must be of type ?int, string given (0)
/home/c/crikon8y/ru.reisennachrussland.com/public_html/local/templates/fortuna/components/bitrix/catalog/catalog1/bitrix/catalog.section/.default/template.php:32

Строка кода 32:
$de = mktime(0,0,0,$de[1],$de[0],$de[2]);

Answer 1

[Алексей Уколов]

Прочитать и перевести текст ошибки. Дальше отсчитать пятый аргумент и проверить, что вы в него передаёте. Если число в формате строки, то достаточно просто привести его к int через стандартный каст языка. Если же там реально какая-то строка, то разбираться откуда она берётся в том месте, где должно быть число.

Comments

[barsik22]

Прошу прощения, но найти тех, кто писал это уже нереально, а сама я не могу разобраться

<?if(!defined("B_PROLOG_INCLUDED") || B_PROLOG_INCLUDED!==true)die();
/** var array $arParams */
/** var array $arResult */
/** @global CMain $APPLICATION */
/** @global CUser $USER */
/** @global CDatabase $DB */
/** var CBitrixComponentTemplate $this */
/** var string $templateName */
/** var string $templateFile */
/** var string $templateFolder */
/** var string $componentPath */
/** var CBitrixComponent $component */
$this->setFrameMode(true);?>

<?
$date_begin = htmlspecialchars(@$_GET["date_begin"]);
if($date_begin == '') $date_begin = date("d.m.Y");
$date_end = htmlspecialchars(@$_GET["date_end"]);
$party = htmlspecialchars(@$_GET["party"]);

$boat = htmlspecialchars(@$_GET["boat"]);
$rout = htmlspecialchars(@$_GET["rout"]);

$db = explode(".", $date_begin);
if(!$db[2]) $db[2] = date("Y", time());
$db = mktime(0,0,0,$db[1],$db[0],$db[2]);

if($db == '') $db = 0;

$de = explode(".", $date_end);
if(!$de[2]) $de[2] = date("Y", time());
$de = mktime(0,0,0,$de[1],$de[0],$de[2]);
if($de == '')$de = 99999*99999;


вот код с начала

[Алексей Уколов]

Ну, значит, вам рано ещё заниматься переходом на новую версию PHP. Надо либо откатиться назад, либо найти тех, кто разбирается и готов вам помочь (данный сервис для этого не предназначен). Либо всё же начать разбираться - всё необходимое я вам для этого дал.
"Код с начала" в моём ответе ничего не меняет.

Answer 2

[Stalker_RED]

Открываем документацию по mktime(), видим, что эта функция принимает только тип int в качестве аргументов. (впрочем, можно и не ходить в документацию, об этом же прямо в тексте ошибки написано).

Смотрим что в нее передается в коде - три нуля, с ними все хорошо. Дальше какие-то значения из массива $de. Смотрим что там в этом массиве, он создается на пару строк выше:$de = explode(".", $date_end);
Смотрим документацию по функции explode() (если вы забыли, что это одна из строковых функций), и видим, что она возвращает массив строк. (Ну и да, об этом тоже прямо в тексте ошибки написано, что передана строка).

Вспоминаем чем отличается строка от числа и как преобразовать одно в другое.

Было сложно? Нет, не особо. Но таких ошибок в движке битрикса могут быть сотни или тысячи. Вот в чем сложность.

Comments

[Алексей Уколов]

таких ошибок в движке битрикса могут быть сотни или тысячи

Я сначала хотел сказать, что надо тупо Битрикс обновить, но ошибки-то в директории local - это не движок.
Но то, что починив одну ошибку можно смело ожидать следующую - факт.

[Stalker_RED]

Алексей Уколов, да, присмотревшись к пути вижу templates/fortuna, то есть это шаблон а не движок.
Возможно движок свежий, и нормально работает с php8, и тогда нужно только подчистить ошибки в шаблонах.