TargetProcessSettingsFromConsoleInput::fromConsoleInput() - Code Metrics - Inspection of "Refactor command-line settings" - sj-i/php-profiler - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#13)

by Shinji

created 2020-08-11 00:44 UTC

fromConsoleInput() A

↳ Parent: TargetProcessSettingsFromConsoleInput

Complexity

Conditions	3
Paths	3

Size

Total Lines	16
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	3
eloc	9
nc	3
nop	1
dl	0
loc	16
rs	9.9666
c	1
b	0
f	0

<?php

/**
 * This file is part of the sj-i/php-profiler package.
 *
 * (c) sji <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace PhpProfiler\Inspector\Settings\TargetProcessSettings;

use PhpProfiler\Inspector\Settings\InspectorSettingsException;
use Symfony\Component\Console\Command\Command;
filter:
    dependency_paths: ["lib/*"]
use Symfony\Component\Console\Input\InputInterface;
filter:
    dependency_paths: ["lib/*"]
use Symfony\Component\Console\Input\InputOption;
filter:
    dependency_paths: ["lib/*"]

final class TargetProcessSettingsFromConsoleInput
{
    public function setOptions(Command $command): void
    {
        $command
            ->addOption(
                'pid',
                'p',
                InputOption::VALUE_REQUIRED,
                'process id'
            )
        ;
    }

    /**
     * @param InputInterface $input
     * @return TargetProcessSettings
     * @throws InspectorSettingsException
     */
    public function fromConsoleInput(InputInterface $input): TargetProcessSettings
    {
        $pid = $input->getOption('pid');
        if (is_null($pid)) {
            throw TargetProcessSettingsException::create(
                TargetProcessSettingsException::PID_NOT_SPECIFIED
            );
        }
        $pid = filter_var($pid, FILTER_VALIDATE_INT);
        if ($pid === false) {
            throw TargetProcessSettingsException::create(
                TargetProcessSettingsException::PID_NOT_SPECIFIED
            );
        }

        return new TargetProcessSettings($pid);
    }
}


1			<?php
2
3			/**
4			* This file is part of the sj-i/php-profiler package.
5			*
6			* (c) sji <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			declare(strict_types=1);
13
14			namespace PhpProfiler\Inspector\Settings\TargetProcessSettings;
15
16			use PhpProfiler\Inspector\Settings\InspectorSettingsException;
17			use Symfony\Component\Console\Command\Command;
			0 ignored issues – show Bug introduced 2020-08-11 00:45 UTC by Report Bug Copy Issue Report The type `Symfony\Component\Console\Command\Command` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
18			use Symfony\Component\Console\Input\InputInterface;
			0 ignored issues – show Bug introduced 2020-06-15 17:31 UTC by Report Bug Copy Issue Report The type `Symfony\Component\Console\Input\InputInterface` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
19			use Symfony\Component\Console\Input\InputOption;
			0 ignored issues – show Bug introduced 2020-08-11 00:45 UTC by Report Bug Copy Issue Report The type `Symfony\Component\Console\Input\InputOption` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
20
21			final class TargetProcessSettingsFromConsoleInput
22			{
23			public function setOptions(Command $command): void
24			{
25			$command
26			->addOption(
27			'pid',
28			'p',
29			InputOption::VALUE_REQUIRED,
30			'process id'
31			)
32			;
33			}
34
35			/**
36			* @param InputInterface $input
37			* @return TargetProcessSettings
38			* @throws InspectorSettingsException
39			*/
40			public function fromConsoleInput(InputInterface $input): TargetProcessSettings
41			{
42			$pid = $input->getOption('pid');
43			if (is_null($pid)) {
44			throw TargetProcessSettingsException::create(
45			TargetProcessSettingsException::PID_NOT_SPECIFIED
46			);
47			}
48			$pid = filter_var($pid, FILTER_VALIDATE_INT);
49			if ($pid === false) {
50			throw TargetProcessSettingsException::create(
51			TargetProcessSettingsException::PID_NOT_SPECIFIED
52			);
53			}
54
55			return new TargetProcessSettings($pid);
56			}
57			}
58

sj-i / php-profiler

Pull Request — master (#13)

fromConsoleInput() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like