Как настроить ошибку 404 не выводится ошибка в headers?

Question

[Антон]

Приветствую, помогите решить проблему:
На сайте неправильно работает страница 404, она не отдаёт ошибку 404.
Погуглив, нашёл проблему в коде в urlrewrite.php вот кусок кода:

19 => 
  array (
    'CONDITION' => '#^/#',
    'RULE' => '',
    'ID' => 'bitrix:catalog',
    'PATH' => '/prod/index.php',
    'SORT' => 100,
  ),

Если добавить категорию 'CONDITION' => '#^/product/#' то всё работает. Получается ошибка с редиректами.

Это правило добавляет модуль(или компонент bitrix:catalog). Но проблема в том что у меня очень много категорий и все они в корне, поэтому убирать это правило не хотелось бы.
Как сделать так чтобы этот редирект(правило) остался?
Я так понимаю нужно компонент bitrix:catalog исправлять?
Подскажите что можно сделать?
Какие есть варианты, может кто сталкивался с такой ошибкой.

Answer 1

[Михаил Ливач]

Нет, достаточно допилить шаблон компонента каталога. Сейчас, полагаю, на неправильный URL открывается корень каталога? Если так, то достаточно в странице sections.php добавить свой код

Comments

[Антон]

"Сейчас, полагаю, на неправильный URL открывается корень каталога?"
сейчас каталожные категории которые расположены в корне открываются правильно - всё работает нормально. Не работает только ошибка 404, если страница не найдена, как описал выше.

[Антон]

Подскажите каким образом можно поймать страницы которые не существуют в sections.php и потом отдать ошибку 404?
Хотя бы примерно

Пытался в sections.php просто добавить до всего кода die(); чтобы остановить редирект на страницу 404, всё равно происходит редирект. Такое ощущение что этот файл не подключается.

Саму ошибку я так понял можно вывести с помощью Tools::process404

[Михаил Ливач]

Антон, во-первых, это не редирект. Это роутинг - странненький, но какой есть.
Во-вторых, Вы же сами написали, что ошибка 404 не показывается из-за правила "направлять всё непонятное на комплексный компонент каталога в файле /prod/index.php". Но так и не написали, а что же именно показывается? Я предположил, что это страница шаблона sections.php - она отвечает за отображение корня каталога и одновременно является вариантом "по умолчанию". Посмотрите код компонента, он довольно короткий: bitrix/components/bitrix/catalog/component.php
Далее: возможно, я ошибся, и компонент всё-таки выбрал какую-то другую страницу. Это легко узнать, добавив отладку в компонент:

var_dump($componentPage);
die;
$this->IncludeComponentTemplate($componentPage);

