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

[BUGFIX] clear the flushed cache tags (#2505) #2508

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

heifiDR
Copy link

@heifiDR heifiDR commented Jul 26, 2024

After investigating how a solution could look like, I am back to the simple way. Unfortunately I found no hook which can be triggerd after foreaching through each element with prepareCacheFlush(). There is no clean way to get another usefull clear_cacheCmd out of the prepareCacheFlush() method. And there is no way to manipulate the $tagsToClear per hook, to let the core do the cache flushing with the custom tags.

With my change, already flushed cache tags get removed from the array. It's not perfect, because a page with multiple items on it is still flushed multiple times. But the solution avoids the exponential increasing workload.

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