Skip to content

Commit 10e9f75

Browse files
author
Marius Burkard
committed
Merge branch 'master' into 'master'
fixed error in mail_backup -> all mail-backups would have been deleted with old … …code because of * in filename See merge request !247
2 parents 2dffea3 + 2bc9e12 commit 10e9f75

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

server/lib/classes/cron.d/500-backup_mail.inc.php

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -224,9 +224,8 @@ public function onRunJob() {
224224
$backups = $app->db->queryAllRecords("SELECT * FROM mail_backup WHERE server_id = ?", $conf['server_id']);
225225
if(is_array($backups) && !empty($backups)){
226226
foreach($backups as $backup){
227-
$mail_backup_dir = $backup_dir.'/mail'.$rec['domain_id'];
228-
$mail_backup_file = 'mail'.$rec['mailuser_id'].'_*';
229-
if(!is_file($mail_backup_dir.'/'.$mail_backup_file)){
227+
$mail_backup_dir = $backup_dir.'/mail'.$backup['parent_domain_id'];
228+
if(!is_file($mail_backup_dir.'/'.$backup['filename'])){
230229
$sql = "DELETE FROM mail_backup WHERE server_id = ? AND parent_domain_id = ? AND filename = ?";
231230
$app->db->query($sql, $conf['server_id'], $backup['parent_domain_id'], $backup['filename']);
232231
if($app->db->dbHost != $app->dbmaster->dbHost) $app->dbmaster->query($sql);

0 commit comments

Comments
 (0)