magento-lts
magento-lts copied to clipboard
Allow ACL settings for system config "groups"
Fixed Issues (if relevant)
- See https://github.com/OpenMage/magento-lts/pull/3451#issuecomment-1728982076
To add ACL-checks to config groups you have to:
- add resource to
adminhtml.xml - adds
acl_resourcenode 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
A brief description on how to test this would be helpful.
Example: https://github.com/OpenMage/magento-lts/pull/4326/commits/70312f39a748dfd390c78574ae79d43cdc32482e.
I am not sure how to disable a user from accessing a field. Can you explain how to test this PR?
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.
Can you explain how to test this PR?
Is updated description enough?
Quality Gate passed
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
Ping @empiricompany
Based on your idea. Please take a minute.
Quality Gate passed
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
Quality Gate passed
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
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.