Skip to content

Comments

fix(orders): Убрать GROUP BY в списке заказов#102

Merged
biz87 merged 1 commit intobetafrom
fix/orders-list-empty
Feb 20, 2026
Merged

fix(orders): Убрать GROUP BY в списке заказов#102
biz87 merged 1 commit intobetafrom
fix/orders-list-empty

Conversation

@Ibochkarev
Copy link
Member

Описание

Исправление пустого списка заказов в менеджере. При 269 заказах в БД таблица отображала пустые строки, но пагинация и статистика работали.

Причина: Условие GROUP BY msOrder.id в OrdersController::getList() приводило к пустому результату (MySQL ONLY_FULL_GROUP_BY). JOIN‑ы по одному адресу, статусу, доставке и оплате — однозначные, поэтому GROUP BY не нужен.

Решение: Удалён GROUP BY msOrder.id из запроса.

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

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

Связанные Issues

Closes #(номер issue)

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

  • Ручное тестирование (проверка списка заказов при 269 записях в БД)
  • Автоматические тесты (PHPStan, ESLint)
  • Тестирование на разных версиях PHP/MODX

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

  • MiniShop3: текущая ветка
  • MODX: 3.x
  • PHP: 8.x

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

До После
Пустой список при 269 заказах Список заказов отображается

Чеклист

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

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

  • Изменён файл: core/components/minishop3/src/Controllers/Api/Manager/OrdersController.php
  • Удаление GROUP BY — единственное изменение, без правок SELECT/JOIN

GROUP BY msOrder.id приводил к пустому результату при наличии записей в БД.
Все JOIN однозначные (адрес, статус, доставка, оплата), поэтому GROUP BY не нужен.
@Ibochkarev Ibochkarev requested a review from biz87 February 19, 2026 18:11
@Ibochkarev Ibochkarev self-assigned this Feb 19, 2026
@biz87 biz87 merged commit e765455 into beta Feb 20, 2026
@Ibochkarev Ibochkarev deleted the fix/orders-list-empty branch February 20, 2026 17:39
@biz87 biz87 mentioned this pull request Feb 20, 2026
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants