RestrictionsCollector::__sleep() - Code Metrics - Inspection of "Fixed `Closure` issue with DataCollector, updated..." - NovikovViktor/RequestLimitBundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( f11675...241e69 )

by Novikov

created 2017-11-29 20:09 UTC

RestrictionsCollector::__sleep() A

↳ Parent: RestrictionsCollector

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace NV\RequestLimitBundle\DataCollector;

use NV\RequestLimitBundle\Storage\StorageManager;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpKernel\DataCollector\DataCollectorInterface;

class RestrictionsCollector implements DataCollectorInterface
{
    /**
     * @var array
     */
    private $data;

    /** @var StorageManager $storageManager */
    private $storageManager;

    /**
     * @inheritdoc
     */
    public function collect(Request $request, Response $response, \Exception $exception = null)
    {
        $this->data = [
            'restrictionsCount' => $this->storageManager->getItemsCount(),
            'restrictions'      => $this->storageManager->fetchAllItems()
        ];
    }

    /**
     * @return array
     */
    public function getRestrictionsCount()
    {
        return $this->data['restrictionsCount'];
    }

    /**
     * @return array
     */
    public function getRestrictions()
    {
        return $this->data['restrictions'];
    }

    /**
     * @return string
     */
    public function getName()
    {
        return 'nv.request_limit.restrictions_collector';
    }

    /**
     * @param StorageManager $manager
     */
    public function setStorageManager(StorageManager $manager)
    {
        $this->storageManager = $manager;
    }

    /**
     * @return array
     */
    function __sleep()

    {
        return ['data'];
    }
}


1			<?php
2
3			namespace NV\RequestLimitBundle\DataCollector;
4
5			use NV\RequestLimitBundle\Storage\StorageManager;
6			use Symfony\Component\HttpFoundation\Request;
7			use Symfony\Component\HttpFoundation\Response;
8			use Symfony\Component\HttpKernel\DataCollector\DataCollectorInterface;
9
10			class RestrictionsCollector implements DataCollectorInterface
11			{
12			/**
13			* @var array
14			*/
15			private $data;
16
17			/** @var StorageManager $storageManager */
18			private $storageManager;
19
20			/**
21			* @inheritdoc
22			*/
23			public function collect(Request $request, Response $response, \Exception $exception = null)
24			{
25			$this->data = [
26			'restrictionsCount' => $this->storageManager->getItemsCount(),
27			'restrictions' => $this->storageManager->fetchAllItems()
28			];
29			}
30
31			/**
32			* @return array
33			*/
34			public function getRestrictionsCount()
35			{
36			return $this->data['restrictionsCount'];
37			}
38
39			/**
40			* @return array
41			*/
42			public function getRestrictions()
43			{
44			return $this->data['restrictions'];
45			}
46
47			/**
48			* @return string
49			*/
50			public function getName()
51			{
52			return 'nv.request_limit.restrictions_collector';
53			}
54
55			/**
56			* @param StorageManager $manager
57			*/
58			public function setStorageManager(StorageManager $manager)
59			{
60			$this->storageManager = $manager;
61			}
62
63			/**
64			* @return array
65			*/
66			function __sleep()
			0 ignored issues – show Best Practice introduced 2017-11-29 20:11 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
67			{
68			return ['data'];
69			}
70			}
71

NovikovViktor / RequestLimitBundle

Push — master ( f11675...241e69 )

RestrictionsCollector::__sleep() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like