и открыв какой-то несуществующий адрес https://mysite.ru/blablabla/ .
Зная, какая страница выбрана, остальное просто. Допустим, это страница "element". Значит, в шаблоне компонента ( где он у Вас лежит, кстати? /bitrix/templates/* ? /local/templates/* ? ) смотрим $arResult . Допустим, компонент решил, что blablabla - это символьный код элемента.
Значит, надо через API запросить элемент с этим кодом (D7 или по старинке, через CIblockElement::GetList() ) c фильтром ['IBLOCK_ID'=>$arParams['IBLOCK_ID'], '=CODE'=> $code ];
Если нашлось 0 записей - элемента нет и надо вызвать Tools::process404() .

Даже если какое-то допущение было неверным, общий принцип остаётся тот же.

[Антон]

Михаил Ливач, категории все открываются через шаблон section.php или element.php
Другие шаблоны не участвуют, по крайней мере я не заметил этого.
Файлы лежат в папке /local/

[Антон]

Михаил Ливач, в файле bitrix/components/bitrix/catalog/component.php нашёл кусок кода который отвечает за открытие ошибки 404 он находится на 254 строке начинается с if ($folder404 != $APPLICATION->GetCurPage(true))

Пытался туда добавить свой вызов

\Bitrix\Iblock\Component\Tools::process404("",true,true,true,"/404.php"); die;

Но происходит тоже самое, страница с ошибкой открывается, а в заголовке 200 Ok вместо 404

[Михаил Ливач]

Антон, разве $b404 переключается в true ? Что-то мне подсказывает, что нет.

[Михаил Ливач]

категории все открываются через шаблон section.php или element.php

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

[Антон]

Михаил Ливач, да верно, в переменной $b404 на несуществующей странице (стр. 404) лежит true
$arVariables["SECTION_ID"] = NULL
$arVariables["SECTION_CODE"] = NULL

[Михаил Ливач]

Антон, поставьте, пожалуйста, отладку в component.php в 237 строке, чтобы получилось так:

if(!$componentPage)
	{
		$componentPage = "sections";
		$b404 = true;
	}
    var_dump($b404, $componentPage, $arVariables); die;

и покажите результат на несуществующей странице (и URL такой страницы)

[Антон]

Михаил Ливач, Извините что сразу не ответил. Не увидел сообщения.

Сделал, получилось так:

bool(false)
string(7) "section"
array(1) {
  ["SECTION_CODE_PATH"]=>
  string(33) "detskie-ekskursii/thytrfhtrfhtrfh"
}

URL такой:
домен/detskie-ekskursii/thytrfhtrfhtrfh

[Михаил Ливач]

Отлично. Что мы видим:
1) ключей SECTION_ID и SECTION_CODE нет, есть SECTION_CODE_PATH с ошибочным URL.
2) $b404 == false и, очевидно, так и останется.

Для существующих разделов каталога $arVariables["SECTION_CODE"] , скорее всего, не пуст - но это надо проверить. Если предположение верно, то в /local/templates/шаблон_сайта/components/bitrix/catalog/имя_шаблона/section.php достаточно добавить:

<?php
if ( empty($arResult['VARIABLES']['SECTION_CODE'])
    && !empty($arResult['VARIABLES']['SECTION_PATH'])
    && CModule::IncludeModule('iblock') ) 
{
    \Bitrix\Iblock\Component\Tools::process404(
        ""
        ,($arParams["SET_STATUS_404"] === "Y")
        ,($arParams["SET_STATUS_404"] === "Y")
        ,($arParams["SHOW_404"] === "Y")
        ,$arParams["FILE_404"]
    );
}

более параноидальный вариант выглядит так:

<?php
if ( empty($arResult['VARIABLES']['SECTION_CODE']) 
    && !empty($arResult['VARIABLES']['SECTION_PATH'])  
    && CModule::IncludeModule('iblock') ) 
{
    $parts = explode('/', $arResult['VARIABLES']['SECTION_PATH']);
    $filter = ['IBLOCK_ID' => $arParams['IBLOCK_ID'], '=CODE' => array_pop($parts)];
    $rs = CIBlockSection::GetList([], $filter, false, ['ID']);
    if ( !$rs->Fetch() ) {
        \Bitrix\Iblock\Component\Tools::process404(
            ""
            ,($arParams["SET_STATUS_404"] === "Y")
            ,($arParams["SET_STATUS_404"] === "Y")
            ,($arParams["SHOW_404"] === "Y")
            ,$arParams["FILE_404"]
        );
    }
}

[Антон]

Михаил Ливач, не работает так тоже (оба способа проверил), такое ощущение что section.php не успевает обработаться, переход на страницу 404 происходит раньше

[Михаил Ливач]

Антон, кастомная 404-я показывается, но со статусом 200 ? Если так, то вставьте туда трассировку, и узнаете, откуда произошёл переход:

<?php
$traceLog = __FILE__.":".__LINE__."\n";
$trace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);
foreach ( $trace as $tr ) {
    if( !array_key_exists('file', $tr) ) continue;
    $traceLog .= "{$tr['file']}:{$tr['line']}\n";
}
echo $traceLog;