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

RelationshipCache - Set 'get' permissions #23684

Merged

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented Jun 3, 2022

Overview

This gives lesser-permissioned users access to the RelationshipCache entity, for Search Kit.

Before

Non-admins could not see related contacts in SearchKit, even if they had permission to view those contacts.

After

Fixed. Permissions for RelationshipCache are now the same as for Relationship.

Technical Details

This duplicates the logic used to check permissions for relationships in the RelationshipCache entity.

…tionship' entity

This gives lesser-permissioned users access to the RelationshipCache entity, for Search Kit.
@civibot
Copy link

civibot bot commented Jun 3, 2022

(Standard links)

@awestuk
Copy link
Contributor

awestuk commented Jun 3, 2022

Tested and works great - non-admins can now access Related Contacts fine, when they have the administer search_kit permission.

@eileenmcnaughton eileenmcnaughton merged commit 1db72bc into civicrm:master Jun 4, 2022
@eileenmcnaughton eileenmcnaughton deleted the relationshipCachePermissions branch June 4, 2022 02:30
@eileenmcnaughton
Copy link
Contributor

thanks for testing @awestuk

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants