Skip to content

Commit 370e447

Browse files
Add cleanup debug output
1 parent ce44598 commit 370e447

File tree

2 files changed

+17
-7
lines changed

2 files changed

+17
-7
lines changed

src/Backup/Cleaner/Abstraction.php

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
use phpbu\App\Result;
88

99
/**
10-
* Cleaner Abstraction.
10+
* Cleaner Abstraction
1111
*
1212
* @package phpbu
1313
* @subpackage Backup
@@ -20,7 +20,14 @@
2020
abstract class Abstraction
2121
{
2222
/**
23-
* Cleanup your backup directory.
23+
* Backup Result to handle events and IO
24+
*
25+
* @var \phpbu\App\Result
26+
*/
27+
protected $result;
28+
29+
/**
30+
* Cleanup your backup directory
2431
*
2532
* @see \phpbu\App\Backup\Cleanup::cleanup()
2633
* @param \phpbu\App\Backup\Target $target
@@ -30,6 +37,7 @@ abstract class Abstraction
3037
*/
3138
public function cleanup(Target $target, Collector $collector, Result $result)
3239
{
40+
$this->result = $result;
3341
foreach ($this->getFilesToDelete($target, $collector) as $file) {
3442
if (!$file->isWritable()) {
3543
throw new Exception(sprintf('can\'t delete file: %s', $file->getPathname()));
@@ -40,7 +48,7 @@ public function cleanup(Target $target, Collector $collector, Result $result)
4048
}
4149

4250
/**
43-
* Simulate the cleanup execution.
51+
* Simulate the cleanup execution
4452
*
4553
* @param \phpbu\App\Backup\Target $target
4654
* @param \phpbu\App\Backup\Collector $collector
@@ -55,7 +63,7 @@ public function simulate(Target $target, Collector $collector, Result $result)
5563
}
5664

5765
/**
58-
* Return list of files to delete.
66+
* Return list of files to delete
5967
*
6068
* @param \phpbu\App\Backup\Target $target
6169
* @param \phpbu\App\Backup\Collector $collector

src/Backup/Cleaner/Outdated.php

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,14 +29,14 @@ class Outdated extends Abstraction implements Simulator
2929
protected $offsetRaw;
3030

3131
/**
32-
* Offset in seconds.
32+
* Offset in seconds
3333
*
3434
* @var integer
3535
*/
3636
protected $offsetSeconds;
3737

3838
/**
39-
* Setup the Cleaner.
39+
* Setup the Cleaner
4040
*
4141
* @see \phpbu\App\Backup\Cleanup::setup()
4242
* @param array $options
@@ -57,7 +57,7 @@ public function setup(array $options)
5757
}
5858

5959
/**
60-
* Return list of files to delete.
60+
* Return list of files to delete
6161
*
6262
* @param \phpbu\App\Backup\Target $target
6363
* @param \phpbu\App\Backup\Collector $collector
@@ -68,9 +68,11 @@ protected function getFilesToDelete(Target $target, Collector $collector)
6868
$minTime = time() - $this->offsetSeconds;
6969
$files = $collector->getBackupFiles();
7070
$delete = [];
71+
$this->result->debug(' found ' . count($files) . ' backup files');
7172

7273
/** @var \phpbu\App\Backup\File\Local $file */
7374
foreach ($files as $file) {
75+
$this->result->debug(' checking ' . $file->getFilename() . ' ' . date('Y.m.d H:i:s', $file->getMTime()));
7476
// last mod date < min date? delete!
7577
if ($file->getMTime() < $minTime) {
7678
$delete[] = $file;

0 commit comments

Comments
 (0)