Victoire\Bundle\PageBundle\Security\Voter\PageOwnerVoter

Complexity

Total Complexity

7

Size/Duplication

Total Lines	35
Duplicated Lines	25.71 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	3
Bugs	1	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	supports()	0	4	2
A	voteOnAttribute()	9	9	4

<?php

namespace Victoire\Bundle\PageBundle\Security\Voter;

use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
use Symfony\Component\Security\Core\Authorization\Voter\Voter;
use Victoire\Bundle\PageBundle\Entity\Page;

/**
 * This class decides yes or no if the user is granted to do some action on a given page.
 */
class PageOwnerVoter extends Voter
{
    private $userClass;

    /**
     * PageOwnerVoter constructor.
     *
     * @param $userClass
     */
    public function __construct($userClass)
    {
        $this->userClass = $userClass;
    }

    /**
     * {@inheritdoc}
     */
    protected function supports($attribute, $subject)
    {
        return $subject instanceof Page && 'PAGE_OWNER' === $attribute;
    }

    /**
     * {@inheritdoc}
     */
    protected function voteOnAttribute($attribute, $subject, TokenInterface $token)

This method seems to be duplicated in your project.

    {
        return $token->getUser() instanceof $this->userClass
            && (
                $token->getUser()->hasRole('ROLE_VICTOIRE')
             || $token->getUser()->hasRole('ROLE_VICTOIRE_DEVELOPER')
             || $subject->getAuthor() === $token->getUser()
            );
    }
}