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

Make sure it actually clears the cached files, not only the internal template cache #7297

Merged
merged 1 commit into from Mar 2, 2015

Conversation

tsteur
Copy link
Member

@tsteur tsteur commented Feb 25, 2015

This fixes some problems that users reported when manually updating. They followed the manual step by step guide yet still some old twig templates were loaded. Noticed we never deleted/cleared the cached twig files which we do now.

…template cache

This fixes some problems that users reported when manually updating. They followed the manual step by step guide yet still some old twig templates were loaded. Noticed we never deleted/cleared the cached twig files which we do now.
@tsteur tsteur changed the title Make sure it actually clears the cached files, not only the internal tem... Make sure it actually clears the cached files, not only the internal template cache Feb 25, 2015
@tsteur tsteur added the Needs Review PRs that need a code review label Feb 25, 2015
@mattab mattab modified the milestone: Piwik 2.11.2 Feb 25, 2015
@mattab
Copy link
Member

mattab commented Mar 2, 2015

Good catch!

mattab pushed a commit that referenced this pull request Mar 2, 2015
Make sure it actually clears the cached files, not only the internal template cache
@mattab mattab merged commit 55dad08 into master Mar 2, 2015
diosmosis pushed a commit that referenced this pull request Mar 6, 2015
@tsteur tsteur deleted the twig_clear_cache_fix branch March 8, 2015 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review PRs that need a code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants