Component::shouldntHandle() - Code Metrics - Inspection of "Now uses cache default config setting" - tylercd100/lern - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — fix/56 ( f6593f...d15d50 )

by Tyler

created 2018-07-03 20:31 UTC

Component::shouldntHandle() A

↳ Parent: Component

Complexity

Conditions	5
Paths	5

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	30

Importance

Changes	1
Bugs	0	Features	0

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

<?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::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			protected function shouldntHandle(Exception $e) {
30			$dontHandle = array_merge($this->dontHandle, $this->absolutelyDontHandle);
31
32			foreach ($dontHandle as $type) {
33			if ($e instanceof $type) {
34			return true;
35			}
36			}
37
38			$sent_at = Cache::get($this->getCacheKey($e));
39			if (empty($sent_at) \|\| $sent_at->addSeconds(1)->lte(Carbon::now())) {
40			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			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			protected function getCacheKey(Exception $e)
54			{
55			return "LERN::".static::class."::".get_class($e);
56			}
57			}

tylercd100 / lern

Push — fix/56 ( f6593f...d15d50 )

Component::shouldntHandle() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like