Component::shouldntHandle() - Code Metrics - Inspection of "debug" - tylercd100/lern - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — fix/56 ( a550dc...7991aa )

by Tyler

created 2018-07-03 20:19 UTC

Component::shouldntHandle() A

↳ Parent: Component

Complexity

Conditions	5
Paths	5

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	5.025

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	17
ccs	9
cts	10
cp	0.9
rs	9.3888
cc	5
nc	5
nop	1
crap	5.025

<?php

namespace Tylercd100\LERN\Components;

use Exception;
use Illuminate\Support\Facades\Cache;
use Carbon\Carbon;

abstract class Component {

    /**
     * @var array
     */
    protected $dontHandle = [];

    /**
     * This array is overwritten in each component
     * 
     * @var array
     */
    protected $absolutelyDontHandle = [];

    /**
     * Determine if the exception is in the "do not handle" list.
     *
     * @param  \Exception  $e
     * @return bool
     */
    protected function shouldntHandle(Exception $e) {
        $dontHandle = array_merge($this->dontHandle, $this->absolutelyDontHandle);

        foreach ($dontHandle as $type) {
            if ($e instanceof $type) {
                return true;
            }
        }

        $sent_at = Cache::store('file')->get($this->getCacheKey($e));
        if (empty($sent_at) || $sent_at->addSeconds(1)->lte(Carbon::now())) {
            var_dump([$this->getCacheKey($e), $sent_at]);

            return false; // The cache is empty or enough time has passed
        } else {
            return true;
        }
    }

    /**
     * Returns the cache key for the exception with the current component
     * 
     * @param \Exception $e
     * @return string
     */
    protected function getCacheKey(Exception $e)
    {
        return "LERN::".static::class."::".get_class($e);
    }
}

1		<?php
2
3		namespace Tylercd100\LERN\Components;
4
5		use Exception;
6		use Illuminate\Support\Facades\Cache;
7		use Carbon\Carbon;
8
9		abstract class Component {
10
11		/**
12		* @var array
13		*/
14		protected $dontHandle = [];
15
16		/**
17		* This array is overwritten in each component
18		*
19		* @var array
20		*/
21		protected $absolutelyDontHandle = [];
22
23		/**
24		* Determine if the exception is in the "do not handle" list.
25		*
26		* @param \Exception $e
27		* @return bool
28		*/
29	27	protected function shouldntHandle(Exception $e) {
30	27	$dontHandle = array_merge($this->dontHandle, $this->absolutelyDontHandle);
31
32	27	foreach ($dontHandle as $type) {
33	27	if ($e instanceof $type) {
34	27	return true;
35		}
36		}
37
38	24	$sent_at = Cache::store('file')->get($this->getCacheKey($e));
39	24	if (empty($sent_at) \|\| $sent_at->addSeconds(1)->lte(Carbon::now())) {
40	24	var_dump([$this->getCacheKey($e), $sent_at]);
		0 ignored issues – show Security Debugging Code introduced 2018-07-03 20:45 UTC by Report Bug Copy Issue Report `var_dump(array($this->ge...cheKey($e), $sent_at));` looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data. Loading history...
41	24	return false; // The cache is empty or enough time has passed
42		} else {
43		return true;
44		}
45		}
46
47		/**
48		* Returns the cache key for the exception with the current component
49		*
50		* @param \Exception $e
51		* @return string
52		*/
53	24	protected function getCacheKey(Exception $e)
54		{
55	24	return "LERN::".static::class."::".get_class($e);
56		}
57		}

tylercd100 / lern

Push — fix/56 ( a550dc...7991aa )

Component::shouldntHandle() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like