Ftp::__construct() - Code Metrics - Inspection of "WIP: Remote cleanup" - sebastianfeldmann/phpbu - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#140)

unknown

created 2018-05-08 07:02 UTC

Ftp::__construct() A

↳ Parent: Ftp

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	5
ccs	0
cts	5
cp	0
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	3
nc	1
nop	2
crap	2

<?php
namespace phpbu\App\Backup\Collector;

use League\Flysystem\Filesystem;
use phpbu\App\Backup\Collector;
use phpbu\App\Backup\Target;

class Ftp extends Collector
{
    /**
     * @var Filesystem
     */
    private $flySystem;

    /**
     * @var string
     */
    private $path;

    /**
     * Ftp constructor.
     *
     * @param \phpbu\App\Backup\Target     $target
     * @param \League\Flysystem\Filesystem $flySystem
     */
    public function __construct(Target $target, Filesystem $flySystem)
    {
        $this->flySystem = $flySystem;
        $this->setUp($target);
    }

    /**
     * Get all created backups.
     *
     * @return \phpbu\App\Backup\File[]
     */
    public function getBackupFiles(): array
    {
        $files = $this->flySystem->listContents();
        foreach ($files as $file) {
            if ($file['type'] != 'file') {
                continue;
            }
            // skip currently created backup
            if ($file['basename'] == $this->target->getFilename()) {
                continue;
            }
            if ($this->isFilenameMatch($file['basename'])) {
                $this->files[] = new \phpbu\App\Backup\File\Ftp($this->flySystem, $file);
            }
        }
        return $this->files;
    }
}


1			<?php
2			namespace phpbu\App\Backup\Collector;
3
4			use League\Flysystem\Filesystem;
5			use phpbu\App\Backup\Collector;
6			use phpbu\App\Backup\Target;
7
8			class Ftp extends Collector
9			{
10			/**
11			* @var Filesystem
12			*/
13			private $flySystem;
14
15			/**
16			* @var string
17			*/
18			private $path;
19
20			/**
21			* Ftp constructor.
22			*
23			* @param \phpbu\App\Backup\Target $target
24			* @param \League\Flysystem\Filesystem $flySystem
25			*/
26			public function __construct(Target $target, Filesystem $flySystem)
27			{
28			$this->flySystem = $flySystem;
29			$this->setUp($target);
30			}
31
32			/**
33			* Get all created backups.
34			*
35			* @return \phpbu\App\Backup\File[]
36			*/
37			public function getBackupFiles(): array
38			{
39			$files = $this->flySystem->listContents();
40			foreach ($files as $file) {
41			if ($file['type'] != 'file') {
42			continue;
43			}
44			// skip currently created backup
45			if ($file['basename'] == $this->target->getFilename()) {
46			continue;
47			}
48			if ($this->isFilenameMatch($file['basename'])) {
49			$this->files[] = new \phpbu\App\Backup\File\Ftp($this->flySystem, $file);
50			}
51			}
52			return $this->files;
53			}
54			}
55

sebastianfeldmann / phpbu

Pull Request — master (#140)

Ftp::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like