Config::sanitize() - Code Metrics - Inspection of "Initial commit" - fatindeed/gitlab-webhookd - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 4bca98...db059b )

by James

created 2019-07-23 10:18 UTC

Config::sanitize() A

↳ Parent: Config

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

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

<?php

namespace App;

use ArrayObject;
use InvalidArgumentException;
use Symfony\Component\Yaml\Yaml;

class Config extends ArrayObject
{
    /**
     * Construct a new config
     *
     * @param  string  $filename
     * @return void
     */
    public function __construct(array $input = [])
    {
        parent::__construct($input + [
            // 'workdir' => getcwd(),
            'port' => 9501,
            'events' => [],
        ]);
        $this->sanitize();
    }

    /**
     * load config.yaml
     *
     * @param  string  $filename
     * @return void
     */
    public function loadYaml(string $filename)
    {
        $config = Yaml::parseFile($filename);
        $this->exchangeArray($config + $this->getArrayCopy());
        $this->sanitize();
    }

    /**
     * Sanitize config values.
     * 
     * @return void
     * 
     * @throws \InvalidArgumentException
     */
    private function sanitize(): void
    {
        // Well-known ports: 0 to 1023
        // Registered/user ports: 1024 to 49151
        // Dynamic/private ports: 49152 to 65535
        $port = filter_var($this['port'], FILTER_VALIDATE_INT, [
            'options' => ['min_range' => 0, 'max_range' => 49151]
        ]);
        if ($port === false) {
            throw new InvalidArgumentException('Invalid port: '.$this['port'].'.', 1001);
        }
        $this['port'] = $port;
    }
}

1		<?php
2
3		namespace App;
4
5		use ArrayObject;
6		use InvalidArgumentException;
7		use Symfony\Component\Yaml\Yaml;
8
9		class Config extends ArrayObject
10		{
11		/**
12		* Construct a new config
13		*
14		* @param string $filename
15		* @return void
16		*/
17	2	public function __construct(array $input = [])
18		{
19	2	parent::__construct($input + [
20		// 'workdir' => getcwd(),
21	2	'port' => 9501,
22		'events' => [],
23		]);
24	2	$this->sanitize();
25	1	}
26
27		/**
28		* load config.yaml
29		*
30		* @param string $filename
31		* @return void
32		*/
33	1	public function loadYaml(string $filename)
34		{
35	1	$config = Yaml::parseFile($filename);
36	1	$this->exchangeArray($config + $this->getArrayCopy());
37	1	$this->sanitize();
38	1	}
39
40		/**
41		* Sanitize config values.
42		*
43		* @return void
44		*
45		* @throws \InvalidArgumentException
46		*/
47	2	private function sanitize(): void
48		{
49		// Well-known ports: 0 to 1023
50		// Registered/user ports: 1024 to 49151
51		// Dynamic/private ports: 49152 to 65535
52	2	$port = filter_var($this['port'], FILTER_VALIDATE_INT, [
53	2	'options' => ['min_range' => 0, 'max_range' => 49151]
54		]);
55	2	if ($port === false) {
56	1	throw new InvalidArgumentException('Invalid port: '.$this['port'].'.', 1001);
57		}
58	1	$this['port'] = $port;
59		}
60		}

fatindeed / gitlab-webhookd

Push — master ( 4bca98...db059b )

Config::sanitize() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like