Skip to content
This repository has been archived by the owner on Aug 18, 2024. It is now read-only.

Remove the static cache in OgAccess::userAccess(). #655

Merged
merged 3 commits into from
Jul 7, 2020
Merged

Remove the static cache in OgAccess::userAccess(). #655

merged 3 commits into from
Jul 7, 2020

Conversation

pfrenssen
Copy link
Contributor

Fixes #654

@pfrenssen
Remove the static cache in OgAccess::userAccess().
76503b5 
This cache has little value since all the methods that we call that access the
database already have a static cache.
@pfrenssen pfrenssen added this to the 8.x-1.0-alpha6 milestone Jul 7, 2020
@pfrenssen pfrenssen merged commit 899a4c1 into 8.x-1.x Jul 7, 2020
@pfrenssen pfrenssen deleted the remove-user-access-cache branch July 7, 2020 14:56
@pfrenssen pfrenssen removed their assignment Jul 7, 2020
@pfrenssen
Copy link
Contributor Author

Thanks for reviewing! I am planning to clean up and improve OgAccess over the coming days. It needs some tender loving care :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@MPParsley MPParsley MPParsley approved these changes

Assignees
No one assigned
Labels
Clean-up Drupal 8
Projects
None yet
Milestone
8.x-1.0-alpha6
Development

Successfully merging this pull request may close these issues.

Remove static cache in OgAccess::userAccess()
2 participants
@pfrenssen @MPParsley