Skip to content

fix(mgr): Fix empty comboOptions suggestions query - #746

Merged
biz87 merged 1 commit into
betafrom
fix/issue-745-option-suggestions
Sep 17, 2026
Merged

biz87 merged 1 commit into
betafrom
fix/issue-745-option-suggestions

Conversation

@Ibochkarev

Copy link
Copy Markdown
Member

Описание

GET /api/mgr/options/suggestions всегда отдавал пустой список: select('DISTINCT value') xPDO превращал в SELECT `DISTINCT value` и MySQL отвечал 1054. Теперь escape('value') + distinct(), при ошибке execute() пишется errorInfo в журнал.

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

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

Связанные Issues

Closes #745

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

cd core/components/minishop3
php -l src/Controllers/Api/Manager/OptionsController.php
# exit 0

./vendor/bin/phpunit tests/Unit/Controllers/Api/Manager/OptionsControllerSuggestionsTest.php
# OK (4 tests, 18 assertions), exit 0
  • Ручное тестирование
  • Автоматические тесты (composer ci:php / composer test, npm run lint:ci, composer stan / GitHub Actions CI)
  • Тестирование на разных версиях PHP/MODX

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

  • MiniShop3: ветка fix/issue-745-option-suggestions от beta
  • MODX: —
  • PHP: 8.4

Скриншоты (если применимо)

не применимо

Чеклист

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

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

После merge имеет смысл вручную проверить подсказки comboOptions на форме товара (связка с #702).

xPDO quoted select('DISTINCT value') as one identifier, so MySQL 1054
emptied comboOptions suggestions. Log errorInfo when execute fails.
@Ibochkarev
Ibochkarev requested a review from biz87 September 17, 2026 02:30
@Ibochkarev Ibochkarev self-assigned this Sep 17, 2026
@Ibochkarev Ibochkarev added the bug Something isn't working label Sep 17, 2026
@biz87
biz87 merged commit 6f8e42d into beta Sep 17, 2026
6 checks passed
@Ibochkarev
Ibochkarev deleted the fix/issue-745-option-suggestions branch September 23, 2026 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Подсказки comboOptions: /api/mgr/options/suggestions всегда пустой из-за select('DISTINCT value')

2 participants