Лишние элементы в результатах поиска?

Question

[Tupic123]

Есть цикл который выводит элементы результатов поиска

<?foreach($arResult['SEARCH'] as $arItem):?>
		<div class="news-item">
		<?
$arFilter = Array("IBLOCK_ID"=>$arItem[15], "ID"=>$arItem["ITEM_ID"], "PROPERTY_FILIAL" => $filial,);
$res = CIBlockElement::GetList(Array(), $arFilter);
if ($ob = $res->GetNextElement()){;
    $arFields = $ob->GetFields(); // поля элемента
    $arProps = $ob->GetProperties(); // свойства элемента

   }

?>
<?php  if ($filial == $arProps['FILIAL']['VALUE']) {

	echo '	<a class="service__link" href="">'.$arFields2['NAME'].'</a>
   		
	<p class="service__field">'.$arProps['EMPTY']['VALUE'].'</p>
	<p class="service__field">'.$arProps['NDS']['VALUE'].'</p>
	<p class="service__field">'.$arProps['FILIAL']['VALUE'].'</p>';
} else {
	echo '';
}   ?>
		</div>
	<?endforeach;?>

Если в фильтрации начинает использоваться свойство, то происходит следующее

Выводится сначала нужный элемент, затем все остальные(видимо равное количеству результатов поиска) начинают заполняться аналогично последнему
Пример с другим value свойства

Тут вывелось два подходящих элемента, а остальные заполнились по последнему
Как бороться с таким?
Как сделать что бы не выводились повторяющиеся элементы и пустые строки?

Comments

[PetrPo]

Слушай, ну в твоем коде плохо буквально все - это скорее мешанина, чем обдуманные действия.

Я так понимаю, этот вопрос - это все та же задача "Фильтрация результатов поиска по свойству инфоблока"?
Давай тогда, с самого начала разберемся, какой компонент используется для поиска - search.page или catalog.search?

[Антон]

И тут тоже не очень
Фильтрация результатов поиска по свойству инфоблока?
GetList внутри foreach совсем не хорошо

[Tupic123]

PetrPo, Используется компонент search.page

<?foreach($arResult['SEARCH'] as $arItem):?>
		<div class="news-item">
		<a href="#" class="service__link"><?echo $arItem["TITLE_FORMATED"]?></a>
		<p class="service__field"><?echo $arItem["ELEMENT"]['PROPERTY_EMPTY_VALUE']?></p>
		<p class="service__field"><?echo $arItem["ELEMENT"]['PROPERTY_NDS_VALUE']?></p>
		<p class="service__field"><?echo $arItem["ELEMENT"]['PROPERTY_FILIAL_VALUE']?></p>
		</div>
	<?endforeach;?>

Поправил код, убрал вывод свойств элемента с помощью GetList, добавив свойства и результаты поиска с помощью модификации result_modifier.php

<?php if (count($arResult["SEARCH"]) > 0) {
 
 $arIDs = array();
 foreach ($arResult["SEARCH"] as $si => $arItem) {
     if ($arItem["MODULE_ID"] == "iblock" && substr($arItem["ITEM_ID"], 0, 1) !== "S") {
         $arIDs[ $arItem['PARAM2'] ][ $arItem["ITEM_ID"] ] = $si;
     }
 }

 CModule::IncludeModule('iblock');

 foreach ($arIDs as $iblockId => $searchIds) {
     $grab = CIBlockElement::GetList(array(), array(
         "IBLOCK_ID"     => $iblockId,
         "ID"             => array_keys($searchIds)
     ), false, false, array(
         "ID",
         "IBLOCK_ID",
         "PREVIEW_PICTURE",
         "PROPERTY_FILIAL",
         "PROPERTY_NDS",
         "PROPERTY_EMPTY",
     ));
     while ($ar = $grab->Fetch()) {
         $ar['PICTURE'] = CFile::GetFileArray($ar["PREVIEW_PICTURE"]);

         $si = $arIDs[ $iblockId ][ $ar["ID"] ];
         $arResult["SEARCH"][ $si ]["ELEMENT"] = $ar;
     }
 }
}
?>

[PetrPo]

