Old backup files not being deleted

It should delete them, have you tried manually deleting them (through the CYAN Backup UI)?

It might be a permissions problem, double check the file permissions and make sure the web server user has permissions to delete the files.

@greg Ross, although I put the backup directory to my home directory, it worked if I delete the backups manually on the CYAN interface. Then I tested moving the backup directory to my WordPress subfolders where the WordPress users definitely have permission to delete files, I got the same result. That is, when I create multiple backups in a roll manually, the older backups won’t be deleted automatically even though I set up the Number of Backups to be 1. Did I miss anything in the setting? Thanks!

That should be everything you have to do, I’ll take a look and see if there’s a bug in it somewhere.

In the mean time, try setting it to 2 or 3 and see if it works.

Sorry, I don’t have enough space to test backup with 3 copies on my server. 2 copies have been very close to the upper limit. I appreciate your efforts!

Ok, I found the issue, line 1461 of cyan-backup.php is:

if (count($backup_files) > $number && $number > 1) {

but should be:

if (count($backup_files) > $number && $number >= 1) {

So there has to be at least 2 backups, otherwise it won’t prune them.

@gregross, I replaced the line, and it works like a charm! Thank you very much for taking care of this issue.

Just one side question: I am considering to set up a remote server to store all the backups offline. I can set up the remote path but does the “Number of Backups to keep” in the “Storage Maintenance” submenu also refer to the copies on the remote server? Or, in other words, the old remote copies will also be deleted when a new copy arrives?

Also, there is a small typo on the interface of “Remote storage” submenu: “You many use the follow place holders: %m = month (01-12), %d = day (01-31), %Y = year (XXXX), %M = month (Jan…Dec), %F = month (January…December)”. The word “many” should be “may”, I think.

Thanks!
Qi

No, management of remote storage is not part of CYAN Backup, it just sends the backup to the remote location it’s up to the remote server to handle any removal of old backup files.

Thanks for the typo, I’ll fix it.