Вывод количества найденных товаров в умном фильтре не работает?

Question

[SteepNET]

Добрый день!
Битрикс, умный фильтр(catalog.smart.filter), на главной фильтрирует вывод не комплексного компонента (catalog.section), все работает! Но появилась странная проблема, ошибок в консоли нет.
Вывод количества найденных товаров показывает всегда максимум , если используются ползунки , если же использовать другие поля фильтра, то все нормально выводит. проблема именно в фильтрах с ползунками.
Шаблон catalog.smart.filter
/bitrix/www/bitrix/components/bitrix/catalog.smart.filter/templates/bootstrap_v4/template.php
~ 795 строка , вывод счетчика с кнопкой перехода, не работает не только счетчик но и переход, просто сбрасывает страницу и все. AJAX в компонентах отключен.
Заметил такую особенность, если фильтровать по другим полям не ползунками, при фильтрации шкала ползунка затемнятся на цене ,но сам шарнир у ползунка не сдвинут, у ползунка кастомизировался ползунок в css.

<div class="smart-filter-popup-result <? if ($arParams["FILTER_VIEW_MODE"] == "VERTICAL") echo $arParams["POPUP_POSITION"] ?>"
                                 id="modef" <? if (!isset($arResult["ELEMENT_COUNT"])) echo 'style="display:none"'; ?>
                                 style="display: inline-block;">
                                <? echo GetMessage("CT_BCSF_FILTER_COUNT", array("#ELEMENT_COUNT#" => '<span id="modef_num">' . intval($arResult["ELEMENT_COUNT"]) . '</span>')); ?>
                                <span class="arrow"></span>
                                <br/>
                                <a href="<? echo $arResult["FILTER_URL"] ?>"
                                   target=""><? echo GetMessage("CT_BCSF_FILTER_SHOW") ?></a>
                            </div>

Comments

[PetrPo]

А catalog.section что выводит? товары с одного корневого раздела или с разных?

[SteepNET]

PetrPo, один инфоблок содержит множество разделов в двух уровнях

[PetrPo]

У тебя товары по какому-то глобальному фильтру (свойству) изначально выводятся? (типа там хит, новинка или еще что-то)

[SteepNET]

PetrPo, arrFilter в настройках инфоблока и все , каких то специальных полей нет просто вводит все из инфоблока и все и сортируется и фильтруется ты знаешь уже как.

[PetrPo]

Попробуй фасетный индекс отключить, если ситауция изменится, снова создай фасету, потому что у меня все норм с ползунками

[SteepNET]

PetrPo, К сожалению не помогло//
Юзаю jquery-3.3.1
Bootstrap v4.4.1

[SteepNET]

PetrPo, Странно, если ставить шаблон .default или visual_horizontal ,то этой проблемы нет, если ставить даже системный не копированный bootstrap_v4 то этот глюк появляется , какая то проблема с bs 4 видимо...
С моим шаблоном идет такой js ,возможно влияет,посмотри пожалуйста...
https://yadi.sk/d/rJvdVparMGhiqA

[PetrPo]

Кстати да, я же поставил чистый битрикс, перед тем как с ползунками решить проблему и поставил шаблон bootstrap_v4 и там не корректно работали ползунки, я не стал париться разбираться почему не работает и сделал на .default шаблоне.
Но если, принципиально bootsrap шаблон нужен, то решение очень простое:
- либо в script.js функция JCSmartFilter.prototype.gatherInputsValues там где case 'text' и другие идут не хватает case 'number'
- либо в template.php у инпутов тип с number на text поменять

Лучше первый вариант, более универсальное решение

[SteepNET]

PetrPo, Просто супер! Работает!
Подскажи такой момент пожалуйста!?
1.Когда используешь один фильтр, и допустим отсортировал товар другим фильтром не цена а скажем цвет,например отсортировался товар в диапазоне от 100 до 150 рублей, должен ли ползунок цены принять эти положения или нет? Это не принципиально просто интерес,должно ли или очередной баг?
2.Тоже наверное баг, после фильтрации все фильтры которые с ползунками, в настройках которых стоит "отображать в свернутом виде" раскрываются , думаю что это ужаснейший баг ...

[PetrPo]

1. Очень двоякая ситуация получается, если ползунки устанавливать, то при следующей фильтрации, даже если ничего не трогать, значения ползунков попадут в url, хотя ты по идеи ты не выставлял фильтр по той же цене. Но на цене хотябы полоска обесцвечивается (то есть виден доступный диапазон), а вот на остальных ползунках вообще ничего не видно
2. Если ты имеешь ввиду, что после того, как ты отфильтровал по параметру, у которого вид ползунки, то вроде как так даже правильнее, чтобы было видно, что этот параметр выбран, ну тут уже сам можешь менять логику, если надо в result_modifier.php

[SteepNET]

PetrPo,
2.Ну вот пример: По умолчанию фильтр настроен ,всего 6 фильтров с ползунками, но 3 в свернутом положении а 3 в развернутом. После того как ты один "развернутый" по умолчанию фильтр с ползунами установил значения, нажал "показать" страница обновилась, и все 6 свойств с ползунками оказались в развернутом положении!

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

