ProcessException::__construct() - Code Metrics - Inspection of "Updated coverage" - soluble-io/soluble-mediatools - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( e85209...2faa2d )

by Sébastien

created 2018-08-08 10:13 UTC

ProcessException::__construct() A

↳ Parent: ProcessException

Complexity

Conditions	4
Paths	2

Size

Total Lines	20
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	4.0092

Importance

Changes

Metric	Value
cc	4
eloc	12
nc	2
nop	3
dl	0
loc	20
ccs	11
cts	12
cp	0.9167
crap	4.0092
rs	9.8666
c	0
b	0
f	0

<?php

declare(strict_types=1);

namespace Soluble\MediaTools\Common\Exception;

use Symfony\Component\Process\Exception as SPException;
use Symfony\Component\Process\Process;

class ProcessException extends RuntimeException implements ProcessExceptionInterface
{
    /** @var Process */
    protected $process;

    /**
     * ProcessException constructor.
     *
     * @param Process                      $process
     * @param SPException\RuntimeException $previousException
     * @param string|null                  $message           if not set will use the previousException message
     */
    public function __construct(Process $process, SPException\RuntimeException $previousException, string $message = null)
    {
        if ($previousException instanceof SPException\ProcessFailedException ||
            $previousException instanceof SPException\ProcessTimedOutException ||
            $previousException instanceof SPException\ProcessSignaledException
        ) {
            $code = $previousException->getProcess()->getExitCode();
        } else {
            $code = 1;
        }

        $msg = $message ?? $previousException->getMessage();

        parent::__construct(
            $msg,
            $code ?? 1,
            $previousException
        );

        $this->process     = $process;
$a = "a";
$ab = "ab";
$abc = "abc";
    }

    /**
     * Return symfony process object.
     */
    public function getProcess(): Process
    {
        return $this->process;
    }

    public function getErrorOutput(): string
    {
        return $this->process->getErrorOutput();
    }

    /**
     * @return SPException\RuntimeException|SPException\ProcessFailedException|SPException\ProcessSignaledException|SPException\ProcessTimedOutException
     */
    public function getSymfonyProcessRuntimeException(): SPException\RuntimeException
    {
        /**
         * @var SPException\RuntimeException
         */
        $previous = $this->getPrevious();

        return $previous;

    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Soluble\MediaTools\Common\Exception;
6
7		use Symfony\Component\Process\Exception as SPException;
8		use Symfony\Component\Process\Process;
9
10		class ProcessException extends RuntimeException implements ProcessExceptionInterface
11		{
12		/** @var Process */
13		protected $process;
14
15		/**
16		* ProcessException constructor.
17		*
18		* @param Process $process
19		* @param SPException\RuntimeException $previousException
20		* @param string\|null $message if not set will use the previousException message
21		*/
22	10	public function __construct(Process $process, SPException\RuntimeException $previousException, string $message = null)
23		{
24	10	if ($previousException instanceof SPException\ProcessFailedException \|\|
25	5	$previousException instanceof SPException\ProcessTimedOutException \|\|
26	10	$previousException instanceof SPException\ProcessSignaledException
27		) {
28	10	$code = $previousException->getProcess()->getExitCode();
29		} else {
30		$code = 1;
31		}
32
33	10	$msg = $message ?? $previousException->getMessage();
34
35	10	parent::__construct(
36	10	$msg,
37	10	$code ?? 1,
38	10	$previousException
39		);
40
41	10	$this->process = $process;
		0 ignored issues – show Coding Style introduced 2018-07-09 17:00 UTC by Report Bug Copy Issue Report Equals sign not aligned correctly; expected 1 space but found 5 spaces This check looks for improperly formatted assignments. Every assignment must have exactly one space before and one space after the equals operator. To illustrate: $a = "a"; $ab = "ab"; $abc = "abc"; will have no issues, while $a = "a"; $ab = "ab"; $abc = "abc"; will report issues in lines 1 and 2. Loading history...
42	10	}
43
44		/**
45		* Return symfony process object.
46		*/
47	1	public function getProcess(): Process
48		{
49	1	return $this->process;
50		}
51
52	1	public function getErrorOutput(): string
53		{
54	1	return $this->process->getErrorOutput();
55		}
56
57		/**
58		* @return SPException\RuntimeException\|SPException\ProcessFailedException\|SPException\ProcessSignaledException\|SPException\ProcessTimedOutException
59		*/
60	1	public function getSymfonyProcessRuntimeException(): SPException\RuntimeException
61		{
62		/**
63		* @var SPException\RuntimeException
64		*/
65	1	$previous = $this->getPrevious();
66
67	1	return $previous;
		0 ignored issues – show Bug Best Practice introduced 2018-07-05 12:49 UTC by Report Bug Copy Issue Report The expression `return $previous` returns the type `Exception` which includes types incompatible with the type-hinted return `Symfony\Component\Proces...eption\RuntimeException`. Loading history...
68		}
69		}
70

soluble-io / soluble-mediatools

Push — master ( e85209...2faa2d )

ProcessException::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like