No Code Attached Yet
avatar brianteeman
brianteeman
24 Jan 2023

or I just don't get it

Steps to reproduce the issue

Create a user with Manager permissions
Login with that user and confirm that for com_content there is no Options button
Log back in as a super user and go to the com_content options and change the permissions to give the Manager permission for the options.

image

Log back in as a manager confirm that for com_content there now is an Options button

Expected result

Clicking on the Options button will display the options for the component

Actual result

403 Access denied
image

Additional comments

The button is displayed because the code checks that you gave permission to the component for that user.

if ($user->authorise('core.admin', 'com_content') || $user->authorise('core.options', 'com_content')) {
$toolbar->preferences('com_content');
}

However you dont get access because you do not have permission to access the admin interface of com_config which can only be set in global configuration and would give access to all the components

Either I am missing something or this is a serious bug in the ACL system.

avatar brianteeman brianteeman - open - 24 Jan 2023
avatar joomla-cms-bot joomla-cms-bot - change - 24 Jan 2023
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 24 Jan 2023
avatar joomdonation
joomdonation - comment - 25 Jan 2023

You are not missing anything. There is a bug in com_config ACL check, also reported before here #36273 . I'm going to make a PR to fix this issue in the next few days if no one is faster.

avatar wojtekxtx
wojtekxtx - comment - 25 Jan 2023

@brianteeman Also noticed it
@joomdonation go for it :)

avatar coolcat-creations
coolcat-creations - comment - 31 Jan 2023

I do not know if this is connected but I was also confused that I gave an user Access to create Users but the buttons for this were not shown because the buttons are assigned to user group special. I thought that the buttons should be also shown depending on the general access and not by module acl. ?

avatar Quy
Quy - comment - 31 Jan 2023

Please test PR #39710

avatar wojtekxtx
wojtekxtx - comment - 1 Feb 2023

I thought that the buttons should be also shown depending on the general access and not by module acl. ?

@coolcat-creations your assumption is correct ?

avatar richard67
richard67 - comment - 7 Feb 2023

Which one shall we close as duplicate report? The older #36273 ? Or this one here?

avatar wojtekxtx
wojtekxtx - comment - 7 Feb 2023

@richard67 If it was up to me, Id close #36273 (especially after reading #36273 (comment))

avatar joomdonation joomdonation - close - 7 Feb 2023
avatar joomdonation
joomdonation - comment - 7 Feb 2023

Closing this issue because there is PR #39710 to address it. The PR is not complete yet, but we will try to work together to get the issue fixed.

avatar joomdonation joomdonation - change - 7 Feb 2023
Status New Closed
Closed_Date 0000-00-00 00:00:00 2023-02-07 15:31:23
Closed_By joomdonation

Add a Comment

Login with GitHub to post a comment