Skip to content

fix(mgr): declare ms3 via addVueConfig on help page - #554

Merged
biz87 merged 1 commit into
betafrom
fix/issue-553-help-ms3-config
Aug 16, 2026
Merged

fix(mgr): declare ms3 via addVueConfig on help page#554
biz87 merged 1 commit into
betafrom
fix/issue-553-help-ms3-config

Conversation

@Ibochkarev

Copy link
Copy Markdown
Member

Описание

Страница справки менеджера писала ms3.config = … без объявления ms3. В консоли ReferenceError: ms3 is not defined, Vue не видел конфиг (логотип из ms3.config.defaultThumb).

Теперь справка вызывает addVueConfig() до Vue-модуля, как orders/customers/settings: var ms3 = { config } плюс токен менеджера.

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

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

Связанные Issues

Closes #553

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

Красный → зелёный: php tests/HelpVueEntryTest.php сначала падал на отсутствии addVueConfig, после правки OK HelpVueEntryTest (exit 0).

php -l core/components/minishop3/controllers/mgr/help.class.php  # exit 0
php -l core/components/minishop3/tests/HelpVueEntryTest.php     # exit 0
cd core/components/minishop3 && composer test:smoke            # exit 0, 71 tests

Vue/ESLint не гонял: фронт не менялся. PHPStan не гонял: новых типов нет.

  • Ручное тестирование
  • Автоматические тесты (composer test:smoke)
  • Тестирование на разных версиях PHP/MODX

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

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

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

Чеклист

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

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

Ext-контроллеры товара/категории с ms3.config = не трогал: там Ext/MODx.load. Ревью code-reviewer и thermo-nuclear: блокеров нет.

Help assigned ms3.config without var ms3, which throws ReferenceError
before the Vue module mounts. Reuse the same helper as other Vue mgr pages.
@Ibochkarev Ibochkarev added the bug Something isn't working label Aug 13, 2026
@Ibochkarev
Ibochkarev requested a review from biz87 August 13, 2026 17:11
@biz87
biz87 merged commit 752568c into beta Aug 16, 2026
3 checks passed
@biz87
biz87 deleted the fix/issue-553-help-ms3-config branch August 16, 2026 08:23
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.

[Bug] ReferenceError: ms3 is not defined на странице справки менеджера

2 participants