Skip to content

fix(mgr): пустая карточка товара без resourcegroup_resource_list (#652) - #653

Open
Ibochkarev wants to merge 1 commit into
betafrom
fix/issue-652-product-update-access-tab
Open

fix(mgr): пустая карточка товара без resourcegroup_resource_list (#652)#653
Ibochkarev wants to merge 1 commit into
betafrom
fix/issue-652-product-update-access-tab

Conversation

@Ibochkarev

Copy link
Copy Markdown
Member

Описание

Без права resourcegroup_resource_list ядро MODX не добавляет вкладку «Права доступа». ms3.panel.UpdateProduct.getFields() всё равно делал tabs.push(accessPermissionsTab) с undefined, из‑за чего addChangeEvent падал и форма товара оставалась пустой.

Правка как в category/update.js: пушим вкладки settings/access только если они есть.

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

  • Исправление бага (non-breaking change)

Связанные Issues

Closes #652

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

# red (unguarded push) → exit 1
# green (with guards) → exit 0
cd core/components/minishop3
php tests/ProductUpdateAccessTabGuardTest.php
# EXIT:0
  • Автоматические тесты (smoke ProductUpdateAccessTabGuardTest, red-green)
  • Ручное тестирование менеджером без resourcegroup_resource_list (нужна отдельная политика на стенде)

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

  • MiniShop3: ветка fix/issue-652-product-update-access-tab
  • PHP: 8.4 (smoke)

Чеклист

  • Код соответствует стилю проекта
  • Изменения не ломают существующую функциональность
  • Обновлён CHANGELOG.md — не в этом PR (релизный процесс)

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

Права ACL не менялись: только защита от undefined в Ext items. Обходной путь из issue (выдать resourcegroup_resource_list) больше не нужен для открытия карточки.

Match category/update.js so missing access-permissions tab does not
leave undefined items and blank the form without resourcegroup_resource_list.
@Ibochkarev
Ibochkarev requested a review from biz87 September 7, 2026 03:56
@Ibochkarev Ibochkarev self-assigned this Sep 7, 2026
@Ibochkarev Ibochkarev added the bug Something isn't working label Sep 7, 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] у пользователя без права resourcegroup_resource_list карточка товара открывается пустой

1 participant