Worker::__construct() - Code Metrics - Inspection of "Methods of Wrapper class marked as deprecated, all..." - EvilFreelancer/easyrsa-php - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( ce1ae9...e9c52d )

by Mr

created 2020-07-07 22:37 UTC

Worker::__construct() A

↳ Parent: Worker

Complexity

Conditions	3
Paths	4

Size

Total Lines	12
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	3
eloc	5
c	1
b	0
f	0
nc	4
nop	1
dl	0
loc	12
rs	10

<?php

declare(strict_types=1);

namespace EasyRSA;

use EasyRSA\Interfaces\ConfigInterface;
use EasyRSA\Interfaces\WorkerInterface;
use RuntimeException;

class Worker implements WorkerInterface
{
    /**
     * @var \EasyRSA\Interfaces\ConfigInterface
     */
    protected ConfigInterface $config;

    /**
     * If need to enable debug mode
     */
    public bool $dryRun = false;

    /**
     * Wrapper constructor, need configuration for normal usage
     *
     * @param \EasyRSA\Interfaces\ConfigInterface|array $config
     *
     * @throws \RuntimeException
     */
    public function __construct($config = [])
    {
        if (is_array($config)) {
            $config = new Config($config);
        }

        if ($config instanceof ConfigInterface) {

            $this->config = $config;
        }

        // Must be set to enable ability execute shell scripts in background
        putenv("EASYRSA_PKI={$this->config->get('certs')}");
    }

    /**
     * @return \EasyRSA\Interfaces\ConfigInterface
     */
    public function getConfig(): ConfigInterface
    {
        return $this->config;
    }

    /**
     * {@inheritdoc}
     */
    public function getContent(string $filename): ?string
    {
        switch ($filename) {
            case 'ca.crt':
            case 'dh.pem':
                $path = $this->config->get('certs') . '/' . $filename;
                break;
            default:
                $ext = pathinfo($filename, PATHINFO_EXTENSION);
                switch ($ext) {
                    case 'crt':
                        $path = $this->config->get('certs') . '/issued/' . $filename;
                        break;
                    case 'key':
                        $path = $this->config->get('certs') . '/private/' . $filename;
                        break;
                    case 'req':
                        $path = $this->config->get('certs') . '/reqs/' . $filename;
                        break;
                    default:
                        return null;
                        break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
                }
                break;
        }

        return file_get_contents($path);
    }

    /**
     * {@inheritdoc}
     *
     * @throws \RuntimeException Of folder with certs unable to create or if is not a folder
     */
    public function exec(string $cmd): array
    {
        $command = $this->config->get('scripts') . '/easyrsa3/easyrsa --batch ' . $cmd;

        // In dry run mode need just return command without real execution
        if ($this->dryRun) {
            $result = [$command];
        } else {
            // Create folder if not exist
            if (!mkdir($this->config->get('certs'), 0755, true) && !is_dir($this->config->get('certs'))) {
                throw new RuntimeException("Folder \"{$this->config->get('certs')}\" can't be created");
            }

            chdir($this->config->get('certs'));
            exec($command, $result);
        }

        return $result;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace EasyRSA;
6
7			use EasyRSA\Interfaces\ConfigInterface;
8			use EasyRSA\Interfaces\WorkerInterface;
9			use RuntimeException;
10
11			class Worker implements WorkerInterface
12			{
13			/**
14			* @var \EasyRSA\Interfaces\ConfigInterface
15			*/
16			protected ConfigInterface $config;
17
18			/**
19			* If need to enable debug mode
20			*/
21			public bool $dryRun = false;
22
23			/**
24			* Wrapper constructor, need configuration for normal usage
25			*
26			* @param \EasyRSA\Interfaces\ConfigInterface\|array $config
27			*
28			* @throws \RuntimeException
29			*/
30			public function __construct($config = [])
31			{
32			if (is_array($config)) {
33			$config = new Config($config);
34			}
35
36			if ($config instanceof ConfigInterface) {
			0 ignored issues – show introduced 2020-07-07 22:45 UTC by Report Bug Copy Issue Report `$config` is always a sub-type of `EasyRSA\Interfaces\ConfigInterface`. Loading history...
37			$this->config = $config;
38			}
39
40			// Must be set to enable ability execute shell scripts in background
41			putenv("EASYRSA_PKI={$this->config->get('certs')}");
42			}
43
44			/**
45			* @return \EasyRSA\Interfaces\ConfigInterface
46			*/
47			public function getConfig(): ConfigInterface
48			{
49			return $this->config;
50			}
51
52			/**
53			* {@inheritdoc}
54			*/
55			public function getContent(string $filename): ?string
56			{
57			switch ($filename) {
58			case 'ca.crt':
59			case 'dh.pem':
60			$path = $this->config->get('certs') . '/' . $filename;
61			break;
62			default:
63			$ext = pathinfo($filename, PATHINFO_EXTENSION);
64			switch ($ext) {
65			case 'crt':
66			$path = $this->config->get('certs') . '/issued/' . $filename;
67			break;
68			case 'key':
69			$path = $this->config->get('certs') . '/private/' . $filename;
70			break;
71			case 'req':
72			$path = $this->config->get('certs') . '/reqs/' . $filename;
73			break;
74			default:
75			return null;
76			break;
			0 ignored issues – show Unused Code introduced 2020-07-04 05:20 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
77			}
78			break;
79			}
80
81			return file_get_contents($path);
82			}
83
84			/**
85			* {@inheritdoc}
86			*
87			* @throws \RuntimeException Of folder with certs unable to create or if is not a folder
88			*/
89			public function exec(string $cmd): array
90			{
91			$command = $this->config->get('scripts') . '/easyrsa3/easyrsa --batch ' . $cmd;
92
93			// In dry run mode need just return command without real execution
94			if ($this->dryRun) {
95			$result = [$command];
96			} else {
97			// Create folder if not exist
98			if (!mkdir($this->config->get('certs'), 0755, true) && !is_dir($this->config->get('certs'))) {
99			throw new RuntimeException("Folder \"{$this->config->get('certs')}\" can't be created");
100			}
101
102			chdir($this->config->get('certs'));
103			exec($command, $result);
104			}
105
106			return $result;
107			}
108			}
109

EvilFreelancer / easyrsa-php

Push — master ( ce1ae9...e9c52d )

Worker::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like