magento-lts icon indicating copy to clipboard operation
magento-lts copied to clipboard

Allow ACL settings for system config "groups"

Open sreichel opened this issue 1 year ago • 8 comments

Fixed Issues (if relevant)

  1. See https://github.com/OpenMage/magento-lts/pull/3451#issuecomment-1728982076

To add ACL-checks to config groups you have to:

  1. add resource to adminhtml.xml
  2. adds acl_resource node to system.xmls config groups. Eg.
<acl_resource>system/config/sitemap/category</acl_resource>

Example usage: https://github.com/OpenMage/magento-lts/pull/4326/commits/70312f39a748dfd390c78574ae79d43cdc32482e

sreichel avatar Oct 31 '24 23:10 sreichel

A brief description on how to test this would be helpful.

kiatng avatar Nov 10 '24 06:11 kiatng

Example: https://github.com/OpenMage/magento-lts/pull/4326/commits/70312f39a748dfd390c78574ae79d43cdc32482e.

sreichel avatar Nov 10 '24 06:11 sreichel

I am not sure how to disable a user from accessing a field. Can you explain how to test this PR?

kiatng avatar Nov 19 '24 07:11 kiatng

This is not used in core, yet!

I did not want to add acl-resources in that PR.

@empiricompany "asked" for that feature. And imho its very useful for extensions.

To test see linked commit.

sreichel avatar Nov 19 '24 08:11 sreichel

Can you explain how to test this PR?

Is updated description enough?

sreichel avatar Dec 09 '24 09:12 sreichel

Ping @empiricompany

Based on your idea. Please take a minute.

sreichel avatar May 10 '25 03:05 sreichel

This PR does nothingt at the moment, but can be used in 3rd-party code.

To add acl_resourcenode to core, i have to make sure all permission still work. Another PR.

Merged.

sreichel avatar Oct 13 '25 14:10 sreichel