ClientRelatedEntityVoter::voteOnAttribute() - Code Metrics - Inspection of "Updated from github" - Fluxter/SaasProviderBundle - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( c5e0f8...795874 )

unknown

created 2020-11-24 14:08 UTC

ClientRelatedEntityVoter::voteOnAttribute() A

↳ Parent: ClientRelatedEntityVoter

Complexity

Conditions	4
Paths	4

Size

Total Lines	15
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
eloc	7
nc	4
nop	3
dl	0
loc	15
rs	10
c	0
b	0
f	0

<?php

/*
 * This file is part of the ClanManager package.
 * (c) Fluxter <https://fluxter.net/>
 * Found us at <https://clanmanager.net>
 */

namespace Fluxter\SaasProviderBundle\Security;

use Fluxter\SaasProviderBundle\Model\SaasClientRelatedInterface;
use Fluxter\SaasProviderBundle\Service\SaasClientService;
use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
filter:
    dependency_paths: ["lib/*"]
use Symfony\Component\Security\Core\Authorization\Voter\Voter;
filter:
    dependency_paths: ["lib/*"]
use Symfony\Component\Security\Core\Authorization\Voter\VoterInterface;
filter:
    dependency_paths: ["lib/*"]

class ClientRelatedEntityVoter extends Voter
{
    private SaasClientService $_clientService;

    public function __construct(SaasClientService $clientService)
    {
        $this->_clientService = $clientService;
    }

    protected function supports($attribute, $subject)
    {
        if (!$subject instanceof SaasClientRelatedInterface) {
            return false;
        }

        return true;
    }

    /**
     * @param string                     $attribute
     * @param SaasClientRelatedInterface $subject
     *
     * @return void
     */
    protected function voteOnAttribute($attribute, $subject, TokenInterface $token)
    {
        if (!$this->_clientService->getCurrentClient()) {
            return false;

        }

        if ($subject->getClient() == null) {
            return true;

        }

        if ($this->_clientService->getCurrentClient() == $subject->getClient()) {
            return true;

        }

        return false;

    }
}


1			<?php
2
3			/*
4			* This file is part of the ClanManager package.
5			* (c) Fluxter <https://fluxter.net/>
6			* Found us at <https://clanmanager.net>
7			*/
8
9			namespace Fluxter\SaasProviderBundle\Security;
10
11			use Fluxter\SaasProviderBundle\Model\SaasClientRelatedInterface;
12			use Fluxter\SaasProviderBundle\Service\SaasClientService;
13			use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
			0 ignored issues – show Bug introduced 2020-01-11 19:23 UTC by Report Bug Copy Issue Report The type `Symfony\Component\Securi...on\Token\TokenInterface` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
14			use Symfony\Component\Security\Core\Authorization\Voter\Voter;
			0 ignored issues – show Bug introduced 2020-01-11 19:23 UTC by Report Bug Copy Issue Report The type `Symfony\Component\Securi...thorization\Voter\Voter` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
15			use Symfony\Component\Security\Core\Authorization\Voter\VoterInterface;
			0 ignored issues – show Bug introduced 2020-01-11 19:23 UTC by Report Bug Copy Issue Report The type `Symfony\Component\Securi...on\Voter\VoterInterface` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
16
17			class ClientRelatedEntityVoter extends Voter
18			{
19			private SaasClientService $_clientService;
20
21			public function __construct(SaasClientService $clientService)
22			{
23			$this->_clientService = $clientService;
24			}
25
26			protected function supports($attribute, $subject)
27			{
28			if (!$subject instanceof SaasClientRelatedInterface) {
29			return false;
30			}
31
32			return true;
33			}
34
35			/**
36			* @param string $attribute
37			* @param SaasClientRelatedInterface $subject
38			*
39			* @return void
40			*/
41			protected function voteOnAttribute($attribute, $subject, TokenInterface $token)
42			{
43			if (!$this->_clientService->getCurrentClient()) {
44			return false;
			0 ignored issues – show Bug Best Practice introduced 2020-01-13 21:41 UTC by Report Bug Copy Issue Report The expression `return false` returns the type `false` which is incompatible with the documented return type `void`. Loading history...
45			}
46
47			if ($subject->getClient() == null) {
48			return true;
			0 ignored issues – show Bug Best Practice introduced 2020-01-13 21:41 UTC by Report Bug Copy Issue Report The expression `return true` returns the type `true` which is incompatible with the documented return type `void`. Loading history...
49			}
50
51			if ($this->_clientService->getCurrentClient() == $subject->getClient()) {
52			return true;
			0 ignored issues – show Bug Best Practice introduced 2020-01-11 19:23 UTC by Report Bug Copy Issue Report The expression `return true` returns the type `true` which is incompatible with the documented return type `void`. Loading history...
53			}
54
55			return false;
			0 ignored issues – show Bug Best Practice introduced 2020-01-11 19:23 UTC by Report Bug Copy Issue Report The expression `return false` returns the type `false` which is incompatible with the documented return type `void`. Loading history...
56			}
57			}
58

Fluxter / SaasProviderBundle

Push — master ( c5e0f8...795874 )

ClientRelatedEntityVoter::voteOnAttribute() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like