Skip to content

chore(mgr): удалить orphan Ext assets (Phase 0 #521) - #530

Merged
biz87 merged 1 commit into
betafrom
fix/issue-522-orphan-ext-cleanup
Aug 11, 2026
Merged

chore(mgr): удалить orphan Ext assets (Phase 0 #521)#530
biz87 merged 1 commit into
betafrom
fix/issue-522-orphan-ext-cleanup

Conversation

@Ibochkarev

Copy link
Copy Markdown
Member

Описание

Phase 0 epic #521: удалены мёртвые ExtJS-ассеты и добавлен smoke-gate против повторного подключения.

  • product/category.tree.js — нигде не грузился (Categories на Vue)
  • utilities/import/panel.js — вкладка Import уже монтирует Vue (ms3:mountVueImport)
  • utilities.class.php больше не вызывает addJavascript(...import/panel.js)
  • OrphanExtAssetsTest.php — файлы удалены + 0 ссылок в controllers/mgr assets

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

  • Рефакторинг (без изменения функциональности)

Связанные Issues

Closes #522
Refs #521

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

Локальный CI-гейт (без полной установки MODX/MySQL), PHP lint + smoke:

cd core/components/minishop3
php -l controllers/mgr/utilities.class.php   # exit 0
php -l tests/OrphanExtAssetsTest.php         # exit 0
php tests/OrphanExtAssetsTest.php            # OK, exit 0
php tests/run-smoke.php                      # OK smoke tests (59), exit 0
  • Автоматические тесты (OrphanExtAssetsTest в smoke suite)
  • Ручное тестирование

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

  • MiniShop3: ветка fix/issue-522-orphan-ext-cleanup
  • PHP: локальный CLI

Чеклист

  • Код соответствует стилю проекта
  • Изменения не ломают существующую функциональность
  • PHPStan: N/A (нет новых typed paths; smoke only)
  • ESLint: N/A (Vue не трогали)
  • Обновлён CHANGELOG.md — нет (не релиз)

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

Delete unused category.tree.js and utilities/import/panel.js; utilities Import tab already uses Vue. OrphanExtAssetsTest prevents re-introducing dead loads.
@Ibochkarev Ibochkarev added enhancement New feature or request javascript Pull requests that update javascript code tech-debt Maintainability / refactor / architecture debt labels Aug 11, 2026
@Ibochkarev
Ibochkarev requested a review from biz87 August 11, 2026 01:57
@biz87
biz87 merged commit f85703e into beta Aug 11, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request javascript Pull requests that update javascript code tech-debt Maintainability / refactor / architecture debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Phase 0 (#521): orphan Ext cleanup

2 participants