TokenStorageUserProvider::getUser() - Code Metrics - Inspection of "Symfony 5 support" - DamienHarper/DoctrineAuditBundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#134)

by Damien

created 2019-12-02 15:02 UTC

TokenStorageUserProvider::getUser() B

↳ Parent: TokenStorageUserProvider

Complexity

Conditions	10
Paths	48

Size

Total Lines	35
Code Lines	21

Duplication

Lines	0
Ratio	0 %

Importance

Changes	9
Bugs	0	Features	0

Metric	Value
cc	10
eloc	21
c	9
b	0
f	0
nc	48
nop	0
dl	0
loc	35
rs	7.6666

How to fix Complexity

<?php

namespace DH\DoctrineAuditBundle\User;

use Exception;
use Symfony\Component\Security\Core\Authentication\Token\SwitchUserToken;
use Symfony\Component\Security\Core\Role\SwitchUserRole;
use Symfony\Component\Security\Core\Security;
use Symfony\Component\Security\Core\User\UserInterface as BaseUserInterface;

class TokenStorageUserProvider implements UserProviderInterface
{
    private $security;

    public function __construct(Security $security)
    {
        $this->security = $security;
    }

    /**
     * @return null|UserInterface
     */
    public function getUser(): ?UserInterface
    {
        try {
            $token = $this->security->getToken();
        } catch (Exception $e) {
            $token = null;
        }

        if (null === $token) {
            return null;
        }

        $tokenUser = $token->getUser();
        if (!($tokenUser instanceof BaseUserInterface)) {
            return null;
        }

        $impersonation = '';
        if ($this->security->isGranted('ROLE_PREVIOUS_ADMIN')) {
            // Symfony > 4.3
            if ($token instanceof SwitchUserToken) {
                $impersonatorUser = $token->getOriginalToken()->getUser();
            } else {
                $impersonatorUser = $this->getImpersonatorUser();
            }

            if (\is_object($impersonatorUser)) {
                $id = method_exists($impersonatorUser, 'getId') ? $impersonatorUser->getId() : null;
                $username = method_exists($impersonatorUser, 'getUsername') ? $impersonatorUser->getUsername() : (string) $impersonatorUser;
                $impersonation = ' [impersonator '.$username.':'.$id.']';
            }
        }
        $id = method_exists($tokenUser, 'getId') ? $tokenUser->getId() : null;

        return new User($id, $tokenUser->getUsername().$impersonation);
    }

    /**
     * @return null|Security
     */
    public function getSecurity(): ?Security
    {
        return $this->security;
    }

    private function getImpersonatorUser()
    {
        $token = $this->security->getToken();

        // Symfony 5
        if (class_exists('\Symfony\Component\Security\Core\Authentication\Token\SwitchUserToken')) {
            if ($token instanceof SwitchUserToken) {
                return $token->getOriginalToken()->getUser();
            }
        }

        // Pre Symfony 5
        $roles = [];
        if (null !== $token) {
            $roles = method_exists($token, 'getRoleNames') ? $token->getRoleNames() : $token->getRoles();
            $roles = method_exists($token, 'getRoleNames') ? $token->getRoleNames() : $token->/** @scrutinizer ignore-call */ getRoles();
        }

        foreach ($roles as $role) {
            if ($role instanceof SwitchUserRole) {
                return $role->getSource()->getUser();
            }
        }

        return null;
    }
}


DamienHarper / DoctrineAuditBundle

Pull Request — master (#134)

TokenStorageUserProvider::getUser() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like