WeDevBr\Bankly\Auth\Auth

Complexity

Total Complexity

12

Size/Duplication

Total Lines	121
Duplicated Lines	12.4 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	login()	0	12	3
A	setClientCredentials()	0	6	1
A	setClientId()	0	5	1
A	setClientSecret()	0	5	1
A	setGrantType()	0	5	1
A	getToken()	0	8	3
A	auth()	15	15	1

<?php

namespace WeDevBr\Bankly\Auth;

use Illuminate\Support\Facades\Http;

/**
 * Class Auth
 *
 * @author Rafael Teixeira <rafael.teixeira@wedev.software>
 * @package WeDevBr\Bankly
 */
class Auth

Since you have declared the constructor as private, maybe you should also declare the class as final.

{
    /** @var self */
    private static $login;

    /** @var string */
    protected $loginUrl;

    /** @var string */
    private $clientId;

    /** @var string */
    private $clientSecret;

    /** @var string */
    protected $grantType = 'client_credentials';

    /** @var string */
    private $token;

    /** @var string */
    private $tokenExpiry;

    /**
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    private function __construct()
    {
        //
    }

    /**
     * Returns the instance of this class
     *
     * @param string|null $loginUrl
     * @return self
     */
    public static function login($loginUrl = null)
    {
        if (is_null(self::$login)) {
            self::$login = new Auth();
        }

        if (is_null($loginUrl)) {
            self::$login->loginUrl = config('bankly')['login_url'];
        }

        return self::$login;
    }

    /**
     * @return self
     */
    public function setClientCredentials()
    {
        $this->clientId = $this->clientId ?? config('bankly')['client_id'];
        $this->clientSecret = $this->clientSecret ?? config('bankly')['client_secret'];
        return $this;
    }

    /**
     * @param null|string $clientId
     * @return self
     */
    public function setClientId($clientId)
    {
        $this->clientId = $clientId;
        return $this;
    }

    /**
     * @param null|string $clientSecret
     * @return self
     */
    public function setClientSecret($clientSecret)
    {
        $this->clientSecret = $clientSecret;
        return $this;
    }

    /**
     * @param string $grantType
     * @return self
     */
    public function setGrantType(string $grantType)
    {
        $this->grantType = $grantType;
        return $this;
    }

    /**
     * @return string
     */
    public function getToken()
    {
        if (now()->unix() > $this->tokenExpiry || !$this->token) {
            $this->auth();
        }

        return $this->token;
    }

    /**
     * @return void
     */
    private function auth(): void

This method seems to be duplicated in your project.

    {
        $this->setClientCredentials();

        //TODO: Add auth for username and password
        $body = [
            'grant_type' => $this->grantType,
            'client_secret' => $this->clientSecret,
            'client_id' => $this->clientId
        ];

        $response = Http::asForm()->post($this->loginUrl, $body)->throw()->json();
        $this->token = $response['access_token'];
        $this->tokenExpiry = now()->addSeconds($response['expires_in'])->unix();
    }
}