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

User cache not flushed on model save #1050

Closed
lcharette opened this issue Nov 14, 2019 · 0 comments
Closed

User cache not flushed on model save #1050

lcharette opened this issue Nov 14, 2019 · 0 comments
Labels
confirmed bug Something isn't working

Comments

@lcharette
Copy link
Member

This means all changes to the CurrentIssue are not displayed until cache expires or is manually flushed.

To replicate issue:

Go to /account/settings and change the current user first name. The change won't be displayed until cache is cleared (default to 2 mins).

Can fix by setting the cache.user.delay to 0 temporarily.

This should be done automatically using the following event :
https://github.com/userfrosting/UserFrosting/blob/master/app/sprinkles/account/src/Database/Models/Events/DeleteUserCacheEvent.php#L30

Can't be replicated in automated testing as caching is disabled by default...

N.B.: Old cache was 120 minutes, but Laravel cache remember method changed between 5.4 and 5.8 from minutes to seconds. So now it's 120 seconds (2 mins). This two need to be addressed

@lcharette lcharette added the confirmed bug Something isn't working label Nov 14, 2019
@lcharette lcharette added this to the 4.3.x milestone Nov 14, 2019
lcharette added a commit to lcharette/UserFrosting that referenced this issue Nov 14, 2019
lcharette added a commit that referenced this issue Nov 14, 2019
Fix for #1050 - User cache not flushed on model save
@lcharette lcharette removed this from the 4.5.x milestone Apr 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
confirmed bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant