Skip to content

fix(manager): ссылка на товар в таблице категории открывается в текущей вкладке#172

Open
Ibochkarev wants to merge 1 commit intobetafrom
fix/category-products-pagetitle-same-tab
Open

fix(manager): ссылка на товар в таблице категории открывается в текущей вкладке#172
Ibochkarev wants to merge 1 commit intobetafrom
fix/category-products-pagetitle-same-tab

Conversation

@Ibochkarev
Copy link
Copy Markdown
Member

Описание

Ссылка с названием товара в таблице товаров категории (MODX manager) больше не открывается в новой вкладке: переход на ?a=resource/update&id=… выполняется в текущей вкладке.

Изменения: у дефолтного шаблона колонки pagetitle убран target="_blank" в CategoryProductsGrid.vue, обновлён сидер грида category-products, добавлена миграция, обновляющая JSON в ms3_grid_fields для уже развёрнутых установок.

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

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

Связанные Issues

— (issue не заводился)

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

Проверка синтаксиса PHP миграции (php -l). Полное ручное тестирование в менеджере и прогон миграций на стенде рекомендуется ревьюеру.

  • Ручное тестирование
  • Автоматические тесты (PHPStan, ESLint)
  • Тестирование на разных версиях PHP/MODX

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

  • MiniShop3: —
  • MODX: —
  • PHP: >=8.2 (как в composer)

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

До После

Чеклист

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

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

После обновления пакета нужно выполнить миграции Phinx. Если миграцию не запускать, можно вручную убрать target="_blank" в Утилиты → Поля таблицы → Товары категории для колонки названия.

Убран target="_blank" у ссылки названия товара в гриде категории:
дефолтный шаблон в CategoryProductsGrid, сидер grid-config,
миграция для существующих БД.
@Ibochkarev Ibochkarev requested a review from biz87 March 28, 2026 07:11
@biz87
Copy link
Copy Markdown
Member

biz87 commented Mar 28, 2026

Не очень понял зачем

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants