Skip to content

feat(vue): Salvage PrimeVue barrel, cache-bust, and i18n - #702

Open
Ibochkarev wants to merge 1 commit into
betafrom
feat/issue-701-primevue-barrel
Open

feat(vue): Salvage PrimeVue barrel, cache-bust, and i18n#702
Ibochkarev wants to merge 1 commit into
betafrom
feat/issue-701-primevue-barrel

Conversation

@Ibochkarev

@Ibochkarev Ibochkarev commented Sep 11, 2026

Copy link
Copy Markdown
Member

Описание

Из закрытого PR #640 в менеджер переносятся куски, которые не зависят от Modx-темы: named-импорты import { X } from 'primevue' вместо primevue/*, Vite-плагин rejectPrimeVueSubpaths, cache-bust vue-модулей по filemtime (vue_module_cache_bust.inc.php), лексиконы ru/en для help, истории заказа, VendorCombo и Uppy Image Editor.

Тема остаётся Aura (preset: Aura, darkModeSelector: none) на VueTools 1.1.2-pl. Плотность контролов, severity="success" на Save и CSS с токенами modx.* сюда не входят. Без централизованной темы из vueTools#22 они ломают дефолтную Aura.

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

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

Связанные Issues

Closes #701

Связано: #640, vueTools#22

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

cd vueManager
npm run lint:ci
# exit 0

npm test
# 46 tests, exit 0

npm run build
# exit 0, rejectPrimeVueSubpaths не сработал

php -l core/components/minishop3/controllers/vue_module_cache_bust.inc.php
php -l core/components/minishop3/controllers/manager.class.php
php -l _build/resolvers/resolver_01_setup.php
# exit 0

composer ci:php и PHPStan по этому diff не гонялись: PHP-слой это include и три вызова в addVueModule. Менеджер в браузере не проверялся.

  • Ручное тестирование
  • Автоматические тесты (composer ci:php / composer test, npm run lint:ci, composer stan / GitHub Actions CI)
  • Тестирование на разных версиях PHP/MODX

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

  • MiniShop3: ветка feat/issue-701-primevue-barrel
  • MODX: n/a (lint/Vitest/Vite)
  • PHP: 8.4.23

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

Не применимо. Критерий #701: менеджер визуально как текущая Aura.

Чеклист

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

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

Import PrimeVue from the VueTools barrel so manager screens share one
Theme instance. Reject primevue/* subpaths at Vite build time, bust
vue-dist cache with filemtime, and keep Aura as the default preset.

ModxManagerTheme and getActiveTheme stay out until vueTools#22.

Fixes #701
@Ibochkarev Ibochkarev added the enhancement New feature or request label Sep 11, 2026
@Ibochkarev
Ibochkarev requested a review from biz87 September 11, 2026 08:55
@Ibochkarev
Ibochkarev marked this pull request as ready for review September 11, 2026 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

1 participant