WidgetCache::clear() - Code Metrics - Inspection of "Merge pull request #362 from paulandrieux/feature/..." - Victoire/victoire - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 904db4...e42307 )

by Leny

created 2016-03-16 17:23 UTC

WidgetCache::clear() A

↳ Parent: WidgetCache

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	4
rs	10
cc	1
eloc	2
nc	1
nop	0

<?php

namespace Victoire\Bundle\WidgetBundle\Cache;

use Predis\Client;
use Symfony\Component\Security\Core\Authorization\AuthorizationChecker;
use Victoire\Bundle\WidgetBundle\Entity\Widget;
use Victoire\Bundle\WidgetBundle\Entity\WidgetSlotInterface;
use Victoire\Bundle\WidgetBundle\Helper\WidgetHelper;

/**
 * This class handle the saving of Widgets.
 * Widgets are stored for a week, but are invalidated as soon as
 * the Widget's or BusinessEntity's updatedAt field is changed.
 */
class WidgetCache
{
    /**
     * @var Client
     */
    private $redis;
    /**
     * @var AuthorizationChecker
     */
    private $authorizationChecker;
    /**
     * @var WidgetHelper
     */
    private $widgetHelper;

    /**
     * WidgetCache constructor.
     *
     * @param Client               $redis
     * @param AuthorizationChecker $authorizationChecker
     * @param WidgetHelper         $widgetHelper
     */
    public function __construct(Client $redis, AuthorizationChecker $authorizationChecker, WidgetHelper $widgetHelper)
    {
        $this->redis = $redis;
        $this->authorizationChecker = $authorizationChecker;
        $this->widgetHelper = $widgetHelper;
    }

    /**
     * @param Widget $widget
     *
     * @return string
     */
    public function fetch(Widget $widget)
    {
        return $this->redis->get($this->getHash($widget));
    }

    /**
     * @param Widget $widget
     * @param        $content
     */
    public function save(Widget $widget, $content)
    {
        $hash = $this->getHash($widget);
        if ($hash) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $this->redis->set($hash, $content);
            $this->redis->expire($hash, $this->widgetHelper->getCacheTimeout($widget)); // cache for a week
        }
    }

    /**
     * clear all redis cache.
     */
    public function clear()
    {
        $this->redis->executeCommand($this->redis->createCommand('FLUSHALL'));
    }

    /**
     * @param Widget $widget
     *
     * @return string
     */
    protected function getHash(Widget $widget)
    {
        $hash = null;
        if (!$widget instanceof WidgetSlotInterface) {
            if ($widget->getMode() == Widget::MODE_BUSINESS_ENTITY
                && ($entity = $widget->getEntity())
                && method_exists($widget->getEntity(), 'getUpdatedAt')) {
                $hash = $this->generateBusinessEntityHash($widget);
            } elseif ($widget->getMode() == Widget::MODE_STATIC) {
                $hash = $this->generateHash($widget);
            }
        }

        return $hash;
    }

    protected function generateBusinessEntityHash(Widget $widget)
    {
        return sprintf('%s-%s-%s--%s-%s-%s',
            $widget->getId(),
            $widget->getUpdatedAt()->getTimestamp(),
            $widget->getCurrentView()->getReference()->getId(),
            $widget->getEntity()->getId(),

            $widget->getEntity()->getUpdatedAt()->getTimestamp(),

            (string) $this->authorizationChecker->isGranted('ROLE_VICTOIRE')
        );
    }

    private function generateHash($widget)
    {
        return sprintf('%s-%s-%s-%s',
            $widget->getId(),
            $widget->getUpdatedAt()->getTimestamp(),
            $widget->getCurrentView()->getReference()->getId(),
            (string) $this->authorizationChecker->isGranted('ROLE_VICTOIRE')
        );
    }
}


1			<?php
2
3			namespace Victoire\Bundle\WidgetBundle\Cache;
4
5			use Predis\Client;
6			use Symfony\Component\Security\Core\Authorization\AuthorizationChecker;
7			use Victoire\Bundle\WidgetBundle\Entity\Widget;
8			use Victoire\Bundle\WidgetBundle\Entity\WidgetSlotInterface;
9			use Victoire\Bundle\WidgetBundle\Helper\WidgetHelper;
10
11			/**
12			* This class handle the saving of Widgets.
13			* Widgets are stored for a week, but are invalidated as soon as
14			* the Widget's or BusinessEntity's updatedAt field is changed.
15			*/
16			class WidgetCache
17			{
18			/**
19			* @var Client
20			*/
21			private $redis;
22			/**
23			* @var AuthorizationChecker
24			*/
25			private $authorizationChecker;
26			/**
27			* @var WidgetHelper
28			*/
29			private $widgetHelper;
30
31			/**
32			* WidgetCache constructor.
33			*
34			* @param Client $redis
35			* @param AuthorizationChecker $authorizationChecker
36			* @param WidgetHelper $widgetHelper
37			*/
38			public function __construct(Client $redis, AuthorizationChecker $authorizationChecker, WidgetHelper $widgetHelper)
39			{
40			$this->redis = $redis;
41			$this->authorizationChecker = $authorizationChecker;
42			$this->widgetHelper = $widgetHelper;
43			}
44
45			/**
46			* @param Widget $widget
47			*
48			* @return string
49			*/
50			public function fetch(Widget $widget)
51			{
52			return $this->redis->get($this->getHash($widget));
53			}
54
55			/**
56			* @param Widget $widget
57			* @param $content
58			*/
59			public function save(Widget $widget, $content)
60			{
61			$hash = $this->getHash($widget);
62			if ($hash) {
			0 ignored issues – show Bug Best Practice introduced 2016-02-29 22:25 UTC by Report Bug Copy Issue Report The expression `$hash` of type `string\|null` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
63			$this->redis->set($hash, $content);
64			$this->redis->expire($hash, $this->widgetHelper->getCacheTimeout($widget)); // cache for a week
65			}
66			}
67
68			/**
69			* clear all redis cache.
70			*/
71			public function clear()
72			{
73			$this->redis->executeCommand($this->redis->createCommand('FLUSHALL'));
74			}
75
76			/**
77			* @param Widget $widget
78			*
79			* @return string
80			*/
81			protected function getHash(Widget $widget)
82			{
83			$hash = null;
84			if (!$widget instanceof WidgetSlotInterface) {
85			if ($widget->getMode() == Widget::MODE_BUSINESS_ENTITY
86			&& ($entity = $widget->getEntity())
87			&& method_exists($widget->getEntity(), 'getUpdatedAt')) {
88			$hash = $this->generateBusinessEntityHash($widget);
89			} elseif ($widget->getMode() == Widget::MODE_STATIC) {
90			$hash = $this->generateHash($widget);
91			}
92			}
93
94			return $hash;
95			}
96
97			protected function generateBusinessEntityHash(Widget $widget)
98			{
99			return sprintf('%s-%s-%s--%s-%s-%s',
100			$widget->getId(),
101			$widget->getUpdatedAt()->getTimestamp(),
102			$widget->getCurrentView()->getReference()->getId(),
103			$widget->getEntity()->getId(),
			0 ignored issues – show Bug introduced 2016-03-01 15:58 UTC by Report Bug Copy Issue Report The method `getId` cannot be called on `$widget->getEntity()` (of type `integer\|double`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
104			$widget->getEntity()->getUpdatedAt()->getTimestamp(),
			0 ignored issues – show Bug introduced 2016-03-01 15:58 UTC by Report Bug Copy Issue Report The method `getUpdatedAt` cannot be called on `$widget->getEntity()` (of type `integer\|double`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
105			(string) $this->authorizationChecker->isGranted('ROLE_VICTOIRE')
106			);
107			}
108
109			private function generateHash($widget)
110			{
111			return sprintf('%s-%s-%s-%s',
112			$widget->getId(),
113			$widget->getUpdatedAt()->getTimestamp(),
114			$widget->getCurrentView()->getReference()->getId(),
115			(string) $this->authorizationChecker->isGranted('ROLE_VICTOIRE')
116			);
117			}
118			}
119

Victoire / victoire

Push — master ( 904db4...e42307 )

WidgetCache::clear() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like