[PetrPo]

У меня все норм, что должно быть открыто - открывается, что должно быть закрыто - закрывается.
Что там может быть не так, если ты конечно не накостомил чего-то. В массиве есть DISPLAY_EXPANDED, формируется в component.php, там есть проверка типа если в $_GET есть свойство тогда жестко задается DISPLAY_EXPANDED = Y в template.php по этому флагу добавляется класс bx-active, от него в css ставится display:block

[SteepNET]

PetrPo, Есть у smart.filter в папке только component_epilog.php с содержимым CJSCore::Init(array('fx', 'popup'));
При фильтрации ползунками ,проверил у зарытых и не участвующих фильтров стоит после фильтрации DISPLAY_EXPANDED = Y
Если же без фильтрации по полям с ползунками делать то у свойств которые должны быть закрыты DISPLAY_EXPANDED = null ,у кого должны быть в раскрытом виде у них DISPLAY_EXPANDED = Y ,тоесть им заведомо не верно задается это значение видимо...

Если это в /home/bitrix/www/bitrix/components/bitrix/catalog.smart.filter ,то я его не модифицировал ...

В style.css есть
.smart-filter-button-box .smart-filter-block,
.smart-filter-parameters-box.bx-active .smart-filter-block { display: block; }

В tempale.php
if ($arItem["DISPLAY_EXPANDED"] == "Y"):?>bx-active<? endif ?>">

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

[SteepNET]

PetrPo, Запутался весь =)
В общем системный компонент я не модифицирую не каким образом, модифицирую только его шаблон в /local/
Да вот этот файл /home/bitrix/www/bitrix/components/bitrix/catalog.smart.filter/component.php я не модифицировал никаким образом.
Проблема такая :
1.Используем фильтр с ползунками ,тогда при обновлении страницы у тех свойств с ползунками у которых
DISPLAY_EXPANDED = null , меняется параметр на DISPLAY_EXPANDED = Y, как и у тех что всегда это значение DISPLAY_EXPANDED = Y
2. При использовании фильтра у свойств не с ползунками, свойство DISPLAY_EXPANDED = null сохраняется

