TicketVoter::voteOnAttribute() - Code Metrics - Inspection of "Api free and reserve ticket" - geekhub-php/CheTheatre - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#147)

unknown

created 2017-05-04 15:00 UTC

TicketVoter::voteOnAttribute() A

↳ Parent: TicketVoter

Complexity

Conditions	3
Paths	3

Size

Total Lines	15
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	8
c	0
b	0
f	0
nc	3
nop	3
dl	0
loc	15
rs	9.4285

<?php

namespace AppBundle\Security;

use AppBundle\Entity\Ticket;
use AppBundle\Entity\User;
use Symfony\Component\Security\Core\Authorization\Voter\Voter;
use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;

class TicketVoter extends Voter
{
    const EDIT = 'edit';

    protected function supports($attribute, $subject)
    {
        if (($attribute !== self::EDIT) && (!$subject instanceof Ticket)) {
            return false;
        }

        return true;
    }

    protected function voteOnAttribute($attribute, $subject, TokenInterface $token)
    {
        $user = $token->getUser();

        if (!$user instanceof User) {
            return false;
        }
        /** @var Ticket $ticket */
        $ticket = $subject;

        switch ($attribute) {
            case self::EDIT:
                return $this->canEdit($ticket, $user);
        }
    }

    private function canEdit(Ticket $ticket, User $user)
    {
        if (!$userOrder = $ticket->getUserOrder()) {
            return true;
        }

        return $user === $userOrder->getUser();
    }
}


1			<?php
2
3			namespace AppBundle\Security;
4
5			use AppBundle\Entity\Ticket;
6			use AppBundle\Entity\User;
7			use Symfony\Component\Security\Core\Authorization\Voter\Voter;
8			use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
9
10			class TicketVoter extends Voter
11			{
12			const EDIT = 'edit';
13
14			protected function supports($attribute, $subject)
15			{
16			if (($attribute !== self::EDIT) && (!$subject instanceof Ticket)) {
17			return false;
18			}
19
20			return true;
21			}
22
23			protected function voteOnAttribute($attribute, $subject, TokenInterface $token)
24			{
25			$user = $token->getUser();
26
27			if (!$user instanceof User) {
28			return false;
29			}
30			/** @var Ticket $ticket */
31			$ticket = $subject;
32
33			switch ($attribute) {
34			case self::EDIT:
35			return $this->canEdit($ticket, $user);
36			}
37			}
38
39			private function canEdit(Ticket $ticket, User $user)
40			{
41			if (!$userOrder = $ticket->getUserOrder()) {
42			return true;
43			}
44
45			return $user === $userOrder->getUser();
46			}
47			}
48

geekhub-php / CheTheatre

Pull Request — master (#147)

TicketVoter::voteOnAttribute() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like