Поиск объектов

Вопрос возник после чтения этих файлов “search.php” и “fn.search.php” MULTIVENDOR версия 4.14.2.SP1
После увиденного на скриншотах которые я приложил ниже, напрашивается вопрос, это так надо или уже исправили, а если не исправили, исправте пожалуйста. Спасибо.

2 лайка

Добрый день!

Спасибо за предоставленную информацию. Пожалуйста, уточните, создает ли этот код какие-либо проблемы с функциональностью?

С Уважением, Серж
Специалист поддержки CS-Cart

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

Пожалуйста, ответьте лучше сами, может ли params[‘compact’] быть не пустым и равным не ‘Y’? Если нет, то хорошо. Если может быть ‘N’ - то плохо.

Думаю, тут имеется ввиду логическая ошибка. Т.к. там не сравнение, а присвоение

Вот поэтому я и спрашиваю, что во многих местах в карте у логических переменных как ни странно возможны не два значения, как ожидается, а три: ‘Y’, ‘N’ и ‘пусто’ :thinking:

1 лайк

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