If a module has more than one permissions, the last permission listed for that module will not have its permissions turned off for all roles.

To reproduce:

1) Find a module with more than one permission
2) Check off all permissions for all roles in the permission
3) Save the permissions
4) Hide the module found in #1
5) Visit permissions page
6) Save permissions
7) Unhide the module
8) Visit the permission page and notice the last permission will have all permission checked off for all roles

CommentFileSizeAuthor
#1 save_perms.patch1.64 KBSteve Dondley

Comments

Steve Dondley’s picture

Title: Not all permissions are unset » Permissions not save correctly
Status: Active » Needs review
StatusFileSize
new1.64 KB

Upon closer investigation, it doesn't look they were trying to get unset for all roles. The original permissions set for the module were supposed to get saved. However, that wasn't working either. Here is a patch.

Steve Dondley’s picture

Title: Permissions not save correctly » Permissions not saved correctly
Assigned: Unassigned » Steve Dondley
robin monks’s picture

Status: Needs review » Fixed

Patch applied to 5.x.

Robin

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.