SMartins\PassportMultiauth\HasMultiAuthApiTokens

Complexity

Total Complexity

2

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Test Coverage

Coverage

5.56%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	createToken()	0	18	1
A	tokens()	0	10	1

<?php

namespace SMartins\PassportMultiauth;

use Illuminate\Container\Container;
use Laravel\Passport\HasApiTokens as BaseHasApiTokens;
use Laravel\Passport\PersonalAccessTokenFactory;
use Laravel\Passport\PersonalAccessTokenResult;
use Laravel\Passport\Token;
use SMartins\PassportMultiauth\Config\AuthConfigHelper;
use SMartins\PassportMultiauth\Exceptions\MissingConfigException;

trait HasMultiAuthApiTokens
{
    use BaseHasApiTokens;

    /**
     * Get all of the access tokens for the user relating with Provider.
     *
     * @return \Illuminate\Database\Eloquent\Collection
     */
    public function tokens()
    {
        return $this->hasMany(Token::class, 'user_id')

It seems like hasMany() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            ->join('oauth_access_token_providers', function ($join) {
                $join->on(
                    'oauth_access_tokens.id', '=', 'oauth_access_token_id'
                )->where('oauth_access_token_providers.provider', '=', AuthConfigHelper::getUserProvider($this));

$this of type SMartins\PassportMultiauth\HasMultiAuthApiTokens is incompatible with the type Illuminate\Contracts\Auth\Authenticatable expected by parameter $user of SMartins\PassportMultiauth\Config\AuthConfigHelper::getUserProvider().

            })->orderBy('created_at', 'desc')
            ->select('oauth_access_tokens.*')
            ->get();
    }

    /**
     * Create a new personal access token for the user and create .
     *
     * @param string $name
     * @param array $scopes
     * @return PersonalAccessTokenResult
     * @throws MissingConfigException
     * @throws \Illuminate\Contracts\Container\BindingResolutionException
     */
    public function createToken($name, array $scopes = [])
    {
        // Backup default provider
        $defaultProvider = config('auth.guards.api.provider');

        $userProvider = AuthConfigHelper::getUserProvider($this);

$this of type SMartins\PassportMultiauth\HasMultiAuthApiTokens is incompatible with the type Illuminate\Contracts\Auth\Authenticatable expected by parameter $user of SMartins\PassportMultiauth\Config\AuthConfigHelper::getUserProvider().

        // Change config to when the token is created set the provider from model creating the token.
        config(['auth.guards.api.provider' => $userProvider]);

        $token = Container::getInstance()->make(PersonalAccessTokenFactory::class)->make(
            $this->getKey(), $name, $scopes

It seems like getKey() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        );

        // Reset config to defaults
        config(['auth.guards.api.provider' => $defaultProvider]);

        return $token;
    }
}