App\Modules\OauthClients\Services\OauthClientService

Complexity

Total Complexity

5

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	revoke()	0	8	1
A	unRevoke()	0	4	1
A	accessTokenExpiredOrRevoked()	0	4	1
A	revokeAccessToken()	0	4	1

<?php namespace App\Modules\OauthClients\Services;

use App\Modules\Core\BaseClasses\BaseService;
use App\Modules\OauthClients\Repositories\OauthClientRepository;

class OauthClientService extends BaseService
{
    /**
     * Init new object.
     *
     * @param   OauthClientRepository $repo
     * @return  void

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

     */
    public function __construct(OauthClientRepository $repo)
    {
        parent::__construct($repo);
    }

    /**
     * Revoke the given client.
     *
     * @param  integer  $clientId
     * @return void
     */
    public function revoke($clientId)

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

    {
        \DB::transaction(function () use ($data) {

The variable $data does not exist. Did you forget to declare it?

            $client = $this->repo->find($clientId);

The variable $clientId does not exist. Did you forget to declare it?

            $this->repo->revokeClientTokens($client);
            $this->repo->save(['id'=> $clientId, 'revoked' => true]);
        });
    }

    /**
     * UnRevoke the given client.
     *
     * @param  integer  $clientId
     * @return void
     */
    public function unRevoke($clientId)
    {
        $this->repo->save(['id'=> $clientId, 'revoked' => false]);
    }

    /**
     * Ensure access token hasn't expired or revoked.
     *
     * @param  string $accessToken
     * @return boolean
     */
    public function accessTokenExpiredOrRevoked($accessToken)
    {
        return $this->oauthClientRepository->accessTokenExpiredOrRevoked($accessToken);

The property oauthClientRepository does not exist. Did you maybe forget to declare it?

    }

    /**
     * Revoke the given access token and all
     * associated refresh tokens.
     *
     * @param  oject $accessToken
     * @return void
     */
    public function revokeAccessToken($accessToken)
    {
        return $this->oauthClientRepository->revokeAccessToken($accessToken);
    }
}