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-04-22 04:29 UTC

TicketVoter::voteOnAttribute() A

↳ Parent: TicketVoter

Complexity

Conditions	3
Paths	3

Size

Total Lines	15
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	3.1406

Importance

Changes

Metric	Value
cc	3
eloc	8
nc	3
nop	3
dl	0
loc	15
ccs	6
cts	8
cp	0.75
crap	3.1406
rs	9.4285
c	0
b	0
f	0

<?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	73	protected function supports($attribute, $subject)
15		{
16	73	if (($attribute !== self::EDIT) && (!$subject instanceof Ticket)) {
17	73	return false;
18		}
19
20	1	return true;
21		}
22
23	1	protected function voteOnAttribute($attribute, $subject, TokenInterface $token)
24		{
25	1	$user = $token->getUser();
26
27	1	if (!$user instanceof User) {
28		return false;
29		}
30		/** @var Ticket $ticket */
31	1	$ticket = $subject;
32
33		switch ($attribute) {
34	1	case self::EDIT:
35	1	return $this->canEdit($ticket, $user);
36		}
37		}
38
39	1	private function canEdit(Ticket $ticket, User $user)
40		{
41	1	if (!$userOrder = $ticket->getUserOrder()) {
42	1	return true;
43		}
44
45	1	return $user === $userOrder->getUser();
46		}
47		}
48

geekhub-php / CheTheatre

Pull Request — master (#147)

TicketVoter::voteOnAttribute() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like