HostSelector::getByRoles() - Code Metrics - Inspection of "Deployer v7" - deployphp/deployer - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — master (#1904)

by Anton

created 2020-04-25 19:54 UTC

HostSelector::getByRoles() A

↳ Parent: HostSelector

Complexity

Conditions	5
Paths	8

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	5

Importance

Changes

Metric	Value
cc	5
nc	8
nop	1
dl	0
loc	17
ccs	9
cts	9
cp	1
crap	5
rs	9.3888
c	0
b	0
f	0

<?php
/* (c) Anton Medvedev <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Deployer\Host;

class HostSelector
{
    private $hosts;

    public function __construct(HostCollection $hosts)
    {
        $this->hosts = $hosts;
    }

    /**
     * @param string $hosts
     * @param string $roles
     * @return Host[]
     */
    public function select($hosts, $roles)
    {
        if (!empty($hosts)) {
            return $this->getByHostnames($hosts);
        }
        if (!empty($roles)) {
            return $this->getByRoles($roles);
        }
        return $this->getAll();
    }

    /**
     * @return Host[]
     */
    public function getAll()
    {
        $hosts = [];
        foreach ($this->hosts as $host) {
            $hosts[] = $host;
        }
        return $hosts;
    }

    /**
     * @param string $hostnames
     * @return Host[]
     */
    public function getByHostnames(string $hostnames)
    {
        $hostnames = Range::expand(array_map('trim', explode(',', $hostnames)));
        return array_map([$this->hosts, 'get'], $hostnames);
    }

    /**
     * @param string $roles
     * @return Host[]
     */
    public function getByRoles(string $roles)
    {
        if (is_string($roles)) {
            $roles = array_map('trim', explode(',', $roles));
        }

        $hosts = [];
        foreach ($this->hosts as $host) {
            foreach ($host->get('roles', []) as $role) {
                if (in_array($role, $roles, true)) {
                    $hosts[] = $host;
                }
            }
        }

        return $hosts;
    }
}


1		<?php
2		/* (c) Anton Medvedev <[email protected]>
3		*
4		* For the full copyright and license information, please view the LICENSE
5		* file that was distributed with this source code.
6		*/
7
8		namespace Deployer\Host;
9
10		class HostSelector
11		{
12		private $hosts;
13
14	5	public function __construct(HostCollection $hosts)
15		{
16	5	$this->hosts = $hosts;
17	5	}
18
19		/**
20		* @param string $hosts
21		* @param string $roles
22		* @return Host[]
23		*/
24		public function select($hosts, $roles)
25		{
26		if (!empty($hosts)) {
27		return $this->getByHostnames($hosts);
28		}
29		if (!empty($roles)) {
30		return $this->getByRoles($roles);
31		}
32		return $this->getAll();
33		}
34
35		/**
36		* @return Host[]
37		*/
38	1	public function getAll()
39		{
40	1	$hosts = [];
41	1	foreach ($this->hosts as $host) {
42	1	$hosts[] = $host;
43		}
44	1	return $hosts;
45		}
46
47		/**
48		* @param string $hostnames
49		* @return Host[]
50		*/
51	1	public function getByHostnames(string $hostnames)
52		{
53	1	$hostnames = Range::expand(array_map('trim', explode(',', $hostnames)));
54	1	return array_map([$this->hosts, 'get'], $hostnames);
55		}
56
57		/**
58		* @param string $roles
59		* @return Host[]
60		*/
61	2	public function getByRoles(string $roles)
62		{
63	2	if (is_string($roles)) {
64	2	$roles = array_map('trim', explode(',', $roles));
65		}
66
67	2	$hosts = [];
68	2	foreach ($this->hosts as $host) {
69	2	foreach ($host->get('roles', []) as $role) {
70	1	if (in_array($role, $roles, true)) {
71	1	$hosts[] = $host;
72		}
73		}
74		}
75
76	2	return $hosts;
77		}
78		}
79

deployphp / deployer

GitHub Access Token became invalid

Pull Request — master (#1904)

HostSelector::getByRoles() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like