Modules\User\Http\Controllers\Admin\Account\ApiKeysController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	index()	0	8	1
A	create()	0	8	1
A	destroy()	0	8	1

<?php namespace Modules\User\Http\Controllers\Admin\Account;

use Modules\Core\Contracts\Authentication;
use Modules\Core\Http\Controllers\Admin\AdminBaseController;
use Modules\User\Entities\UserToken;
use Modules\User\Repositories\UserTokenRepository;

class ApiKeysController extends AdminBaseController
{
    /**
     * @var Authentication
     */
    private $auth;
    /**
     * @var UserTokenRepository
     */
    private $userToken;

    public function __construct(Authentication $auth, UserTokenRepository $userToken)
    {
        parent::__construct();

        $this->auth = $auth;
        $this->userToken = $userToken;
    }

    public function index()
    {
        $tokens = $this->userToken->allForUser($this->auth->id());

        $this->assetPipeline->requireJs('clipboard.js');

        return view('user::admin.account.api-keys.index', compact('tokens'));
    }

    public function create()
    {
        $this->userToken->generateFor($this->auth->id());

        flash(trans('user:users.token generated'));

It seems like trans('user:users.token generated') targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, flash() does only seem to accept string|null, maybe add an additional type check?

        return redirect()->route('admin.account.api.index');
    }

    public function destroy(UserToken $userToken)
    {
        $this->userToken->destroy($userToken);

        flash(trans('core::core.messages.resource deleted', ['name' => 'Api Token']));

It seems like trans('core::core.messages.resource deleted', array('name' => 'Api Token')) targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, flash() does only seem to accept string|null, maybe add an additional type check?

        return redirect()->route('admin.account.api.index');
    }
}