CacheAllExcept::shouldCache() - Code Metrics - bramstroker/zf2-fullpage-cache - Measure and Improve Code Quality continuously with Scrutinizer

CacheAllExcept::shouldCache() C
last analyzed 2017-11-20 13:40 UTC

↳ Parent: CacheAllExcept

Complexity

Conditions	19
Paths	82

Size

Total Lines	56
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	56
rs	6.5437
c	0
b	0
f	0
cc	19
eloc	31
nc	82
nop	1

How to fix Long Method Complexity

<?php
/**
 * @author Bram Gerritsen [email protected]
 * @copyright (c) Bram Gerritsen 2013
 * @license http://opensource.org/licenses/mit-license.php
 */

namespace StrokerCache\Strategy;

use Zend\Mvc\MvcEvent;

use StrokerCache\Exception\BadConfigurationException;

class CacheAllExcept extends AbstractStrategy
{
    /**
     * @var array
     */
    protected $except;

    /**
     * {@inheritDoc}
     */
    public function shouldCache(MvcEvent $event)
    {
        $except = $this->getExcept();

        if (!isset($except['namespaces']) && !isset($except['controllers']) && !isset($except['actions'])) {
            throw new BadConfigurationException(
                  "At least one of ['namespaces', 'controllers', 'actions'] keys has to be set in the "
                . "\$config['strokercache']['strategies']['enabled']['" . __CLASS__ . "']['except'][] "
                . "configuration array."
            );
        }

        $routeMatch = $event->getRouteMatch();

        if (null === $routeMatch) {
            return false;
        }

        $controller  = $this->normalize($routeMatch->getParam('controller'));
        $action      = $this->normalize($routeMatch->getParam('action'));

        $shouldCache = true;

        if (true === $shouldCache && isset($except['namespaces'])) {
            foreach ($except['namespaces'] as $exceptNamespace) {
                if (0 === strpos($controller, $this->normalize($exceptNamespace))) {
                    $shouldCache = false;
                    break 1;
                }
            }
        }

        if (true === $shouldCache && isset($except['controllers'])) {
            foreach ($except['controllers'] as $exceptController) {
                if ($controller === $this->normalize($exceptController)) {
                    $shouldCache = false;
                    break 1;
                }
            }
        }

        if (true === $shouldCache && isset($except['actions'])) {
            foreach ($except['actions'] as $exceptController => $exceptActions) {
                if ($controller === $this->normalize($exceptController)) {
                    foreach ($exceptActions as $exceptAction) {
                        if ($action === $this->normalize($exceptAction)) {
                            $shouldCache = false;
                            break 2;
                        }
                    }
                }
            }
        }

        return $shouldCache;
    }

    /**
     * @return array
     */
    public function getExcept()
    {
        return $this->except;
    }

    /**
     * @param array $except
     * @return $this
     */
    public function setExcept(array $except)
    {
        $this->except = $except;

        return $this;
    }

    /**
     * Normalize names before comparing
     *
     * @param string $string
     * @return string
     */
    protected function normalize($string)
    {
        $string = trim($string);

        $string = strtolower($string);

        return $string;
    }
}


1			<?php
2			/**
3			* @author Bram Gerritsen [email protected]
4			* @copyright (c) Bram Gerritsen 2013
5			* @license http://opensource.org/licenses/mit-license.php
6			*/
7
8			namespace StrokerCache\Strategy;
9
10			use Zend\Mvc\MvcEvent;
11
12			use StrokerCache\Exception\BadConfigurationException;
13
14			class CacheAllExcept extends AbstractStrategy
15			{
16			/**
17			* @var array
18			*/
19			protected $except;
20
21			/**
22			* {@inheritDoc}
23			*/
24			public function shouldCache(MvcEvent $event)
25			{
26			$except = $this->getExcept();
27
28			if (!isset($except['namespaces']) && !isset($except['controllers']) && !isset($except['actions'])) {
29			throw new BadConfigurationException(
30			"At least one of ['namespaces', 'controllers', 'actions'] keys has to be set in the "
31			. "\$config['strokercache']['strategies']['enabled']['" . __CLASS__ . "']['except'][] "
32			. "configuration array."
33			);
34			}
35
36			$routeMatch = $event->getRouteMatch();
37
38			if (null === $routeMatch) {
39			return false;
40			}
41
42			$controller = $this->normalize($routeMatch->getParam('controller'));
43			$action = $this->normalize($routeMatch->getParam('action'));
44
45			$shouldCache = true;
46
47			if (true === $shouldCache && isset($except['namespaces'])) {
48			foreach ($except['namespaces'] as $exceptNamespace) {
49			if (0 === strpos($controller, $this->normalize($exceptNamespace))) {
50			$shouldCache = false;
51			break 1;
52			}
53			}
54			}
55
56			if (true === $shouldCache && isset($except['controllers'])) {
57			foreach ($except['controllers'] as $exceptController) {
58			if ($controller === $this->normalize($exceptController)) {
59			$shouldCache = false;
60			break 1;
61			}
62			}
63			}
64
65			if (true === $shouldCache && isset($except['actions'])) {
66			foreach ($except['actions'] as $exceptController => $exceptActions) {
67			if ($controller === $this->normalize($exceptController)) {
68			foreach ($exceptActions as $exceptAction) {
69			if ($action === $this->normalize($exceptAction)) {
70			$shouldCache = false;
71			break 2;
72			}
73			}
74			}
75			}
76			}
77
78			return $shouldCache;
79			}
80
81			/**
82			* @return array
83			*/
84			public function getExcept()
85			{
86			return $this->except;
87			}
88
89			/**
90			* @param array $except
91			* @return $this
92			*/
93			public function setExcept(array $except)
94			{
95			$this->except = $except;
96
97			return $this;
98			}
99
100			/**
101			* Normalize names before comparing
102			*
103			* @param string $string
104			* @return string
105			*/
106			protected function normalize($string)
107			{
108			$string = trim($string);
109
110			$string = strtolower($string);
111
112			return $string;
113			}
114			}
115

bramstroker / zf2-fullpage-cache

GitHub Access Token became invalid

CacheAllExcept::shouldCache() C last analyzed 2017-11-20 13:40 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like

CacheAllExcept::shouldCache() C
last analyzed 2017-11-20 13:40 UTC