rhertogh\Yii2Oauth2Server\controllers\console\PersonalAccessToken\Oauth2GeneratePatAction::run()

Complexity

Conditions	10
Paths	32

Size

Total Lines	72
Code Lines	46

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	110

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function run()
    {
        $module = $this->controller->module;

        $clientIdentifier = $this->controller->client;
        $clientSecret = $this->controller->clientSecret;
        $userId = $this->controller->user;
        $scope = $this->controller->scope;

        if (empty($clientIdentifier)) {
            $clientIdentifier = $this->controller->prompt('Client Identifier?', [
                'required' => true,
                'default' => $defaultClientIdentifier ?? null,

The variable $defaultClientIdentifier seems to never exist and therefore isset should always be false.

                'validator' => function ($input, &$error) {
                    $client = $this->getClient($input);
                    if ($client) {

$client is of type rhertogh\Yii2Oauth2Server\interfaces\models\Oauth2ClientInterface, thus it always evaluated to true.

                        if (!$client->validateGrantType(Oauth2Module::GRANT_TYPE_IDENTIFIER_PERSONAL_ACCESS_TOKEN)) {
                            $error = 'The "Personal Access Token" grant type is not enabled for client "' . $input . '".';
                            return false;
                        }
                    } else {
                        $error = 'No client with identifier "' . $input . '" found.';
                        return false;
                    }
                    return true;
                },
            ]);
        }

        $client = $this->getClient($clientIdentifier);

        if (empty($clientSecret) && $client->isConfidential()) {
            $clientSecret = $this->controller->prompt('Client Secret?', [
                'required' => true,
                'default' => $defaultClientSecret ?? null,

The variable $defaultClientSecret seems to never exist and therefore isset should always be false.

            ]);
        }

        if (empty($userId)) {
            $userId = $this->controller->prompt('User Identifier?', [
                'required' => true,
                'default' => $defaultUserId ?? null,

The variable $defaultUserId seems to never exist and therefore isset should always be false.

                'validator' => function ($input, &$error) {
                    $user = $this->getUser($input);
                    if (!$user) {
                        $error = 'No user with identifier "' . $input . '" found.';
                        return false;
                    }
                    return true;
                },
            ]);
        }

        if (empty($scope)) {
            $scope = $this->controller->prompt('Scope?', [
                'required' => false,
                'default' => $defaultScope ?? null,

The variable $defaultScope seems to never exist and therefore isset should always be false.

            ]);
        }

        $result = $module->generatePersonalAccessToken(
            $clientIdentifier,
            $userId,
            $scope,
            $clientSecret,
        );

        foreach ($result as $key => $value) {
            $this->controller->stdout($key . ': ' . $value . PHP_EOL);
        }

        return ExitCode::OK;
    }