Попробовал на .default шаблоне,там поведение нормальное, 3 раскрытых свойства с ползунками так и остаются раскрытыми,а те что скрыты ,так и остаются скрытыми. Хотелось бы такого добиться на bs4 шаблоне...но что там сломано =(

UPD:
Как оказалось, с шаблоном default если нажимать на ссылку которая выводиться с кол-вом найденного , то поведение нормальное, а если нажимать на кнопку "Показать" - "Сбросить" показать ,то такая же ситуация ,раскрываются все свойства с ползунками...

[SteepNET]

Создал тестовый сайт https://1013lab.bitrixlabs.ru/catalog/shoes/filter...
В нем 4 свойства с ползунками на bootstrap4 и там все нормально =\

[SteepNET]

PetrPo, Кажется понял в чем дело, в фильтр для показа передаются все сразу свойства, всех ползунков,даже если их не трогал , в данном примере выбираю для фильтрации только одно свойство arrFilter_P1_MIN=90&arrFilter_P1_MAX=157 а в GET попадают все 6 свойств с ползунками...

/?arrFilter_P1_MIN=90&arrFilter_P1_MAX=134&arrFilter_4_MIN=2000&arrFilter_4_MAX=2400&arrFilter_25_MIN=1&arrFilter_25_MAX=2&arrFilter_27_MIN=2000&arrFilter_27_MAX=4800&arrFilter_2_MIN=5&arrFilter_2_MAX=30&arrFilter_3_MIN=500&arrFilter_3_MAX=1024&set_filter=%D0%9F%D0%BE%D0%BA%D0%B0%D0%B7%D0%B0%D1%82%D1%8C

А у default шаблона передается только ?set_filter=y&arrFilter_P1_MAX=144 опять же если нажимать на ссылку где количество элементов, если нажимать на обычный кнопки "показать" то тоже вываливает такой большой фильтр...

[PetrPo]

Да, блин, че-то полемика затягивается. Ты data-price проставил в template.php? его в 4 местах надо поставить (2 для min и 2 для max), без него естественно у тебя в фильтр попадут значения ползунков даже если ты их не трогал, value в input проставлен ведь! Проверь, поставь где надо этот параметр, так сказать абстрагируйся от того, что он data-price называется, нужен он не только для цены, но и для других ползунков тоже.
Конечно, я соглашусь, что тупо называть все опираясь на price, если это используется не только для цены, просто если посмотреть шаблон и script.js все функции, классы и прочее называется с использованием price, вот я и решил от общей картины не отходить, думал так будет понятнее

[SteepNET]

PetrPo, Прости, моя вина,смотрел на это не поправил...
Теперь кнопка которая с счетчиком передает все норм.
Но как быть с кнопкой "Показать" ,она по прежнему передает в GET все свойства. И Так же кнопка "Сбросить" хоть и сбрасывает но в GET все валиться так же....

[PetrPo]

Вообщем, если у тебя фильтр без ЧПУ, придется еще кастомить
В script.js function JCSmartFilter где if-ки идут замени их

if (params && params.SEF_SET_FILTER_URL) {
	this.bindUrlToButton('set_filter', params.SEF_SET_FILTER_URL);
	this.sef = true;
}
else {
	this.bindUrlToButton('set_filter', window.location.href);
}

if (params && params.SEF_DEL_FILTER_URL) {
	this.bindUrlToButton('del_filter', params.SEF_DEL_FILTER_URL);
}
else {
	this.bindUrlToButton('del_filter', ajaxURL);
}

В функции JCSmartFilter.prototype.postHandler проверка if (result.SEF_SET_FILTER_URL) после нее добавь

else {
	this.bindUrlToButton('set_filter', BX.util.htmlspecialcharsback(result.FILTER_URL));
}

В файле result_modifier.php в самом конце

$arHiddens = array();
foreach($arResult['HIDDEN'] as $arHidden) {
	$arHiddens[] = $arHidden['CONTROL_NAME'].'='.$arHidden['HTML_VALUE'];
}

if($arHiddens) {
	$arResult['FORM_ACTION'] = $arResult['FORM_ACTION'].'?'.implode('&', $arHiddens);
}

[SteepNET]

PetrPo, Все супер! Спасибо большое! Ты просто прелесть!
Но почему то "Сбросить" не пашет, наверное что то я прозевал...
Если закоментить условие

else {
		this.bindUrlToButton('del_filter', ajaxURL);
	}

То норм..,но при этом все GET вываливаются в адрес..
С ЧПУ надо еще дописывать?
"AJAX_MODE" => "N" должен быть включен у bitrix:catalog.section ?

[PetrPo]

В resullt_modifier не надо ничего добавлять это лишнее, в $arResult['FORM_ACTION'] уже и так все это есть оказывается

Полностью первая функция

function JCSmartFilter(ajaxURL, viewMode, params)
{
	this.ajaxURL = ajaxURL;
	this.form = null;
	this.timer = null;
	this.cacheKey = '';
	this.cache = [];
	this.popups = [];
	this.inputsPrice = {};
	this.viewMode = viewMode;

	if (params && params.SEF_SET_FILTER_URL) {
		this.bindUrlToButton('set_filter', params.SEF_SET_FILTER_URL);
		this.sef = true;
	}
	else {
		this.bindUrlToButton('set_filter', window.location.href);
	}
	
	if (params && params.SEF_DEL_FILTER_URL) {
		this.bindUrlToButton('del_filter', params.SEF_DEL_FILTER_URL);
	}
	else {
		this.bindUrlToButton('del_filter', ajaxURL);
	}
}

Вот первый параметр передаваемый в эту функцию и в this.bindUrlToButton('del_filter', ajaxURL) должны одинаково называться ajaxURL

[SteepNET]

PetrPo, https://yadi.sk/d/LRpsXYu7VMoQdw
Удалил прежде добавленное в resullt_modifier ,в общем то код JCSmartFilter не изменился с прежним вариантом.
По прежнему не работает "Сброс"
Если комментить

else {
    this.bindUrlToButton('del_filter', ajaxURL);
  }

То работает но вываливает все в GET
В файл добавил настройки bitrix:catalog.smart.filter в самый верх, посмотри пожалуйста что не так ?

[PetrPo]

С ЧПУ для комплексного компонента как раз первый мой код и нужен был.
Без комплексного компонента (как у тебя на главной странице) для ЧПУ надо будет дописывать... новый битрикс))

[PetrPo]

пиши console.log(ajaxURL) смотри что приходит, по всей видимости ничего... в template.php ищешь где вызывается JCSmartFilter смотришь что в него передается, должен быть arResult['FORM_ACTION'] если все так. смотри что в этом самом FORM_ACTION... вообщем дебаж, я не экстрасенс, чтобы угадывать почему у тебя не работает

[SteepNET]

PetrPo,
console.log(ajaxURL) = "/"
arResult['FORM_ACTION'] = "/"

<script type="text/javascript">
    var smartFilter = new JCSmartFilter('<?echo CUtil::JSEscape($arResult["FORM_ACTION"])?>', '<?=CUtil::JSEscape($arParams["FILTER_VIEW_MODE"])?>', <?=CUtil::PhpToJSObject($arResult["JS_FILTER_PARAMS"])?>);
</script>

Оно и переходит в "/" и GET очищается при нажатии на "Сброс" но ползунки не сбрасываются и catalog.section не очищается от фильтра...

[PetrPo]

Дело в параметре умного фильтра SAVE_IN_SESSION = Y, допиши

var mark = ajaxURL.indexOf('?') !== -1 ? '&' : '?';
this.bindUrlToButton('del_filter', ajaxURL + mark + 'del_filter=Y');

[SteepNET]

PetrPo, Спасибо за терпение! Теперь все работает отлично!
Пойду учить JS ..