Skip to content

fix: remove action from miniShop3 menu configuration#94

Merged
biz87 merged 1 commit intobetafrom
fix-issue-11
Feb 16, 2026
Merged

fix: remove action from miniShop3 menu configuration#94
biz87 merged 1 commit intobetafrom
fix-issue-11

Conversation

@Ibochkarev
Copy link
Copy Markdown
Member

Описание

У пункта меню «miniShop3» при установке задавалось поле Действие = mgr/orders. При клике по пункту подменю (Заказы, Клиенты, Настройки и т.д.) на мгновение появлялось и сразу срабатывал переход по этому action, из‑за чего нельзя было выбрать подпункт.

В _build/elements/menus.php у родительского пункта miniShop3 поле action заменено с 'mgr/orders' на ''. После установки/обновления пакета клик по «miniShop3» только раскрывает подменю, без перехода на другую страницу.

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

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

Связанные Issues

Closes #25

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

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

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

  • MiniShop3: 1.4.1-beta1
  • MODX: —
  • PHP: —

После установки/обновления: клик по «miniShop3» раскрывает подменю без редиректа; подпункты (Заказы, Клиенты и т.д.) по клику открывают нужные страницы.

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

До После
Клик по «miniShop3» сразу переводит на страницу заказов, подменю не выбрать Клик по «miniShop3» раскрывает подменю, подпункты доступны

Чеклист

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

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

Правка в одном файле: _build/elements/menus.php, одна строка ('action' => 'mgr/orders''action' => ''). По комментарию @biz87 MODX поддерживает пустой action у родительских пунктов меню.

@Ibochkarev Ibochkarev self-assigned this Feb 15, 2026
@Ibochkarev Ibochkarev requested a review from biz87 February 15, 2026 15:54
@biz87 biz87 merged commit 2913edb into beta Feb 16, 2026
@biz87 biz87 deleted the fix-issue-11 branch February 16, 2026 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants