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

Clear Cache not clearing cache directories #868

Closed
raamdev opened this Issue Dec 28, 2016 · 5 comments

Comments

Projects
None yet
3 participants
@raamdev
Copy link
Contributor

raamdev commented Dec 28, 2016

We have several reports of Comet Cache Lite v161221 not deleting directories in wp-content/cache/comet-cache/cache/http/example-com/* when you click the 'Clear Cache' button. Cache files get deleted, but any and all directories remain. Comet Cache Lite v160917 and v160706 are also reported as having this issue.

I have not been able to reproduce this, but there are at least three confirmed reports of this issue:

Users have double-checked permissions and even installed other caching plugins to see if those plugins delete directories fine (they do). Please see the above links for the details of each report.

@renzms

This comment has been minimized.

Copy link

renzms commented Dec 28, 2016

@raamdev

Unable to reproduce error

Tested using Comet Cache Lite v161221, with single site and multisite.

Using PHP 5.4/7

Comet Cache Lite v161221 clears files and directories properly.

screen shot 2016-12-29 at 12 58 31 am

screen shot 2016-12-29 at 1 04 23 am

@eurobank

This comment has been minimized.

Copy link

eurobank commented Dec 29, 2016

Doesn't work for me also. CloudLinux Shared Hosting with PHP 7, using LiteSpeed webserver (if this makes any difference).

Directories remain.

Tested with other cache plugins and works as expected.

raamdev referenced this issue in wpsharks/comet-cache-pro Dec 30, 2016

jaswsinc
@raamdev

This comment has been minimized.

Copy link
Contributor

raamdev commented Dec 30, 2016

After extensive tests, we discovered the bug that was causing this issue; see wpsharks/comet-cache-pro@3db638d. A fix for this issue will go out with the next release.

@raamdev raamdev removed the needs testing label Dec 30, 2016

@raamdev raamdev added this to the Next Release milestone Dec 30, 2016

@renzms

This comment has been minimized.

Copy link

renzms commented Feb 3, 2017

Confirmed working in Comet Cache™ Pro v170201-RC

Clear button clears out all empty directories

@raamdev

This comment has been minimized.

Copy link
Contributor

raamdev commented Feb 20, 2017

Comet Cache v170220 has been released and includes changes from this GitHub Issue. See the v170220 announcement for further details.


This issue will now be locked to further updates. If you have something to add related to this GitHub Issue, please open a new GitHub Issue and reference this one (#868).

@raamdev raamdev closed this Feb 20, 2017

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