Skip to content

fix(vue): reuse ms3_menu_remove for product link confirm header - #559

Merged
biz87 merged 1 commit into
betafrom
fix/issue-556-link-remove-title
Aug 16, 2026
Merged

fix(vue): reuse ms3_menu_remove for product link confirm header#559
biz87 merged 1 commit into
betafrom
fix/issue-556-link-remove-title

Conversation

@Ibochkarev

Copy link
Copy Markdown
Member

Описание

Заголовок confirm удаления связи показывал ключ ms3_menu_remove_title. Такого ключа в лексиконах нет. Текст вопроса уже переведён через ms3_menu_remove_confirm.

Header берёт существующий ms3_menu_remove («Удалить» / «Remove») — тот же ключ, что toast и aria-label. Новых строк в lexicon нет.

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

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

Связанные Issues

Closes #556

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

Красный → зелёный: ProductLinksVueTabTest падал на ms3_menu_remove_title, после замены exit 0.

php tests/ProductLinksVueTabTest.php                         # exit 0
php -l tests/ProductLinksVueTabTest.php                      # exit 0
npx eslint src/components/product/ProductLinksTab.vue        # exit 0
composer test:smoke                                          # exit 0, 70 tests
  • Ручное тестирование
  • Автоматические тесты (composer test:smoke, ESLint на изменённом Vue)
  • Тестирование на разных версиях PHP/MODX

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

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

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

Чеклист

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

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

Новый ключ ms3_menu_remove_title не добавлял: в manager.inc.php уже есть ms3_menu_remove. Ревью code-reviewer и thermo-nuclear: блокеров нет.

The title key ms3_menu_remove_title does not exist. The confirm body
already uses manager lexicon; the same Remove string is enough.
@Ibochkarev Ibochkarev added the bug Something isn't working label Aug 13, 2026
@Ibochkarev
Ibochkarev requested a review from biz87 August 13, 2026 17:26
@biz87
biz87 merged commit 6568a84 into beta Aug 16, 2026
3 checks passed
@biz87
biz87 deleted the fix/issue-556-link-remove-title 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] Ключ ms3_menu_remove_title в confirm удаления связи товара

2 participants