DoesProcessException::processException() - Code Metrics - Inspection of "cleanUP" - alecrabbit/php-traits - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( aa4e87...f3213a )

by Alec

created 2018-12-28 14:53 UTC

DoesProcessException::processException() B

↳ Parent: DoesProcessException

Complexity

Conditions	10
Paths	11

Size

Total Lines	23
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	12
CRAP Score	10

Importance

Changes

Metric	Value
eloc	15
dl	0
loc	23
rs	7.6666
c	0
b	0
f	0
ccs	12
cts	12
cp	1
cc	10
nc	11
nop	1
crap	10

How to fix Complexity

<?php
/**
 * User: alec
 * Date: 28.12.18
 * Time: 15:21
 */

namespace AlecRabbit\Traits;

trait DoesProcessException
{
    /** @var bool */
    protected $throwOnError = true;

    /** @return self */
    public function doNotThrowOnError(): self
    {
        $this->throwOnError = false;
        return $this;
    }

    /** @return self */
    public function throwOnError(): self
    {
        $this->throwOnError = true;
        return $this;
    }

    /** @return bool */
    public function doesThrowsOnError(): bool
    {
        return $this->throwOnError;
    }

    /**
     * @param \Throwable $e
     * @throws \Throwable
     */
    protected function processException(\Throwable $e): void
    {
        if ($this->throwOnError) {
            throw $e;
        }
        if (\defined('APP_DEBUG') && APP_DEBUG) {
            $hasDumpFunction = \function_exists('dump');
            if (\defined('DEBUG_DUMP_EXCEPTION') && DEBUG_DUMP_EXCEPTION) {
                $exceptionMessage = '[' . \get_class($e) . '] ' . $e->getMessage();
                if ($hasDumpFunction) {
                    dump($exceptionMessage, $e->getTraceAsString());
                } else {
                    // @codeCoverageIgnoreStart
                    var_dump($exceptionMessage, $e->getTraceAsString());

                    // @codeCoverageIgnoreEnd
                }
            }
            if (\defined('DEBUG_DUMP_EXCEPTION_CLASS') && DEBUG_DUMP_EXCEPTION_CLASS) {
                if ($hasDumpFunction) {
                    dump($e);
                } else {
                    // @codeCoverageIgnoreStart
                    var_dump($e);
                    // @codeCoverageIgnoreEnd
                }
            }
        }
    }
}


1		<?php
2		/**
3		* User: alec
4		* Date: 28.12.18
5		* Time: 15:21
6		*/
7
8		namespace AlecRabbit\Traits;
9
10		trait DoesProcessException
11		{
12		/** @var bool */
13		protected $throwOnError = true;
14
15		/** @return self */
16	1	public function doNotThrowOnError(): self
17		{
18	1	$this->throwOnError = false;
19	1	return $this;
20		}
21
22		/** @return self */
23	1	public function throwOnError(): self
24		{
25	1	$this->throwOnError = true;
26	1	return $this;
27		}
28
29		/** @return bool */
30	2	public function doesThrowsOnError(): bool
31		{
32	2	return $this->throwOnError;
33		}
34
35		/**
36		* @param \Throwable $e
37		* @throws \Throwable
38		*/
39	2	protected function processException(\Throwable $e): void
40		{
41	2	if ($this->throwOnError) {
42	1	throw $e;
43		}
44	1	if (\defined('APP_DEBUG') && APP_DEBUG) {
45	1	$hasDumpFunction = \function_exists('dump');
46	1	if (\defined('DEBUG_DUMP_EXCEPTION') && DEBUG_DUMP_EXCEPTION) {
47	1	$exceptionMessage = '[' . \get_class($e) . '] ' . $e->getMessage();
48	1	if ($hasDumpFunction) {
49	1	dump($exceptionMessage, $e->getTraceAsString());
50		} else {
51		// @codeCoverageIgnoreStart
52		var_dump($exceptionMessage, $e->getTraceAsString());
		1 ignored issue – show Security Debugging Code introduced 2018-12-28 14:10 UTC by Report Bug Copy Issue Report `var_dump($exceptionMessa...$e->getTraceAsString())` looks like debug code. Are you sure you do not want to remove it? Loading history...
53		// @codeCoverageIgnoreEnd
54		}
55		}
56	1	if (\defined('DEBUG_DUMP_EXCEPTION_CLASS') && DEBUG_DUMP_EXCEPTION_CLASS) {
57	1	if ($hasDumpFunction) {
58	1	dump($e);
59		} else {
60		// @codeCoverageIgnoreStart
61		var_dump($e);
62		// @codeCoverageIgnoreEnd
63		}
64		}
65		}
66	1	}
67		}
68

alecrabbit / php-traits

Push — master ( aa4e87...f3213a )

DoesProcessException::processException() B

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like