xiaodi\JWTAuth\Manager

Complexity

Total Complexity

5

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	refresh()	0	2	1
A	hasBlacklist()	0	3	1
A	login()	0	2	1
A	logout()	0	3	1

<?php

declare(strict_types=1);

namespace xiaodi\JWTAuth;

use Lcobucci\JWT\Token;
use xiaodi\JWTAuth\Blacklist;

class Manager
{
    private $blacklist;

    public function __construct(Blacklist $blacklist)
    {
        $this->blacklist = $blacklist;
    }

    public function login(Token $token)

The parameter $token is not used and could be removed.

    {
    }

    public function logout(Token $token)
    {
        $this->blacklist->add($token);
    }

    public function refresh(Token $token)

The parameter $token is not used and could be removed.

    {
    }

    /**
     * Undocumented function
     *
     * @param Token $token
     * @return boolean
     */
    public function hasBlacklist(Token $token)
    {
        return $this->blacklist->add($token);

The expression return $this->blacklist->add($token) returns the type void which is incompatible with the documented return type boolean.

Are you sure the usage of $this->blacklist->add($token) targeting xiaodi\JWTAuth\Blacklist::add() seems to always return null.

    }
}