Skip to content

fix(mgr): не грузить отсутствующий vue-dist/main.min.css на product/update - #504

Open
Ibochkarev wants to merge 1 commit into
betafrom
fix/issue-503-main-min-css
Open

fix(mgr): не грузить отсутствующий vue-dist/main.min.css на product/update#504
Ibochkarev wants to merge 1 commit into
betafrom
fix/issue-503-main-min-css

Conversation

@Ibochkarev

Copy link
Copy Markdown
Member

Описание

На странице редактирования товара менеджер запрашивал css/mgr/vue-dist/main.min.css, которого Vite не создаёт (CSS выходит под именами исходников вроде primeicons.min.css). В Network был 404.

Убраны addCss(main.min.css) и неиспользуемый addVueModule(main.min.js). Вкладки товара по-прежнему поднимает product-tabs с primeicons и DynamicField.

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

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

Связанные Issues

Closes #503

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

Локальный CI-гейт:

php -l core/components/minishop3/controllers/product/update.class.php   # exit 0
php -l core/components/minishop3/tests/ProductUpdateVueAssetsTest.php     # exit 0
cd core/components/minishop3 && composer test:smoke                       # exit 0, OK smoke tests (36)
cd core/components/minishop3 && composer test                             # exit 0, 109 tests
  • Ручное тестирование
  • Автоматические тесты (composer test:smoke / composer test)
  • Тестирование на разных версиях PHP/MODX

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

  • MiniShop3: ветка fix/issue-503-main-min-css от beta
  • MODX: n/a (smoke без установки)
  • PHP: 8.4.17

Чеклист

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

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

Альтернатива из issue (фиксировать имя CSS в Vite для entry main) не нужна: main.min.js на этой странице ничего не монтирует, стили уже даёт product-tabs.

Vite never emits main.min.css; product-tabs already registers the needed
CSS/JS. Drop unused main.min.js too. Closes #503.
@Ibochkarev
Ibochkarev requested a review from biz87 August 8, 2026 16:59
@Ibochkarev Ibochkarev added the bug Something isn't working label Aug 8, 2026
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] product/update грузит несуществующий vue-dist/main.min.css (404)

1 participant