chg: [ACL] fix permissions for org admins
- also, fix a bug with the simple permissions being ignoredpull/79/head
parent
dad310f434
commit
5483357e1c
|
@ -145,24 +145,24 @@ class ACLComponent extends Component
|
||||||
'view' => ['*']
|
'view' => ['*']
|
||||||
],
|
],
|
||||||
'SharingGroups' => [
|
'SharingGroups' => [
|
||||||
'add' => ['perm_admin'],
|
'add' => ['perm_org_admin'],
|
||||||
'addOrg' => ['perm_admin'],
|
'addOrg' => ['perm_org_admin'],
|
||||||
'delete' => ['perm_admin'],
|
'delete' => ['perm_org_admin'],
|
||||||
'edit' => ['perm_admin'],
|
'edit' => ['perm_org_admin'],
|
||||||
'index' => ['*'],
|
'index' => ['*'],
|
||||||
'listOrgs' => ['*'],
|
'listOrgs' => ['*'],
|
||||||
'removeOrg' => ['perm_admin'],
|
'removeOrg' => ['perm_org_admin'],
|
||||||
'view' => ['*']
|
'view' => ['*']
|
||||||
],
|
],
|
||||||
'Users' => [
|
'Users' => [
|
||||||
'add' => ['perm_admin'],
|
'add' => ['perm_org_admin'],
|
||||||
'delete' => ['perm_admin'],
|
'delete' => ['perm_org_admin'],
|
||||||
'edit' => ['*'],
|
'edit' => ['*'],
|
||||||
'index' => ['perm_admin'],
|
'index' => ['perm_org_admin'],
|
||||||
'login' => ['*'],
|
'login' => ['*'],
|
||||||
'logout' => ['*'],
|
'logout' => ['*'],
|
||||||
'register' => ['*'],
|
'register' => ['*'],
|
||||||
'toggle' => ['perm_admin'],
|
'toggle' => ['perm_org_admin'],
|
||||||
'view' => ['*']
|
'view' => ['*']
|
||||||
]
|
]
|
||||||
);
|
);
|
||||||
|
@ -290,6 +290,12 @@ class ACLComponent extends Component
|
||||||
if ($allConditionsMet) {
|
if ($allConditionsMet) {
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
} else {
|
||||||
|
foreach ($this->aclList[$controller][$action] as $permission) {
|
||||||
|
if ($this->user['role'][$permission]) {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return false;
|
return false;
|
||||||
|
|
Loading…
Reference in New Issue