access check could be much better

Created on 9 February 2016, over 8 years ago
Updated 1 February 2023, over 1 year ago

Note: I am filing this based on reading the code. It is not tested.

return AccessResult::allowedIf(!$field->isEmpty() && $field->access('view', $account));

could be

return AccessResult::allowedIf(!$field->isEmpty())->andIf($field->access('view', $account, TRUE));

📌 Task
Status

Fixed

Version

2.0

Component

Code

Created by

🇨🇦Canada chx

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024