Tupic123, так мы какой вопрос разбирать будем, про то как отфильтровать результаты поиска по свойству PROPERTY_FILIAL? или про что?

UPD
я че-то тебя совсем не понимаю, ты спрашиваешь про одно, а потом в описании просто огород городишь

[Tupic123]

PetrPo, Ну, я хотел прояснить то, что передал сам вывод свойств дабы избежать использование GetList. Вопрос тот же: как отфильтровать результаты по свойству PROPERTY_FILIAL

[PetrPo]

Tupic123, то что ты пытаешься сделать через result_modifier - это не правильно, будет некорректно работать постраничка, надо добавить глобальный фильтр до вызова компонента. Т.е. на index.php с вызовом компонента search.page
1. В параметр компонента FILTER_NAME пишешь например
"FILTER_NAME" => "searchFilter"
2. Перед вызовом компонента, добавляешь код

$itemIds = [];

// здесь укажи ИД своего инфоблока (или массив инфоблоков)
$iblockId = 1;

// здесь значение из get (я наугад написал filial)
$filial = $_GET['filial'];

$iterator = \CIBlockElement::GetList([], ['IBLOCK_ID' => $iblockId, 'PROPERTY_FILIAL' => $filial], false, false, ['ID']);
while($row = $iterator->Fetch()) {
	$itemIds[] = $row['ID'];
}

$GLOBALS['searchFilter'] = ['ITEM_ID' => $itemIds];

Все, теперь в $arResult['SEARCH'] уже будут отфильтрованные элементы, дальше уже можешь в result_modifier доставать дополнительные поля, желательно не делать запросы в цикле.

Также желательно код выше закешировать, пример как это сделать можно посмотреть в файле /bitrix/components/bitrix/catalog/templates/bootstrap_v4/section.php

[Tupic123]

PetrPo, Спасибо большое, решение сработало!
Но моя ошибка оказалось настолько глупой... Я вызывал доп.фильтр и подхватывал свойство через $_GET не перед вызовом компонента, а в его шаблоне...

[PetrPo]

Tupic123, отметь решением

[Tupic123]

PetrPo, Отметил, еще раз благодарю за помощь :)

[Tupic123]

PetrPo, А можно еще спросить? Вот я вывел значения свойств, как добавить еще вывод имени свойств?
То есть филиал: значение

[PetrPo]

Tupic123, либо GetNextElement используй вместо Fetch - у тебя в описание есть пример, только в modifier конечно все это выносить надо, либо отдельный запрос делай к таблице свойств инфоблока

$properties = [];

$iblockId = 1;
$iblockPropertyCodes = ['CODE_1', 'CODE_2'];
$iterator = \Bitrix\Iblock\PropertyTable::getList([
	'select' => ['ID', 'CODE', 'NAME'],
	'filter' => ['=IBLOCK_ID' => $iblockId, 'CODE' => $iblockPropertyCodes]
]);
while($row = $iterator->fetch()) {
	$properties[$row['CODE']] = $row;
}

В $iblockPropertyCodes свои коды укажи

Answer 1

[PetrPo]

На index.php с вызовом компонента search.page
1. В параметр компонента FILTER_NAME пишешь например
"FILTER_NAME" => "searchFilter"
2. Перед вызовом компонента, добавляешь код

$itemIds = [];

// здесь укажи ИД своего инфоблока (или массив инфоблоков)
$iblockId = 1;

// здесь значение из get (я наугад написал filial)
$filial = $_GET['filial'];

$iterator = \CIBlockElement::GetList([], ['IBLOCK_ID' => $iblockId, 'PROPERTY_FILIAL' => $filial], false, false, ['ID']);
while($row = $iterator->Fetch()) {
	$itemIds[] = $row['ID'];
}

$GLOBALS['searchFilter'] = ['ITEM_ID' => $itemIds];

Все, теперь в $arResult['SEARCH'] уже будут отфильтрованные элементы, дальше уже можешь в result_modifier доставать дополнительные поля, желательно не делать запросы в цикле.

Также желательно код выше закешировать, пример как это сделать можно посмотреть в файле /bitrix/components/bitrix/catalog/templates/bootstrap_v4/section.php