fix(acl): Resource permission descriptions and class_map Resource Type UI - #4
Open
Ibochkarev wants to merge 2 commits into
Open
fix(acl): Resource permission descriptions and class_map Resource Type UI#4Ibochkarev wants to merge 2 commits into
Ibochkarev wants to merge 2 commits into
Conversation
…s#14479) Hide Resource Type combo without class_map (keep class_key as hidden). Align en permission descriptions with soft-delete, publish-only, edit_locked vs steal_locks, and view_document vs Get/GetList.
Collapse Resource Type field to a single conditional xtype, clarify view_document vs Get/GetList policies, and note class_map is not a create-type gate.
Ibochkarev
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Что сделано
Частичный фикс для #14479 по перепроверке 2026 года.
Большинство ключей Resource уже делает то, что ожидают процессоры и дерево. Create / Update / Delete проверяют тип-ключи (
new_weblinkи прочие) при выборе производногоclass_key. Этот PR закрывает оставшиеся UX- и лексикон-пробелы.class_map— панель create/edit, окно create и quick create больше не монтируютmodx-combo-class-derivatives. Они сохраняют скрытыйclass_keyс текущим значением. Это убирает access-denied попап изSystem/Derivatives/GetList. Процессоры всё равно проверяютnew_*/edit_*на POST.publish_documentтолько публикация.delete_documentэто soft-delete (purge_deletedдля перманентного).edit_lockedотносится к Elements, а не к блокировкам ресурсов в сессии.steal_locks/ объектsteal_lockоба отмечены. Тип-ключи указывают, что сидят поверхnew_document.view_documentvsResource/Get/GetListуточнено.class_mapпривязан к комбо Resource Type.Также убран дублирующий скрытый
class_keyрядом с комбо в quick create.Как проверить
class_map
class_key: откройте create/edit ресурса — нет комбо Resource Type, нет ошибки Derivatives/GetList, сохранение держитclass_key.class_map: комбо грузится как раньше.Тип-ключи (уже в процессорах; smoke)
new_weblink: создать WebLink или сменить тип на WebLink → отказ.new_weblink+new_document: те же операции успешны.Лексикон
PHPUnit
8 тестов, exit 0.
Связанные
Refs modxcms#14479
Связанные: modxcms#14467 (trash /
purge_deleted), modxcms#14434 (дерево ресурсов)Совместимость
Только менеджер UI + en лексикон. Другие локали держат старые формулировки до синхронизации Crowdin. Без записи в CHANGELOG.
Оригинал: modxcms#17011