Gesdinet\JWTRefreshTokenBundle\EventListener\AttachRefreshTokenOnSuccessListener

Complexity

Total Complexity

6

Size/Duplication

Total Lines	64
Duplicated Lines	10.94 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

87.1%

Importance

Changes	9
Bugs	2	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
B	attachRefreshToken()	7	50	5

<?php

/*
 * This file is part of the GesdinetJWTRefreshTokenBundle package.
 *
 * (c) Gesdinet <http://www.gesdinet.com/>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Gesdinet\JWTRefreshTokenBundle\EventListener;

use Gesdinet\JWTRefreshTokenBundle\Model\RefreshTokenManagerInterface;
use Gesdinet\JWTRefreshTokenBundle\Entity\RefreshToken;
use Lexik\Bundle\JWTAuthenticationBundle\Event\AuthenticationSuccessEvent;
use Symfony\Component\Security\Core\User\UserInterface;
use Symfony\Component\Validator\Validator\ValidatorInterface;

class AttachRefreshTokenOnSuccessListener
{
    protected $userRefreshTokenManager;
    protected $ttl;
    protected $validator;

    public function __construct(RefreshTokenManagerInterface $refreshTokenManager, $ttl, ValidatorInterface $validator)
    {
        $this->refreshTokenManager = $refreshTokenManager;

The property refreshTokenManager does not seem to exist. Did you mean userRefreshTokenManager?

        $this->ttl = $ttl;
        $this->validator = $validator;
    }

    public function attachRefreshToken(AuthenticationSuccessEvent $event)
    {
        $data = $event->getData();
        $user = $event->getUser();
        $request = $event->getRequest();

        if (!$user instanceof UserInterface) {
            return;
        }

        $refreshTokenString = null;

$refreshTokenString is not used, you could remove the assignment.

        if ($request->headers->get('content_type') == 'application/json') {

This code seems to be duplicated across your project.

            $content = $request->getContent();
            $params = !empty($content) ? json_decode($content, true) : array();
            $refreshTokenString = trim($params['refresh_token']);
        } else {
            $refreshTokenString = $request->request->get('refresh_token');
        }

        if ($refreshTokenString) {
            $data['refresh_token'] = $refreshTokenString;
        } else {
            $datetime = new \DateTime();
            $datetime->modify('+'.$this->ttl.' seconds');

            $refreshToken = $this->refreshTokenManager->create();

The property refreshTokenManager does not seem to exist. Did you mean userRefreshTokenManager?

            $refreshToken->setUsername($user->getUsername());
            $refreshToken->setRefreshToken();
            $refreshToken->setValid($datetime);

//            $valid = false;

49% of this comment could be valid code. Did you maybe forget this after debugging?

//            while (false === $valid) {
//                $valid = true;
//                $errors = $this->validator->validate($refreshToken);
//                if ($errors->count() > 0) {
//                    foreach ($errors as $error) {
//                        if ('refreshToken' === $error->getPropertyPath()) {
//                            $valid = false;
//                            $refreshToken->setRefreshToken();
//                        }
//                    }
//                }
//            }

            $this->refreshTokenManager->save($refreshToken);

The property refreshTokenManager does not seem to exist. Did you mean userRefreshTokenManager?

            $data['refresh_token'] = $refreshToken->getRefreshToken();
        }

        $event->setData($data);
    }
}