Victoire\Bundle\BusinessEntityBundle\Security\Voter\BusinessEntityOwnerVoter

Complexity

Total Complexity

8

Size/Duplication

Total Lines	37
Duplicated Lines	21.62 %

Coupling/Cohesion

Components	2
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	supports()	0	6	3
A	voteOnAttribute()	8	8	4

<?php

namespace Victoire\Bundle\BusinessEntityBundle\Security\Voter;

use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
use Symfony\Component\Security\Core\Authorization\Voter\Voter;
use Victoire\Bundle\BusinessEntityBundle\Helper\BusinessEntityHelper;

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

    /**
     * @param $userClass
     * @param BusinessEntityHelper $businessEntityHelper
     */
    public function __construct($userClass, BusinessEntityHelper $businessEntityHelper)
    {
        $this->userClass = $userClass;
        $this->businessEntityHelper = $businessEntityHelper;
    }

    /**
     * {@inheritdoc}
     */
    protected function supports($attribute, $subject)
    {
        return method_exists($subject, 'getAuthor')
            && $this->businessEntityHelper->findByEntityInstance($subject)
            && 'BUSINESS_ENTITY_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()
            );
    }
}