Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"User Group Access to Context" window displaying incorrect permissions when reopened #16386

Open
muzzwood opened this issue Feb 9, 2023 · 0 comments · May be fixed by #16469
Open

"User Group Access to Context" window displaying incorrect permissions when reopened #16386

muzzwood opened this issue Feb 9, 2023 · 0 comments · May be fixed by #16469
Assignees
Labels
bug The issue in the code or project, which should be addressed.

Comments

@muzzwood
Copy link
Contributor

muzzwood commented Feb 9, 2023

Bug report

Summary

When editing a user group's context access policy, if you select a new policy from the dropdown, then close the window without saving, and reopen it, the "Permissions in selected policy" field is not refreshed and still shows the permissions of the policy you changed the dropdown to.

Step to reproduce

Go to ACLs -> Edit user group -> Access Permissions tab

Observed behavior

Screenshot 2023-02-09 at 09-50-58 User Group test MODX 3

For example, in the screenshot, "Load Only" was initially set. I opened the window and changed it to "Load, List and View". The permissions for the new policy were shown. I then closed the window without saving, and opened it again.
The dropdown correctly shows Load Only but the permissions for Load, List and View are still rendered.

Expected behavior

The permissions rendered should match the selected policy when opening the window.

Environment

MODX 3.x git

@muzzwood muzzwood added the bug The issue in the code or project, which should be addressed. label Feb 9, 2023
@smg6511 smg6511 self-assigned this Sep 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug The issue in the code or project, which should be addressed.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants