PageVoter::vote() - Code Metrics - Inspection of "Merge pull request #14 from zicht/hotfix/attribute..." - zicht/page-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — release/2.5.x ( 561eb6...9a29e9 )

unknown

created 2017-08-14 13:03 UTC

PageVoter::vote() B

↳ Parent: PageVoter

Complexity

Conditions	5
Paths	5

Size

Total Lines	18
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	30

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	18
ccs	0
cts	9
cp	0
rs	8.8571
cc	5
eloc	9
nc	5
nop	3
crap	30

<?php
/**
 * @author Rik van der Kemp <[email protected]>
 * @copyright Zicht Online <http://www.zicht.nl>
 */
namespace Zicht\Bundle\PageBundle\Security\Voter;

use Symfony\Component\Security\Core\Authorization\Voter\VoterInterface;
use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
use Zicht\Bundle\PageBundle\Model\PageInterface;

/**
 * Votes for pages to be public for anyone.
 */
class PageVoter extends AbstractVoter
{
    /**
     * @{inheritDoc}
     */
    public function supportsClass($class)
    {
        $supportedClass = 'Zicht\Bundle\PageBundle\Entity\Page';

        return $supportedClass === $class || is_subclass_of($class, $supportedClass);

    }

    /**
     * @{inheritDoc}
     */
    public function vote(TokenInterface $token, $object, array $attributes)
    {
        // check if class of this object is supported by this voter
        if (!$this->supportsClass(get_class($object))) {
            return VoterInterface::ACCESS_ABSTAIN;
        }

        foreach ($attributes as $attribute) {
            if (!$this->supportsAttribute($attribute)) {
                continue;
            }

            if ($this->isPublic($object)) {
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                return VoterInterface::ACCESS_GRANTED;
            }
        }
        return VoterInterface::ACCESS_ABSTAIN;
    }


    /**
     * Checks whether the given page is public. If not, it abstains from voting
     *
     * @param PageInterface $page
     * @return bool
     */
    public function isPublic(PageInterface $page)
    {
        return $page->isPublic();
    }
}


1			<?php
2			/**
3			* @author Rik van der Kemp <[email protected]>
4			* @copyright Zicht Online <http://www.zicht.nl>
5			*/
6			namespace Zicht\Bundle\PageBundle\Security\Voter;
7
8			use Symfony\Component\Security\Core\Authorization\Voter\VoterInterface;
9			use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
10			use Zicht\Bundle\PageBundle\Model\PageInterface;
11
12			/**
13			* Votes for pages to be public for anyone.
14			*/
15			class PageVoter extends AbstractVoter
16			{
17			/**
18			* @{inheritDoc}
19			*/
20			public function supportsClass($class)
21			{
22			$supportedClass = 'Zicht\Bundle\PageBundle\Entity\Page';
23
24			return $supportedClass === $class \|\| is_subclass_of($class, $supportedClass);
			0 ignored issues – show Bug introduced 2017-03-15 11:29 UTC by Report Bug Copy Issue Report Due to PHP Bug #53727, `is_subclass_of` might return inconsistent results on some PHP versions if `$supportedClass` can be an interface. If so, you could instead use `ReflectionClass::implementsInterface`. Loading history...
25			}
26
27			/**
28			* @{inheritDoc}
29			*/
30			public function vote(TokenInterface $token, $object, array $attributes)
31			{
32			// check if class of this object is supported by this voter
33			if (!$this->supportsClass(get_class($object))) {
34			return VoterInterface::ACCESS_ABSTAIN;
35			}
36
37			foreach ($attributes as $attribute) {
38			if (!$this->supportsAttribute($attribute)) {
39			continue;
40			}
41
42			if ($this->isPublic($object)) {
			0 ignored issues – show Documentation introduced 2017-03-15 11:29 UTC by Report Bug Copy Issue Report `$object` is of type `object\|null`, but the function expects a `object<Zicht\Bundle\Page...le\Model\PageInterface>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
43			return VoterInterface::ACCESS_GRANTED;
44			}
45			}
46			return VoterInterface::ACCESS_ABSTAIN;
47			}
48
49
50			/**
51			* Checks whether the given page is public. If not, it abstains from voting
52			*
53			* @param PageInterface $page
54			* @return bool
55			*/
56			public function isPublic(PageInterface $page)
57			{
58			return $page->isPublic();
59			}
60			}
61

zicht / page-bundle

Push — release/2.5.x ( 561eb6...9a29e9 )

PageVoter::vote() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like