Skip to content

Update Driver.php#47

Open
ytripet wants to merge 1 commit intomasterfrom
auth-patch-permissions-not-working
Open

Update Driver.php#47
ytripet wants to merge 1 commit intomasterfrom
auth-patch-permissions-not-working

Conversation

@ytripet
Copy link
Copy Markdown
Contributor

@ytripet ytripet commented Aug 15, 2022

//if ($resource_permissions !== 0) $role_passed = count($action);
this line in each section seem to just break the thing
so i commented out
and
if ($permission->action == 'all' && in_array($permission->resource, $resource) /*&& $permission->item_id === $resource_id*/) {
seems that our permission checkboxes is not up to date with this code
we only give permission for page type but when the permission is requested it has a resource_id but our permission does not
so we need to bypass that line

//if ($resource_permissions !== 0) $role_passed = count($action);
8:26
this line in each section seem to just break the thing
8:26
so i commented out
8:27
and
8:27
if ($permission->action == 'all' && in_array($permission->resource, $resource) /*&& $permission->item_id === $resource_id*/) {
8:27
seems that our permission checkboxes is not up to date with this code
8:28
we only give permission for page type but when the permission is requested it has a resource_id but our permission does not
8:28
so we need to bypass that line
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant