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

Test Setup Failed

Pull Request — master (#1904)

by Anton

created 2020-04-25 19:50 UTC

HostSelector::getByRoles() A

↳ Parent: HostSelector

Complexity

Conditions	5
Paths	8

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	5.0729

Importance

Changes

Metric	Value
cc	5
nc	8
nop	1
dl	0
loc	17
ccs	6
cts	7
cp	0.8571
crap	5.0729
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		public function __construct(HostCollection $hosts)
15		{
16		$this->hosts = $hosts;
17		}
18
19		/**
20		* @param string $hosts
21		* @param string $roles
22		* @return Host[]
23		*/
24	26	public function select($hosts, $roles)
25		{
26	26	if (!empty($hosts)) {
27	26	return $this->getByHostnames($hosts);
28	26	}
29		if (!empty($roles)) {
30		return $this->getByRoles($roles);
31		}
32		return $this->getAll();
33		}
34
35	22	/**
36		* @return Host[]
37	22	*/
38		public function getAll()
39		{
40	22	$hosts = [];
41	18	foreach ($this->hosts as $host) {
42		$hosts[] = $host;
43		}
44	22	return $hosts;
45		}
46	4
47		/**
48	3	* @param string $hostnames
49	1	* @return Host[]
50		*/
51		public function getByHostnames(string $hostnames)
52		{
53		$hostnames = Range::expand(array_map('trim', explode(',', $hostnames)));
54	4	return array_map([$this->hosts, 'get'], $hostnames);
55	3	}
56	1
57		/**
58		* @param string $roles
59	4	* @return Host[]
60		*/
61		public function getByRoles(string $roles)
62		{
63		if (is_string($roles)) {
64	18	$roles = array_map('trim', explode(',', $roles));
65	17	}
66	17
67		$hosts = [];
68		foreach ($this->hosts as $host) {
69		foreach ($host->get('roles', []) as $role) {
70		if (in_array($role, $roles, true)) {
71	20	$hosts[] = $host;
72	1	}
73	1	}
74		}
75
76		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