Component::getCacheKey() - Code Metrics - Inspection of "Missing use statement!" - tylercd100/lern - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — fix/56 ( d15d50...729cbd )

by Tyler

created 2018-07-03 20:33 UTC

Component::getCacheKey() A

↳ Parent: Component

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	4
ccs	2
cts	2
cp	1
rs	10
cc	1
nc	1
nop	1
crap	1

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

tylercd100 / lern

Push — fix/56 ( d15d50...729cbd )

Component::getCacheKey() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like