ApiExclusionStrategy::shouldSkipClass() - Code Metrics - Inspection of "Api core new tests" - kamilabs/api-core-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#47)

by Iakov

created 2018-11-06 12:29 UTC

ApiExclusionStrategy::shouldSkipClass() A

↳ Parent: ApiExclusionStrategy

Complexity

Conditions	3
Paths	2

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

Metric	Value
eloc	4
dl	0
loc	8
ccs	0
cts	5
cp	0
rs	10
c	0
b	0
f	0
cc	3
nc	2
nop	2
crap	12

<?php


namespace Kami\ApiCoreBundle\Bridge\JmsSerializer\Exclusion;

use Doctrine\Common\Collections\Collection;
use JMS\Serializer\Context;
use JMS\Serializer\Exclusion\ExclusionStrategyInterface;
use JMS\Serializer\Metadata\ClassMetadata;
use JMS\Serializer\Metadata\PropertyMetadata;
use Kami\ApiCoreBundle\Model\Pageable;
use Kami\ApiCoreBundle\Model\PageRequest;
use Kami\ApiCoreBundle\Security\AccessManager;

class ApiExclusionStrategy implements ExclusionStrategyInterface
{
    private $accessManager;

    public function __construct(AccessManager $accessManager)
    {
        $this->accessManager = $accessManager;
    }

    public function shouldSkipClass(ClassMetadata $metadata, Context $context)
    {
        if (in_array($metadata->reflection->getName(), [Pageable::class, PageRequest::class])
            || $metadata->reflection->implementsInterface(Collection::class)) {
            return false;
        }

        return !$this->accessManager->canAccessResource($metadata->reflection);
    }

    public function shouldSkipProperty(PropertyMetadata $property, Context $context)
    {
        if (in_array($property->class, [Pageable::class, PageRequest::class])) {
            return false;
        }

        return !$this->accessManager->canAccessProperty($property->reflection);
    }

}


1		<?php
2
3
4		namespace Kami\ApiCoreBundle\Bridge\JmsSerializer\Exclusion;
5
6		use Doctrine\Common\Collections\Collection;
7		use JMS\Serializer\Context;
8		use JMS\Serializer\Exclusion\ExclusionStrategyInterface;
9		use JMS\Serializer\Metadata\ClassMetadata;
10		use JMS\Serializer\Metadata\PropertyMetadata;
11		use Kami\ApiCoreBundle\Model\Pageable;
12		use Kami\ApiCoreBundle\Model\PageRequest;
13		use Kami\ApiCoreBundle\Security\AccessManager;
14
15		class ApiExclusionStrategy implements ExclusionStrategyInterface
16		{
17		private $accessManager;
18
19	1	public function __construct(AccessManager $accessManager)
20		{
21	1	$this->accessManager = $accessManager;
22	1	}
23
24		public function shouldSkipClass(ClassMetadata $metadata, Context $context)
25		{
26		if (in_array($metadata->reflection->getName(), [Pageable::class, PageRequest::class])
27		\|\| $metadata->reflection->implementsInterface(Collection::class)) {
28		return false;
29		}
30
31		return !$this->accessManager->canAccessResource($metadata->reflection);
32		}
33
34		public function shouldSkipProperty(PropertyMetadata $property, Context $context)
35		{
36		if (in_array($property->class, [Pageable::class, PageRequest::class])) {
37		return false;
38		}
39
40		return !$this->accessManager->canAccessProperty($property->reflection);
41		}
42
43		}
44

kamilabs / api-core-bundle

Pull Request — master (#47)

ApiExclusionStrategy::shouldSkipClass() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like