[Ibochkarev]

Описание

Fenom и PHP могут передать в msProducts sortby массивом (['menuindex' => 'ASC']). Сниппет делал (string), получал "Array", и подстановка effective menuindex (#625 / #737) не срабатывала: каталог молча сортировался по родному menuindex товара.

Массив нормализуется в JSON (pdoTools это умеет), JOIN msCategoryMember и CASE включаются так же, как для строки/JSON.

Тип изменений

• Исправление бага (non-breaking change)
• Новая функциональность (non-breaking change)
• Breaking change (изменение, ломающее обратную совместимость)
• Рефакторинг (без изменения функциональности)
• Документация
• Другое (опишите):

Связанные Issues

Closes #740

Связано: #625 / #627, #736 / #737.

Как это было протестировано?

php -l core/components/minishop3/src/Services/Category/CategoryProductMenuindexService.php
php -l core/components/minishop3/elements/snippets/ms3_products.php
# exit 0

php core/components/minishop3/tests/CategoryProductMenuindexServiceTest.php
# OK

Smoke: ['menuindex' => 'ASC'] ≡ JSON {"menuindex":"ASC"}; ['menuindex' => 'DESC', 'pagetitle' => 'ASC'] подставляет только ключ menuindex.

• Ручное тестирование
• Автоматические тесты (composer ci:php / composer test, npm run lint:ci, composer stan / GitHub Actions CI)
• Тестирование на разных версиях PHP/MODX

Конфигурация тестирования:

• MiniShop3: ветка fix/issue-740-sortby-array
• MODX: —
• PHP: 8.4

Чеклист

• Код соответствует стилю проекта
• Добавлены/обновлены комментарии в сложных местах
• Изменения не ломают существующую функциональность
• Лексиконы добавлены на двух языках (ru/en)
• PHPStan проходит без новых ошибок (composer stan / CI job PHPStan)
• ESLint проходит без ошибок (npm run lint:ci для Vue)
• Обновлён CHANGELOG.md (для значимых изменений)

Дополнительные заметки

Array без menuindex по-прежнему может сломать блок sortbyOptions (TypeError на preg_match) — так было и до фикса. С menuindex в массиве теперь уходит JSON-строка.