Skip to content

fix(manager): переключатель черновиков и фильтры в списке заказов#303

Open
Ibochkarev wants to merge 2 commits into
betafrom
fix/302-orders-grid-show-drafts
Open

fix(manager): переключатель черновиков и фильтры в списке заказов#303
Ibochkarev wants to merge 2 commits into
betafrom
fix/302-orders-grid-show-drafts

Conversation

@Ibochkarev
Copy link
Copy Markdown
Member

Описание

В Vue-гриде заказов добавлен чекбокс «Показывать черновики», который передаёт show_drafts в API и позволяет видеть записи со статусом черновика без правки системной настройки MODX. Бэкенд принимает параметр запроса show_drafts поверх ms3_order_show_drafts.

Исправлена передача фильтров по колонкам: динамические поля отправляются с префиксом filter_, как ожидает OrdersController::applyFilter() (по аналогии с CustomersGrid).

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

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

Связанные Issues

Closes #302

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

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

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

  • MiniShop3: 1.11.1-beta1
  • MODX: 3.x
  • PHP: 8.2+

Проверки:

  • php -l для изменённых PHP-файлов
  • npm run lint для OrdersGrid.vue
  • npm run build для vueManager (артефакты в gitignore)

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

До После
В БД 4 заказа, в гриде 1 (только не-черновик) Чекбокс «Показывать черновики» включает все записи

Чеклист

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

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

  • Чекбокс по умолчанию берёт значение из ms3_order_show_drafts через ms3.config.order_show_drafts.
  • Счётчик «Заказов / На сумму» по-прежнему считает только статусы из ms3_status_for_stat — это отдельная логика статистики, не общее число строк грида.

Vue orders grid lacked a way to include draft rows despite ms3_order_show_drafts,
and column filters were sent without the filter_ prefix expected by the API.
@Ibochkarev Ibochkarev requested a review from biz87 May 25, 2026 06:32
Enhanced the Vue orders grid by adding a tooltip for the order statistics, clarifying that only placed orders are counted, excluding drafts. Updated English and Russian lexicons accordingly. Improved the logic in the OrdersController to handle draft visibility more clearly.
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.

[Bug] В списке заказов не отображаются черновики и нет переключателя в Vue-менеджере

1 participant