Skip to content
This repository has been archived by the owner on Feb 23, 2019. It is now read-only.

A tweak to the previous Minify Notification Update #54 #63

Merged
merged 2 commits into from
Sep 17, 2016

Conversation

amiga-500
Copy link
Collaborator

I had forgotten to increment the minify version when the minify cache was also cleared. I had only incremented it when the page cache was cleared. Although in most cases it is the page cache that when cleared (say, due to a plugin removal) it should be incremented, but for cases where the user adjusts their minify settings to say smaller file length sizes, it can potentially result in different minify file names and so its best to also increment the version variable for this case too -- i forgot about that.

I also moved the minify version update function into the define.php file so the function could be accessed from one central location instead of being duplicated twice (for pg cache class and minify class).

@amiga-500 amiga-500 closed this Sep 17, 2016
@amiga-500 amiga-500 reopened this Sep 17, 2016
@nigrosimone nigrosimone merged commit cea3039 into master Sep 17, 2016
@nigrosimone nigrosimone deleted the Minify-Notification-Update-2 branch September 17, 2016 04:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants